From: Sean Wang <sean.wang@kernel.org>
To: "Quan Zhou (周全)" <Quan.Zhou@mediatek.com>
Cc: "linux-mediatek@lists.infradead.org"
<linux-mediatek@lists.infradead.org>,
"Shengxi Xu (徐胜喜)" <shengxi.xu@mediatek.com>,
"Deren Wu (武德仁)" <Deren.Wu@mediatek.com>,
"Shayne Chen (陳軒丞)" <Shayne.Chen@mediatek.com>,
"nbd@nbd.name" <nbd@nbd.name>,
"lorenzo@kernel.org" <lorenzo@kernel.org>,
"Sean Wang" <Sean.Wang@mediatek.com>,
"Hao Zhang (张浩)" <hao.zhang@mediatek.com>,
"Ryder Lee" <Ryder.Lee@mediatek.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH v2] wifi: mt76: mt7921e: add LED control support
Date: Wed, 17 Apr 2024 19:50:42 -0700 [thread overview]
Message-ID: <CAGp9Lzq6g8YQ=_eZHJX4qQjv-F2anGnPw8f88rHD2Vo1iwiGWg@mail.gmail.com> (raw)
In-Reply-To: <37ab8a32893ea9a1cbc33b6dae26b57127ea4e16.camel@mediatek.com>
On Wed, Apr 17, 2024 at 6:43 PM Quan Zhou (周全) <Quan.Zhou@mediatek.com> wrote:
>
> On Wed, 2024-04-17 at 15:06 -0700, Sean Wang wrote:
> >
> > External email : Please do not click links or open attachments until
> > you have verified the sender or the content.
> > HI Quan,
> >
> > On Wed, Apr 10, 2024 at 11:00 PM Quan Zhou <quan.zhou@mediatek.com>
> > wrote:
> > >
> > > From: Hao Zhang <hao.zhang@mediatek.com>
> > >
> > > Introduce wifi LED switch control, add flow to Control a wifi
> > > gpio pin based on the status of WIFI radio, if the pin is connected
> > > to an LED, the LED will indicate the status of the WiFi radio.
> > >
> > > Signed-off-by: Hao Zhang <hao.zhang@mediatek.com>
> > > Co-developed-by: Quan Zhou <quan.zhou@mediatek.com>
> > > Signed-off-by: Quan Zhou <quan.zhou@mediatek.com>
> > > ---
> > > v2:
> > > fix to avoid wake device when Hardware interface not pcie
> > > ---
> > > .../wireless/mediatek/mt76/mt76_connac_mcu.h | 1 +
> > > .../net/wireless/mediatek/mt76/mt7921/main.c | 27
> > ++++++++++++++++++-
> > > .../net/wireless/mediatek/mt76/mt7921/mcu.c | 14 ++++++++++
> > > .../wireless/mediatek/mt76/mt7921/mt7921.h | 5 ++++
> > > .../net/wireless/mediatek/mt76/mt7921/pci.c | 8 +++++-
> > > 5 files changed, 53 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> > b/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> > > index 836cc4d5b1d2..4c2de556dee1 100644
> > > --- a/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> > > +++ b/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> > > @@ -1189,6 +1189,7 @@ enum {
> > > MCU_EXT_CMD_EFUSE_ACCESS = 0x01,
> > > MCU_EXT_CMD_RF_REG_ACCESS = 0x02,
> > > MCU_EXT_CMD_RF_TEST = 0x04,
> > > + MCU_EXT_CMD_ID_RADIO_ON_OFF_CTRL = 0x05,
> > > MCU_EXT_CMD_PM_STATE_CTRL = 0x07,
> > > MCU_EXT_CMD_CHANNEL_SWITCH = 0x08,
> > > MCU_EXT_CMD_SET_TX_POWER_CTRL = 0x11,
> > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/main.c
> > b/drivers/net/wireless/mediatek/mt76/mt7921/main.c
> > > index ca36de34171b..ea6a113b7b36 100644
> > > --- a/drivers/net/wireless/mediatek/mt76/mt7921/main.c
> > > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/main.c
> > > @@ -242,6 +242,15 @@ int __mt7921_start(struct mt792x_phy *phy)
> > >
> > > ieee80211_queue_delayed_work(mphy->hw, &mphy->mac_work,
> > > MT792x_WATCHDOG_TIME);
> > > + if (mt76_is_mmio(mphy->dev)) {
> >
> > I guess the led control MCU command is not limited to PCIe devices,
> > they should be able to be extended even on MT7921 USB and SDIO
> > devices, right ? if so, I think we can drop the MMIO limitation
> > condition to support more scenarios and to make it easier to
> > understand.
> >
> Hi Sean,
>
> This software flow involves chip GPIO control and is related to the
> module's circuit design. Only the PCIe module can provide support for
> this, so can't drop.
Hi Quan,
Thanks for clearing that up quickly. I guess we can add it just for
MT7921E. I have another question: Will the new command you added work
with older firmware, or is it made only for the most recent firmware?
I'm worried it might not be compatible with the older MT7921 firmware.
Sean
>
> > > + err = mt7921_mcu_radio_led_ctrl(phy->dev,
> > EXT_CMD_RADIO_LED_CTRL_ENABLE);
> > > + if (err)
> > > + return err;
> > > +
> > > + err = mt7921_mcu_radio_led_ctrl(phy->dev,
> > EXT_CMD_RADIO_ON_LED);
> > > + if (err)
> > > + return err;
> > > + }
> > >
> > > return 0;
> > > }
> > > @@ -259,6 +268,22 @@ static int mt7921_start(struct ieee80211_hw
> > *hw)
> > > return err;
> > > }
> > >
> > > +static void mt7921_stop(struct ieee80211_hw *hw)
> > > +{
> > > + struct mt792x_dev *dev = mt792x_hw_dev(hw);
> > > + int err = 0;
> > > +
> > > + if (mt76_is_mmio(&dev->mt76)) {
> > > + mt792x_mutex_acquire(dev);
> > > + err = mt7921_mcu_radio_led_ctrl(dev,
> > EXT_CMD_RADIO_OFF_LED);
> > > + mt792x_mutex_release(dev);
> > > + if (err)
> > > + return;
> > > + }
> > > +
> > > + mt792x_stop(hw);
> > > +}
> > > +
> > > static int
> > > mt7921_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif
> > *vif)
> > > {
> > > @@ -1372,7 +1397,7 @@ static void mt7921_mgd_complete_tx(struct
> > ieee80211_hw *hw,
> > > const struct ieee80211_ops mt7921_ops = {
> > > .tx = mt792x_tx,
> > > .start = mt7921_start,
> > > - .stop = mt792x_stop,
> > > + .stop = mt7921_stop,
> > > .add_interface = mt7921_add_interface,
> > > .remove_interface = mt792x_remove_interface,
> > > .config = mt7921_config,
> > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
> > b/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
> > > index 8b4ce32a2cd1..2ebf0ffe78d5 100644
> > > --- a/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
> > > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
> > > @@ -606,6 +606,20 @@ int mt7921_run_firmware(struct mt792x_dev
> > *dev)
> > > }
> > > EXPORT_SYMBOL_GPL(mt7921_run_firmware);
> > >
> > > +int mt7921_mcu_radio_led_ctrl(struct mt792x_dev *dev, u8 value)
> > > +{
> > > + struct {
> > > + u8 ctrlid;
> > > + u8 rsv[3];
> > > + } __packed req = {
> > > + .ctrlid = value,
> > > + };
> > > +
> > > + return mt76_mcu_send_msg(&dev->mt76,
> > MCU_EXT_CMD(ID_RADIO_ON_OFF_CTRL),
> > > + &req, sizeof(req), false);
> > > +}
> > > +EXPORT_SYMBOL_GPL(mt7921_mcu_radio_led_ctrl);
> > > +
> > > int mt7921_mcu_set_tx(struct mt792x_dev *dev, struct ieee80211_vif
> > *vif)
> > > {
> > > struct mt792x_vif *mvif = (struct mt792x_vif *)vif-
> > >drv_priv;
> > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> > b/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> > > index 3016636d18c6..07023eb9e5b5 100644
> > > --- a/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> > > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> > > @@ -27,6 +27,10 @@
> > > #define MCU_UNI_EVENT_ROC 0x27
> > > #define MCU_UNI_EVENT_CLC 0x80
> > >
> > > +#define EXT_CMD_RADIO_LED_CTRL_ENABLE 0x1
> > > +#define EXT_CMD_RADIO_ON_LED 0x2
> > > +#define EXT_CMD_RADIO_OFF_LED 0x3
> > > +
> > > enum {
> > > UNI_ROC_ACQUIRE,
> > > UNI_ROC_ABORT,
> > > @@ -196,6 +200,7 @@ int mt7921_mcu_fw_log_2_host(struct mt792x_dev
> > *dev, u8 ctrl);
> > > void mt7921_mcu_rx_event(struct mt792x_dev *dev, struct sk_buff
> > *skb);
> > > int mt7921_mcu_set_rxfilter(struct mt792x_dev *dev, u32 fif,
> > > u8 bit_op, u32 bit_map);
> > > +int mt7921_mcu_radio_led_ctrl(struct mt792x_dev *dev, u8 value);
> > >
> > > static inline u32
> > > mt7921_reg_map_l1(struct mt792x_dev *dev, u32 addr)
> > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > > index 0b69b225bc16..f768e9389ac6 100644
> > > --- a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > > @@ -427,6 +427,10 @@ static int mt7921_pci_suspend(struct device
> > *device)
> > > wait_event_timeout(dev->wait,
> > > !dev->regd_in_progress, 5 * HZ);
> > >
> > > + err = mt7921_mcu_radio_led_ctrl(dev,
> > EXT_CMD_RADIO_OFF_LED);
> > > + if (err < 0)
> > > + goto restore_suspend;
> > > +
> > > err = mt76_connac_mcu_set_hif_suspend(mdev, true);
> > > if (err)
> > > goto restore_suspend;
> > > @@ -525,9 +529,11 @@ static int mt7921_pci_resume(struct device
> > *device)
> > > mt76_connac_mcu_set_deep_sleep(&dev->mt76, false);
> > >
> > > err = mt76_connac_mcu_set_hif_suspend(mdev, false);
> > > + if (err < 0)
> > > + goto failed;
> > >
> > > mt7921_regd_update(dev);
> > > -
> > > + err = mt7921_mcu_radio_led_ctrl(dev, EXT_CMD_RADIO_ON_LED);
> > > failed:
> > > pm->suspended = false;
> > >
> > > --
> > > 2.18.0
> > >
> > >
next prev parent reply other threads:[~2024-04-18 2:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-11 5:41 [PATCH v2] wifi: mt76: mt7921e: add LED control support Quan Zhou
2024-04-11 7:42 ` Lorenzo Bianconi
2024-04-17 22:06 ` Sean Wang
2024-04-18 1:43 ` Quan Zhou (周全)
2024-04-18 2:50 ` Sean Wang [this message]
2024-04-18 5:48 ` Quan Zhou (周全)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAGp9Lzq6g8YQ=_eZHJX4qQjv-F2anGnPw8f88rHD2Vo1iwiGWg@mail.gmail.com' \
--to=sean.wang@kernel.org \
--cc=Deren.Wu@mediatek.com \
--cc=Quan.Zhou@mediatek.com \
--cc=Ryder.Lee@mediatek.com \
--cc=Sean.Wang@mediatek.com \
--cc=Shayne.Chen@mediatek.com \
--cc=hao.zhang@mediatek.com \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-wireless@vger.kernel.org \
--cc=lorenzo@kernel.org \
--cc=nbd@nbd.name \
--cc=shengxi.xu@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).