On 24-05-16 23:05, Rafał Miłecki wrote: > This is helpful for debugging, without this all I was getting from "iw" > command on device with BCM43602 was: >> command failed: Too many open files in system (-23) > > Signed-off-by: Rafał Miłecki <zaj...@gmail.com> > --- > V2: s/in/if/ in commit message > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > index 1652a48..f7b7e29 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > @@ -2031,7 +2031,7 @@ static int brcmf_p2p_request_p2p_if(struct > brcmf_p2p_info *p2p, > err = brcmf_fil_iovar_data_set(ifp, "p2p_ifadd", &if_request, > sizeof(if_request)); > if (err) > - return err; > + brcmf_err("p2p_ifadd failed %d\n", err);
I would prefer adding a more generic failure message including ifname and type in brcmf_cfg80211_add_iface() in cfg80211.c. Regards, Arend > > return err; > } >