Re: [PATCH net-next 6/8] wireless: brcmfmac: Use netif_rx().

Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx> · Tue, 8 Mar 2022 15:59:50 +0100

On 3/5/2022 11:12 PM, Sebastian Andrzej Siewior wrote:
Since commit
    baebdf48c3600 ("net: dev: Makes sure netif_rx() can be invoked in any context.")

the function netif_rx() can be used in preemptible/thread context as
well as in interrupt context.

Use netif_rx().

Looks good to me but that commit only seems to exist in net-next repo. 
So you want to take this patch through the net-next repo as well?

Regards,
Arend

Cc: Arend van Spriel <aspriel@xxxxxxxxx>
Cc: Chi-hsien Lin <chi-hsien.lin@xxxxxxxxxxxx>
Cc: Chung-hsien Hsu <chung-hsien.hsu@xxxxxxxxxxxx>
Cc: Franky Lin <franky.lin@xxxxxxxxxxxx>
Cc: Hante Meuleman <hante.meuleman@xxxxxxxxxxxx>
Cc: Kalle Valo <kvalo@xxxxxxxxxx>
Cc: SHA-cyfmac-dev-list@xxxxxxxxxxxx
Cc: Wright Feng <wright.feng@xxxxxxxxxxxx>
Cc: brcm80211-dev-list.pdl@xxxxxxxxxxxx
Cc: linux-wireless@xxxxxxxxxxxxxxx
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
---
  .../broadcom/brcm80211/brcmfmac/bcdc.c         |  4 ++--
  .../broadcom/brcm80211/brcmfmac/core.c         | 18 +++++-------------
  .../broadcom/brcm80211/brcmfmac/core.h         |  2 +-
  .../broadcom/brcm80211/brcmfmac/fwsignal.c     | 10 +++++-----
  .../broadcom/brcm80211/brcmfmac/fwsignal.h     |  2 +-
  .../broadcom/brcm80211/brcmfmac/msgbuf.c       |  5 ++---
  .../broadcom/brcm80211/brcmfmac/proto.h        |  6 +++---
  7 files changed, 19 insertions(+), 28 deletions(-)
Attachment:
smime.p7s

Description: S/MIME Cryptographic Signature