package/kernel/mac80211/patches/376-0003-brcmfmac-Change-error-print-on-wlan0-existence.patch

   1 From: Hante Meuleman <meuleman@broadcom.com>
   2 Date: Wed, 25 Nov 2015 11:32:39 +0100
   3 Subject: [PATCH] brcmfmac: Change error print on wlan0 existence.
   4
   5 During initialization of the device, but also on some other
   6 moments the driver prints an error that the netdev already exists.
   7 This is a result of the way the driver is initializing the
   8 firmware and not really an error. The code is not treating it as
   9 an error either. This error print has resulted in many questions
  10 by users and is confusing and incorrect. This patch changes the
  11 error log into a debug info log.
  12
  13 Reviewed-by: Arend Van Spriel <arend@broadcom.com>
  14 Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
  15 Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
  16 Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
  17 Signed-off-by: Arend van Spriel <arend@broadcom.com>
  18 Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
  19 ---
  20
  21 --- a/drivers/net/wireless/brcm80211/brcmfmac/core.c
  22 +++ b/drivers/net/wireless/brcm80211/brcmfmac/core.c
  23 @@ -811,14 +811,15 @@ struct brcmf_if *brcmf_add_if(struct brc
  24          * in case we missed the BRCMF_E_IF_DEL event.
  25          */
  26         if (ifp) {
  27 -               brcmf_err("ERROR: netdev:%s already exists\n",
  28 -                         ifp->ndev->name);
  29                 if (ifidx) {
  30 +                       brcmf_err("ERROR: netdev:%s already exists\n",
  31 +                                 ifp->ndev->name);
  32                         netif_stop_queue(ifp->ndev);
  33                         brcmf_net_detach(ifp->ndev);
  34                         drvr->iflist[bsscfgidx] = NULL;
  35                 } else {
  36 -                       brcmf_err("ignore IF event\n");
  37 +                       brcmf_dbg(INFO, "netdev:%s ignore IF event\n",
  38 +                                 ifp->ndev->name);
  39                         return ERR_PTR(-EINVAL);
  40                 }
  41         }