Hi, On 5/9/23 12:04, matthias.bgg@xxxxxxxxxx wrote: > From: Matthias Brugger <mbrugger@xxxxxxxx> > > The message is attach and detach function are merly for debugging, > change them from pr_err to pr_debug. > > Signed-off-by: Matthias Brugger <mbrugger@xxxxxxxx> Thank you, I had writing this same patch on my own TODO list :) Patch looks good to me: Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx> Regards, Hans > --- > > drivers/net/wireless/broadcom/brcm80211/brcmfmac/wcc/core.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/wcc/core.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/wcc/core.c > index 02de99818efa..5573a47766ad 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/wcc/core.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/wcc/core.c > @@ -12,13 +12,13 @@ > > static int brcmf_wcc_attach(struct brcmf_pub *drvr) > { > - pr_err("%s: executing\n", __func__); > + pr_debug("%s: executing\n", __func__); > return 0; > } > > static void brcmf_wcc_detach(struct brcmf_pub *drvr) > { > - pr_err("%s: executing\n", __func__); > + pr_debug("%s: executing\n", __func__); > } > > const struct brcmf_fwvid_ops brcmf_wcc_ops = {