Skip to content

Commit 4454eea

Browse files
Arend van Sprielsmb49
authored andcommitted
wifi: brcmfmac: fix regression for Broadcom PCIe wifi devices
BugLink: https://bugs.launchpad.net/bugs/2008946 commit ed05cb1 upstream. A sanity check was introduced considering maximum flowrings above 256 as insane and effectively aborting the device probe. This resulted in regression for number of users as the value turns out to be sane after all. Fixes: 2aca4f3 ("brcmfmac: return error when getting invalid max_flowrings from dongle") Reported-by: chainofflowers <chainofflowers@posteo.net> Link: https://lore.kernel.org/all/4781984.GXAFRqVoOG@luna/ Reported-by: Christian Marillat <marillat@debian.org> Link: https://bugzilla.kernel.org/show_bug.cgi?id=216894 Cc: stable@vger.kernel.org Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com> Signed-off-by: Kalle Valo <kvalo@kernel.org> Link: https://lore.kernel.org/r/20230111112419.24185-1-arend.vanspriel@broadcom.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Kamal Mostafa <kamal@canonical.com> Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
1 parent cb952b9 commit 4454eea

File tree

1 file changed

+1
-1
lines changed
  • drivers/net/wireless/broadcom/brcm80211/brcmfmac

1 file changed

+1
-1
lines changed

drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1109,7 +1109,7 @@ static int brcmf_pcie_init_ringbuffers(struct brcmf_pciedev_info *devinfo)
11091109
BRCMF_NROF_H2D_COMMON_MSGRINGS;
11101110
max_completionrings = BRCMF_NROF_D2H_COMMON_MSGRINGS;
11111111
}
1112-
if (max_flowrings > 256) {
1112+
if (max_flowrings > 512) {
11131113
brcmf_err(bus, "invalid max_flowrings(%d)\n", max_flowrings);
11141114
return -EIO;
11151115
}

0 commit comments

Comments
 (0)