351-0016-brcmfmac-change-rx_seq-check-log-from-error-print-to.patch 1.1 KB

123456789101112131415161718192021222324252627282930
  1. From: Wright Feng <wright.feng@broadcom.com>
  2. Date: Fri, 3 Jun 2016 23:31:11 +0200
  3. Subject: [PATCH] brcmfmac: change rx_seq check log from error print to debug
  4. print
  5. The bus rx sequence is not in order because that control and event
  6. frames always cause immediate send, but data frames may be held
  7. for glomming in firmware side. It is not actually an error as the
  8. packets are still processed even if the RX sequence is not in order.
  9. Therefor the error message is rephrased and changed to a debug
  10. message.
  11. Reviewed-by: Arend Van Spriel <arend@broadcom.com>
  12. Signed-off-by: Wright Feng <wright.feng@broadcom.com>
  13. Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
  14. Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
  15. ---
  16. --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
  17. +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
  18. @@ -1384,8 +1384,7 @@ static int brcmf_sdio_hdparse(struct brc
  19. return -ENXIO;
  20. }
  21. if (rd->seq_num != rx_seq) {
  22. - brcmf_err("seq %d: sequence number error, expect %d\n",
  23. - rx_seq, rd->seq_num);
  24. + brcmf_dbg(SDIO, "seq %d, expected %d\n", rx_seq, rd->seq_num);
  25. bus->sdcnt.rx_badseq++;
  26. rd->seq_num = rx_seq;
  27. }