From: Daniel Golle <daniel@makrotopia.org>
To: Shayne Chen <shayne.chen@mediatek.com>
Cc: Felix Fietkau <nbd@nbd.name>,
linux-wireless <linux-wireless@vger.kernel.org>,
Lorenzo Bianconi <lorenzo@kernel.org>,
Ryder Lee <ryder.lee@mediatek.com>,
Evelyn Tsai <evelyn.tsai@mediatek.com>,
linux-mediatek <linux-mediatek@lists.infradead.org>,
StanleyYP Wang <StanleyYP.Wang@mediatek.com>
Subject: Re: [PATCH 1/3] wifi: mt76: mt7996: extend flexibility of mt7996_mcu_get_eeprom()
Date: Thu, 26 Sep 2024 15:13:44 +0100 [thread overview]
Message-ID: <ZvVsGKmHHX2q_HRx@makrotopia.org> (raw)
In-Reply-To: <20240926032440.15978-1-shayne.chen@mediatek.com>
On Thu, Sep 26, 2024 at 11:24:38AM +0800, Shayne Chen wrote:
> Support passing customized buffer pointer and length to
> mt7996_mcu_get_eeprom().
>
> This is the preparation for adding more variants support which needs to
> prefetch FEM module from efuse, and also fixes potential OOB issue when
> reading the last efuse block.
>
> Co-developed-by: StanleyYP Wang <StanleyYP.Wang@mediatek.com>
> Signed-off-by: StanleyYP Wang <StanleyYP.Wang@mediatek.com>
> Signed-off-by: Shayne Chen <shayne.chen@mediatek.com>
Tested-by: Daniel Golle <daniel@makrotopia.org>
> ---
> drivers/net/wireless/mediatek/mt76/mt7996/eeprom.c | 9 +++++++--
> drivers/net/wireless/mediatek/mt76/mt7996/mcu.c | 14 ++++++++++----
> drivers/net/wireless/mediatek/mt76/mt7996/mt7996.h | 2 +-
> 3 files changed, 18 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7996/eeprom.c b/drivers/net/wireless/mediatek/mt76/mt7996/eeprom.c
> index 4a8237118287..861aba68a725 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7996/eeprom.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7996/eeprom.c
> @@ -86,8 +86,13 @@ static int mt7996_eeprom_load(struct mt7996_dev *dev)
> /* read eeprom data from efuse */
> block_num = DIV_ROUND_UP(MT7996_EEPROM_SIZE, eeprom_blk_size);
> for (i = 0; i < block_num; i++) {
> - ret = mt7996_mcu_get_eeprom(dev, i * eeprom_blk_size);
> - if (ret < 0)
> + u32 len = eeprom_blk_size;
> +
> + if (i == block_num - 1)
> + len = MT7996_EEPROM_SIZE % eeprom_blk_size;
> + ret = mt7996_mcu_get_eeprom(dev, i * eeprom_blk_size,
> + NULL, len);
> + if (ret && ret != -EINVAL)
> return ret;
> }
> }
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7996/mcu.c b/drivers/net/wireless/mediatek/mt76/mt7996/mcu.c
> index 327337b31279..fa7832f625d7 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7996/mcu.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7996/mcu.c
> @@ -3548,7 +3548,7 @@ int mt7996_mcu_set_eeprom(struct mt7996_dev *dev)
> &req, sizeof(req), true);
> }
>
> -int mt7996_mcu_get_eeprom(struct mt7996_dev *dev, u32 offset)
> +int mt7996_mcu_get_eeprom(struct mt7996_dev *dev, u32 offset, u8 *buf, u32 buf_len)
> {
> struct {
> u8 _rsv[4];
> @@ -3577,15 +3577,21 @@ int mt7996_mcu_get_eeprom(struct mt7996_dev *dev, u32 offset)
> valid = le32_to_cpu(*(__le32 *)(skb->data + 16));
> if (valid) {
> u32 addr = le32_to_cpu(*(__le32 *)(skb->data + 12));
> - u8 *buf = (u8 *)dev->mt76.eeprom.data + addr;
> +
> + if (!buf)
> + buf = (u8 *)dev->mt76.eeprom.data + addr;
> + if (!buf_len || buf_len > MT7996_EEPROM_BLOCK_SIZE)
> + buf_len = MT7996_EEPROM_BLOCK_SIZE;
>
> skb_pull(skb, 48);
> - memcpy(buf, skb->data, MT7996_EEPROM_BLOCK_SIZE);
> + memcpy(buf, skb->data, buf_len);
> + } else {
> + ret = -EINVAL;
> }
>
> dev_kfree_skb(skb);
>
> - return 0;
> + return ret;
> }
>
> int mt7996_mcu_get_eeprom_free_block(struct mt7996_dev *dev, u8 *block_num)
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7996/mt7996.h b/drivers/net/wireless/mediatek/mt76/mt7996/mt7996.h
> index ab8c9070630b..55aa5f6ab77d 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7996/mt7996.h
> +++ b/drivers/net/wireless/mediatek/mt76/mt7996/mt7996.h
> @@ -476,7 +476,7 @@ int mt7996_mcu_set_fixed_rate_ctrl(struct mt7996_dev *dev,
> int mt7996_mcu_set_fixed_field(struct mt7996_dev *dev, struct ieee80211_vif *vif,
> struct ieee80211_sta *sta, void *data, u32 field);
> int mt7996_mcu_set_eeprom(struct mt7996_dev *dev);
> -int mt7996_mcu_get_eeprom(struct mt7996_dev *dev, u32 offset);
> +int mt7996_mcu_get_eeprom(struct mt7996_dev *dev, u32 offset, u8 *buf, u32 buf_len);
> int mt7996_mcu_get_eeprom_free_block(struct mt7996_dev *dev, u8 *block_num);
> int mt7996_mcu_get_chip_config(struct mt7996_dev *dev, u32 *cap);
> int mt7996_mcu_set_ser(struct mt7996_dev *dev, u8 action, u8 set, u8 band);
> --
> 2.39.2
>
prev parent reply other threads:[~2024-09-26 14:27 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-26 3:24 [PATCH 1/3] wifi: mt76: mt7996: extend flexibility of mt7996_mcu_get_eeprom() Shayne Chen
2024-09-26 3:24 ` [PATCH 2/3] wifi: mt76: mt7996: add suuport for more variants Shayne Chen
2024-09-26 14:14 ` Daniel Golle
2024-09-26 3:24 ` [PATCH 3/3] wifi: mt76: mt7996: set correct background radar capability Shayne Chen
2024-09-26 14:14 ` Daniel Golle
2024-09-26 14:13 ` Daniel Golle [this message]
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=ZvVsGKmHHX2q_HRx@makrotopia.org \
--to=daniel@makrotopia.org \
--cc=StanleyYP.Wang@mediatek.com \
--cc=evelyn.tsai@mediatek.com \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-wireless@vger.kernel.org \
--cc=lorenzo@kernel.org \
--cc=nbd@nbd.name \
--cc=ryder.lee@mediatek.com \
--cc=shayne.chen@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).