[Raspbian-devel] [PATCH] drivers: wireless: brcmfmac: use correct interface to record log info

[Phil Elwell]

Meng Li,

I don't know what the raspbian-devel mailing list is intended for, but
what you have is a kernel patch so it needs to go to the kernel
mailing lists. Use scripts/get_maintainer.pl in the kernel tree on
your patch to find out which mailing lists it should be sent to.

I think you would be wise to also read
Documentation/process/submitting-patches.rst, which describes the
whole process of getting a patch accepted.

Phil

On Wed, 10 Mar 2021 at 04:51, <Meng.Li at windriver.com> wrote:
>
> From: Meng Li <Meng.Li at windriver.com>
>
> Use brcmf_info interface that causes lots of debug info recorded
> in log message file. So, refer to code that is used to record debug
> info in this file, replace brcmf_info with the more reasonable
> brcmf_dbg interface and set INFO parameter.
>
> Signed-off-by: Meng Li <Meng.Li at windriver.com>
> ---
>  drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
> index 525bf62d5b3c..32b232998f1e 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
> @@ -2938,7 +2938,7 @@ brcmf_cfg80211_set_power_mgmt(struct wiphy *wiphy, struct net_device *ndev,
>                 brcmf_dbg(INFO, "Do not enable power save for P2P clients\n");
>                 pm = PM_OFF;
>         }
> -       brcmf_info("power save %s\n", (pm ? "enabled" : "disabled"));
> +       brcmf_dbg(INFO, "power save %s\n", (pm ? "enabled" : "disabled"));
>
>         err = brcmf_fil_cmd_int_set(ifp, BRCMF_C_SET_PM, pm);
>         if (err) {
> --
> 2.17.1
>