linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: <sean.wang@mediatek.com>
To: <nbd@nbd.name>, <lorenzo.bianconi@redhat.com>
Cc: <sean.wang@mediatek.com>, <Soul.Huang@mediatek.com>,
	<YN.Chen@mediatek.com>, <Leon.Yen@mediatek.com>,
	<Eric-SY.Chang@mediatek.com>, <Mark-YW.Chen@mediatek.com>,
	<Deren.Wu@mediatek.com>, <km.lin@mediatek.com>,
	<robin.chiu@mediatek.com>, <Eddie.Chen@mediatek.com>,
	<ch.yeh@mediatek.com>, <posh.sun@mediatek.com>,
	<ted.huang@mediatek.com>, <Eric.Liang@mediatek.com>,
	<Stella.Chang@mediatek.com>, <Tom.Chou@mediatek.com>,
	<steve.lee@mediatek.com>, <jsiuda@google.com>,
	<frankgor@google.com>, <jemele@google.com>,
	<abhishekpandit@google.com>, <shawnku@google.com>,
	<linux-wireless@vger.kernel.org>,
	<linux-mediatek@lists.infradead.org>
Subject: [PATCH v5 08/17] mt76: mt7921: make all event parser reusable between mt7921s and mt7921e
Date: Tue, 19 Oct 2021 07:11:38 +0800	[thread overview]
Message-ID: <a2f85e705fdb9e6c9b59c5b68d2f166cfe844f10.1634598341.git.objelf@gmail.com> (raw)
In-Reply-To: <cover.1634598341.git.objelf@gmail.com>

From: Sean Wang <sean.wang@mediatek.com>

The longer event such as the event for mcu_get_nic_capability would hold
the data in paged fragment skb for the SDIO device so we turn the skb to
be linearized skb before accessing it to reuse the same event parser
betweem mt7921s and mt7921e.

Tested-by: Deren Wu <deren.wu@mediatek.com>
Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
---
 drivers/net/wireless/mediatek/mt76/mt7921/mcu.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c b/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
index 10b992103ade..e315eb4186d3 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/mcu.c
@@ -455,7 +455,12 @@ mt7921_mcu_rx_unsolicited_event(struct mt7921_dev *dev, struct sk_buff *skb)
 
 void mt7921_mcu_rx_event(struct mt7921_dev *dev, struct sk_buff *skb)
 {
-	struct mt7921_mcu_rxd *rxd = (struct mt7921_mcu_rxd *)skb->data;
+	struct mt7921_mcu_rxd *rxd;
+
+	if (skb_linearize(skb))
+		return;
+
+	rxd = (struct mt7921_mcu_rxd *)skb->data;
 
 	if (rxd->eid == 0x6) {
 		mt76_mcu_rx_event(&dev->mt76, skb);
-- 
2.25.1


  parent reply	other threads:[~2021-10-18 23:13 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-18 23:11 [PATCH v5 00/17] Add MT7921 SDIO WiFi support sean.wang
2021-10-18 23:11 ` [PATCH v5 01/17] mt76: mt7921: refactor mac.c to be bus independent sean.wang
2021-10-18 23:11 ` [PATCH v5 02/17] mt76: mt7921: refactor dma.c to be pcie specific sean.wang
2021-10-18 23:11 ` [PATCH v5 03/17] mt76: mt7921: refactor mcu.c to be bus independent sean.wang
2021-10-18 23:11 ` [PATCH v5 04/17] mt76: mt7921: refactor init.c " sean.wang
2021-10-18 23:11 ` [PATCH v5 05/17] mt76: mt7921: add MT7921_COMMON module sean.wang
2021-10-18 23:11 ` [PATCH v5 06/17] mt76: connac: move mcu reg access utility routines in mt76_connac_lib module sean.wang
2021-10-18 23:11 ` [PATCH v5 07/17] mt76: mt7663s: rely on mcu reg access utility sean.wang
2021-10-18 23:11 ` sean.wang [this message]
2021-10-18 23:11 ` [PATCH v5 09/17] mt76: mt7921: use physical addr to unify register access sean.wang
2021-10-18 23:11 ` [PATCH v5 10/17] mt76: sdio: move common code in mt76_sdio module sean.wang
2021-10-18 23:11 ` [PATCH v5 11/17] mt76: sdio: introduce parse_irq callback sean.wang
2021-10-18 23:11 ` [PATCH v5 12/17] mt76: sdio: extend sdio module to support CONNAC2 sean.wang
2021-10-18 23:11 ` [PATCH v5 13/17] mt76: connac: extend mcu_get_nic_capability sean.wang
2021-10-18 23:11 ` [PATCH v5 14/17] mt76: mt7921: rely on mcu_get_nic_capability sean.wang
2021-10-18 23:11 ` [PATCH v5 15/17] mt76: mt7921: refactor mt7921_mcu_send_message sean.wang
2021-10-18 23:11 ` [PATCH v5 16/17] mt76: mt7921: introduce mt7921s support sean.wang
2021-10-18 23:11 ` [PATCH v5 17/17] mt76: mt7921s: add reset support sean.wang

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=a2f85e705fdb9e6c9b59c5b68d2f166cfe844f10.1634598341.git.objelf@gmail.com \
    --to=sean.wang@mediatek.com \
    --cc=Deren.Wu@mediatek.com \
    --cc=Eddie.Chen@mediatek.com \
    --cc=Eric-SY.Chang@mediatek.com \
    --cc=Eric.Liang@mediatek.com \
    --cc=Leon.Yen@mediatek.com \
    --cc=Mark-YW.Chen@mediatek.com \
    --cc=Soul.Huang@mediatek.com \
    --cc=Stella.Chang@mediatek.com \
    --cc=Tom.Chou@mediatek.com \
    --cc=YN.Chen@mediatek.com \
    --cc=abhishekpandit@google.com \
    --cc=ch.yeh@mediatek.com \
    --cc=frankgor@google.com \
    --cc=jemele@google.com \
    --cc=jsiuda@google.com \
    --cc=km.lin@mediatek.com \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=lorenzo.bianconi@redhat.com \
    --cc=nbd@nbd.name \
    --cc=posh.sun@mediatek.com \
    --cc=robin.chiu@mediatek.com \
    --cc=shawnku@google.com \
    --cc=steve.lee@mediatek.com \
    --cc=ted.huang@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 NNTP newsgroup(s).