mwifiex: Read a PCI register after writing the TX ring write pointer
authorJonas Dreßler <verdre@v0yd.nl>
Mon, 11 Oct 2021 13:32:23 +0000 (15:32 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 18 Oct 2021 12:29:51 +0000 (15:29 +0300)
commite5f4eb8223aa740237cd463246a7debcddf4eda1
tree7e88fde9a1a94a975ed5c7d6a34689720c4ea41f
parentff1cc2fa3055ee4c83839f38b74b4ee370a2291c
mwifiex: Read a PCI register after writing the TX ring write pointer

On the 88W8897 PCIe+USB card the firmware randomly crashes after setting
the TX ring write pointer. The issue is present in the latest firmware
version 15.68.19.p21 of the PCIe+USB card.

Those firmware crashes can be worked around by reading any PCI register
of the card after setting that register, so read the PCI_VENDOR_ID
register here. The reason this works is probably because we keep the bus
from entering an ASPM state for a bit longer, because that's what causes
the cards firmware to crash.

This fixes a bug where during RX/TX traffic and with ASPM L1 substates
enabled (the specific substates where the issue happens appear to be
platform dependent), the firmware crashes and eventually a command
timeout appears in the logs.

BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=109681
Cc: stable@vger.kernel.org
Signed-off-by: Jonas Dreßler <verdre@v0yd.nl>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20211011133224.15561-2-verdre@v0yd.nl
drivers/net/wireless/marvell/mwifiex/pcie.c