MIRRORS/linux-yocto
1
0
Fork 0
mirror of git://git.yoctoproject.org/linux-yocto.git synced 2025-07-06 13:55:22 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

wifi: mt76: mt7915: retry mcu messages

Browse Source 
In some cases MCU messages can get lost. Instead of failing completely,
attempt to recover by re-sending them.

Link: https://patch.msgid.link/20240827093011.18621-14-nbd@nbd.name
Signed-off-by: Felix Fietkau <nbd@nbd.name>
This commit is contained in:
Felix Fietkau 2024-08-27 11:30:01 +02:00
parent 8351a4a40b
commit 3688c18b65
3 changed files with 25 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
drivers/net/wireless/mediatek/mt76/mcu.c
View File

@ -73,6 +73,8 @@ int mt76_mcu_skb_send_and_get_msg(struct mt76_dev *dev, struct sk_buff *skb,
int cmd, bool wait_resp, int cmd, bool wait_resp,
struct sk_buff **ret_skb) struct sk_buff **ret_skb)
{ {
unsigned int retry = 0;
struct sk_buff *orig_skb = NULL;
unsigned long expires; unsigned long expires;
int ret, seq; int ret, seq;
@ -81,6 +83,14 @@ int mt76_mcu_skb_send_and_get_msg(struct mt76_dev *dev, struct sk_buff *skb,
mutex_lock(&dev->mcu.mutex); mutex_lock(&dev->mcu.mutex);
if (dev->mcu_ops->mcu_skb_prepare_msg) {
ret = dev->mcu_ops->mcu_skb_prepare_msg(dev, skb, cmd, &seq);
if (ret < 0)
goto out;
}
retry:
orig_skb = skb_get(skb);
ret = dev->mcu_ops->mcu_skb_send_msg(dev, skb, cmd, &seq); ret = dev->mcu_ops->mcu_skb_send_msg(dev, skb, cmd, &seq);
if (ret < 0) if (ret < 0)
goto out; goto out;
@ -94,6 +104,14 @@ int mt76_mcu_skb_send_and_get_msg(struct mt76_dev *dev, struct sk_buff *skb,
do { do {
skb = mt76_mcu_get_response(dev, expires); skb = mt76_mcu_get_response(dev, expires);
if (!skb && !test_bit(MT76_MCU_RESET, &dev->phy.state) &&
retry++ < dev->mcu_ops->max_retry) {
dev_err(dev->dev, "Retry message %08x (seq %d)\n",
cmd, seq);
skb = orig_skb;
goto retry;
}
ret = dev->mcu_ops->mcu_parse_response(dev, cmd, skb, seq); ret = dev->mcu_ops->mcu_parse_response(dev, cmd, skb, seq);
if (!ret && ret_skb) if (!ret && ret_skb)
*ret_skb = skb; *ret_skb = skb;
@ -101,7 +119,9 @@ int mt76_mcu_skb_send_and_get_msg(struct mt76_dev *dev, struct sk_buff *skb,
dev_kfree_skb(skb); dev_kfree_skb(skb);
} while (ret == -EAGAIN); } while (ret == -EAGAIN);
out: out:
dev_kfree_skb(orig_skb);
mutex_unlock(&dev->mcu.mutex); mutex_unlock(&dev->mcu.mutex);
return ret; return ret;

3
drivers/net/wireless/mediatek/mt76/mt76.h
View File

@ -230,11 +230,14 @@ struct mt76_queue {
}; };
struct mt76_mcu_ops { struct mt76_mcu_ops {
unsigned int max_retry;
u32 headroom; u32 headroom;
u32 tailroom; u32 tailroom;
int (*mcu_send_msg)(struct mt76_dev *dev, int cmd, const void *data, int (*mcu_send_msg)(struct mt76_dev *dev, int cmd, const void *data,
int len, bool wait_resp); int len, bool wait_resp);
int (*mcu_skb_prepare_msg)(struct mt76_dev *dev, struct sk_buff *skb,
int cmd, int *seq);
int (*mcu_skb_send_msg)(struct mt76_dev *dev, struct sk_buff *skb, int (*mcu_skb_send_msg)(struct mt76_dev *dev, struct sk_buff *skb,
int cmd, int *seq); int cmd, int *seq);
int (*mcu_parse_response)(struct mt76_dev *dev, int cmd, int (*mcu_parse_response)(struct mt76_dev *dev, int cmd,

7
drivers/net/wireless/mediatek/mt76/mt7915/mcu.c
View File

@ -191,11 +191,6 @@ mt7915_mcu_send_message(struct mt76_dev *mdev, struct sk_buff *skb,
{ {
struct mt7915_dev *dev = container_of(mdev, struct mt7915_dev, mt76); struct mt7915_dev *dev = container_of(mdev, struct mt7915_dev, mt76);
enum mt76_mcuq_id qid; enum mt76_mcuq_id qid;
int ret;
ret = mt76_connac2_mcu_fill_message(mdev, skb, cmd, wait_seq);
if (ret)
return ret;
if (cmd == MCU_CMD(FW_SCATTER)) if (cmd == MCU_CMD(FW_SCATTER))
qid = MT_MCUQ_FWDL; qid = MT_MCUQ_FWDL;
@ -2382,7 +2377,9 @@ int mt7915_mcu_init_firmware(struct mt7915_dev *dev)
int mt7915_mcu_init(struct mt7915_dev *dev) int mt7915_mcu_init(struct mt7915_dev *dev)
{ {
static const struct mt76_mcu_ops mt7915_mcu_ops = { static const struct mt76_mcu_ops mt7915_mcu_ops = {
.max_retry = 3,
.headroom = sizeof(struct mt76_connac2_mcu_txd), .headroom = sizeof(struct mt76_connac2_mcu_txd),
.mcu_skb_prepare_msg = mt76_connac2_mcu_fill_message,
.mcu_skb_send_msg = mt7915_mcu_send_message, .mcu_skb_send_msg = mt7915_mcu_send_message,
.mcu_parse_response = mt7915_mcu_parse_response, .mcu_parse_response = mt7915_mcu_parse_response,
}; };