Skip to content
Permalink
Browse files

brcmfmac: change rx_seq check log from error print to debug print

The bus rx sequence is not in order because that control and event
frames always cause immediate send, but data frames may be held
for glomming in firmware side. It is not actually an error as the
packets are still processed even if the RX sequence is not in order.
Therefore the error message is rephrased and changed to a debug
message.

[ Patch from Broadcom ]

See: #1313
  • Loading branch information...
pelwell authored and popcornmix committed Jun 6, 2016
1 parent 60d819d commit 3eec51e089392ff3d7859f5a6779e008e6c3a16d
Showing with 1 addition and 2 deletions.
  1. +1 −2 drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
@@ -1386,8 +1386,7 @@ static int brcmf_sdio_hdparse(struct brcmf_sdio *bus, u8 *header,
return -ENXIO;
}
if (rd->seq_num != rx_seq) {
brcmf_err("seq %d: sequence number error, expect %d\n",
rx_seq, rd->seq_num);
brcmf_dbg(SDIO, "seq %d, expected %d\n", rx_seq, rd->seq_num);
bus->sdcnt.rx_badseq++;
rd->seq_num = rx_seq;
}

0 comments on commit 3eec51e

Please sign in to comment.
You can’t perform that action at this time.