All of lore.kernel.org
 help / color / mirror / Atom feed
* brcmfmac: set p2p_disc error on BCM4350
@ 2018-01-07 23:49 William Dauchy
  2018-01-08  9:31 ` Arend van Spriel
  0 siblings, 1 reply; 6+ messages in thread
From: William Dauchy @ 2018-01-07 23:49 UTC (permalink / raw)
  To: linux-wireless; +Cc: Arend Van Spriel

Hello,

Using the last v4.14.x I am getting this error at each boot, on my
BCM4350 802.11ac Wireless Network Adapter:

brcmfmac: brcmf_p2p_create_p2pdev: set p2p_disc error
brcmfmac: brcmf_cfg80211_add_iface: add iface p2p-dev-wlp58s0 type 10
failed: err=-16

The firmware used is:
brcmfmac: brcmf_c_preinit_dcmds: Firmware version = wl0: Oct 22 2015
06:16:26 version 7.35.180.119 (r594535) FWID 01-e791c176

I was wondering if this error could be fixed?

Best,
-- 
William

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: brcmfmac: set p2p_disc error on BCM4350
  2018-01-07 23:49 brcmfmac: set p2p_disc error on BCM4350 William Dauchy
@ 2018-01-08  9:31 ` Arend van Spriel
  2018-01-08 10:30   ` William Dauchy
  0 siblings, 1 reply; 6+ messages in thread
From: Arend van Spriel @ 2018-01-08  9:31 UTC (permalink / raw)
  To: William Dauchy, linux-wireless

On 1/8/2018 12:49 AM, William Dauchy wrote:
> Hello,
>
> Using the last v4.14.x I am getting this error at each boot, on my
> BCM4350 802.11ac Wireless Network Adapter:
>
> brcmfmac: brcmf_p2p_create_p2pdev: set p2p_disc error
> brcmfmac: brcmf_cfg80211_add_iface: add iface p2p-dev-wlp58s0 type 10
> failed: err=-16
>
> The firmware used is:
> brcmfmac: brcmf_c_preinit_dcmds: Firmware version = wl0: Oct 22 2015
> 06:16:26 version 7.35.180.119 (r594535) FWID 01-e791c176
>
> I was wondering if this error could be fixed?

This has been reported before, but not been able to look into it. Diving 
into firmware it seems that error can only occur if the mac address for 
the p2pdev interface is already used. So if possible can you rebuild the 
driver with the patch below applied and provide dmesg output and output 
of 'ifconfig -a'.

Regards,
Arend
---8<----------------------------------------------------------------

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c 
b/drivers/n
index 2ee5413..f96ad37 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
@@ -2065,6 +2065,7 @@ static struct wireless_dev 
*brcmf_p2p_create_p2pdev(struc
         int err;
         u32 bsscfgidx;

+       brcmf_err("enter: mac=%pM\n", addr);
         if (p2p->bss_idx[P2PAPI_BSSCFG_DEVICE].vif)
                 return ERR_PTR(-ENOSPC);

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: brcmfmac: set p2p_disc error on BCM4350
  2018-01-08  9:31 ` Arend van Spriel
@ 2018-01-08 10:30   ` William Dauchy
  2018-01-15 10:29     ` William Dauchy
  0 siblings, 1 reply; 6+ messages in thread
From: William Dauchy @ 2018-01-08 10:30 UTC (permalink / raw)
  To: Arend van Spriel; +Cc: linux-wireless

Hello Arend,

Thanks for your quick answer.

On Mon, Jan 08, 2018 at 10:31:52AM +0100, Arend van Spriel wrote:
> This has been reported before, but not been able to look into it. Diving
> into firmware it seems that error can only occur if the mac address for the
> p2pdev interface is already used. So if possible can you rebuild the driver
> with the patch below applied and provide dmesg output and output of
> 'ifconfig -a'.
> 
> Regards,
> Arend
> ---8<----------------------------------------------------------------
> 
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> b/drivers/n
> index 2ee5413..f96ad37 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> @@ -2065,6 +2065,7 @@ static struct wireless_dev
> *brcmf_p2p_create_p2pdev(struc
>         int err;
>         u32 bsscfgidx;
> 
> +       brcmf_err("enter: mac=%pM\n", addr);
>         if (p2p->bss_idx[P2PAPI_BSSCFG_DEVICE].vif)
>                 return ERR_PTR(-ENOSPC);
> 
>

Here is my brcmfmac dmesg output with patch applied
(please tell me if you need complete dmesg)

brcmfmac: brcmf_fw_map_chip_to_name: using brcm/brcmfmac4350-pcie.bin for chip 0x004350(17232) rev 0x000008
brcmfmac 0000:3a:00.0: Direct firmware load for brcm/brcmfmac4350-pcie.txt failed with error -2
brcmfmac: brcmf_c_preinit_dcmds: Firmware version = wl0: Oct 22 2015 06:16:26 version 7.35.180.119 (r594535) FWID 01-e791c176
brcmfmac 0000:3a:00.0 wlp58s0: renamed from wlan0
brcmfmac: brcmf_p2p_create_p2pdev: enter: mac=00:00:00:00:00:00
brcmfmac: brcmf_p2p_create_p2pdev: set p2p_disc error
brcmfmac: brcmf_cfg80211_add_iface: add iface p2p-dev-wlp58s0 type 10 failed: err=-16

ifconfig shows:

wlp58s0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet 192.168.18.77  netmask 255.255.255.0  broadcast 192.168.18.255
        inet6 fe80::3517:e1be:5bc7:899  prefixlen 64  scopeid 0x20<link>
        ether 30:52:cb:83:71:33  txqueuelen 1000  (Ethernet)
        RX packets 26557  bytes 34149730 (32.5 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 14965  bytes 2298777 (2.1 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

Best,
-- 
William

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: brcmfmac: set p2p_disc error on BCM4350
  2018-01-08 10:30   ` William Dauchy
@ 2018-01-15 10:29     ` William Dauchy
  2018-01-15 19:48       ` Arend van Spriel
  0 siblings, 1 reply; 6+ messages in thread
From: William Dauchy @ 2018-01-15 10:29 UTC (permalink / raw)
  To: Arend van Spriel; +Cc: linux-wireless

Hello Arend,

Did you had the chance to look into this?

On Mon, Jan 08, 2018 at 11:30:12AM +0100, William Dauchy wrote:
> Hello Arend,
> 
> Thanks for your quick answer.
> 
> On Mon, Jan 08, 2018 at 10:31:52AM +0100, Arend van Spriel wrote:
> > This has been reported before, but not been able to look into it. Diving
> > into firmware it seems that error can only occur if the mac address for the
> > p2pdev interface is already used. So if possible can you rebuild the driver
> > with the patch below applied and provide dmesg output and output of
> > 'ifconfig -a'.
> > 
> > Regards,
> > Arend
> > ---8<----------------------------------------------------------------
> > 
> > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> > b/drivers/n
> > index 2ee5413..f96ad37 100644
> > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
> > @@ -2065,6 +2065,7 @@ static struct wireless_dev
> > *brcmf_p2p_create_p2pdev(struc
> >         int err;
> >         u32 bsscfgidx;
> > 
> > +       brcmf_err("enter: mac=%pM\n", addr);
> >         if (p2p->bss_idx[P2PAPI_BSSCFG_DEVICE].vif)
> >                 return ERR_PTR(-ENOSPC);
> > 
> >
> 
> Here is my brcmfmac dmesg output with patch applied
> (please tell me if you need complete dmesg)
> 
> brcmfmac: brcmf_fw_map_chip_to_name: using brcm/brcmfmac4350-pcie.bin for chip 0x004350(17232) rev 0x000008
> brcmfmac 0000:3a:00.0: Direct firmware load for brcm/brcmfmac4350-pcie.txt failed with error -2
> brcmfmac: brcmf_c_preinit_dcmds: Firmware version = wl0: Oct 22 2015 06:16:26 version 7.35.180.119 (r594535) FWID 01-e791c176
> brcmfmac 0000:3a:00.0 wlp58s0: renamed from wlan0
> brcmfmac: brcmf_p2p_create_p2pdev: enter: mac=00:00:00:00:00:00
> brcmfmac: brcmf_p2p_create_p2pdev: set p2p_disc error
> brcmfmac: brcmf_cfg80211_add_iface: add iface p2p-dev-wlp58s0 type 10 failed: err=-16
> 
> ifconfig shows:
> 
> wlp58s0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
>         inet 192.168.18.77  netmask 255.255.255.0  broadcast 192.168.18.255
>         inet6 fe80::3517:e1be:5bc7:899  prefixlen 64  scopeid 0x20<link>
>         ether 30:52:cb:83:71:33  txqueuelen 1000  (Ethernet)
>         RX packets 26557  bytes 34149730 (32.5 MiB)
>         RX errors 0  dropped 0  overruns 0  frame 0
>         TX packets 14965  bytes 2298777 (2.1 MiB)
>         TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

Best,
-- 
William

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: brcmfmac: set p2p_disc error on BCM4350
  2018-01-15 10:29     ` William Dauchy
@ 2018-01-15 19:48       ` Arend van Spriel
  2018-01-15 20:03         ` William Dauchy
  0 siblings, 1 reply; 6+ messages in thread
From: Arend van Spriel @ 2018-01-15 19:48 UTC (permalink / raw)
  To: William Dauchy; +Cc: linux-wireless

On 1/15/2018 11:29 AM, William Dauchy wrote:
> Hello Arend,
>
> Did you had the chance to look into this?

Thanks for the ping, but no not really. I just noticed the mac address 
in the p2p-dev create request is the null address and I recall that is 
handled, but I will need to look into that. I will get back to you if I 
have done so. Feel free to ping again when it is taking to long ;-)

Now the error is only an issue if you intend to setup a P2P connection 
with some other device. For running in station or AP mode you can ignore 
this. So if you have any specific issue and suspect this to be causing 
it, please elaborate on the specific issue you see.

Regards,
Arend

> On Mon, Jan 08, 2018 at 11:30:12AM +0100, William Dauchy wrote:
>> Hello Arend,
>>
>> Thanks for your quick answer.
>>
>> On Mon, Jan 08, 2018 at 10:31:52AM +0100, Arend van Spriel wrote:
>>> This has been reported before, but not been able to look into it. Diving
>>> into firmware it seems that error can only occur if the mac address for the
>>> p2pdev interface is already used. So if possible can you rebuild the driver
>>> with the patch below applied and provide dmesg output and output of
>>> 'ifconfig -a'.
>>>
>>> Regards,
>>> Arend
>>> ---8<----------------------------------------------------------------
>>>
>>> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
>>> b/drivers/n
>>> index 2ee5413..f96ad37 100644
>>> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
>>> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
>>> @@ -2065,6 +2065,7 @@ static struct wireless_dev
>>> *brcmf_p2p_create_p2pdev(struc
>>>          int err;
>>>          u32 bsscfgidx;
>>>
>>> +       brcmf_err("enter: mac=%pM\n", addr);
>>>          if (p2p->bss_idx[P2PAPI_BSSCFG_DEVICE].vif)
>>>                  return ERR_PTR(-ENOSPC);
>>>
>>>
>>
>> Here is my brcmfmac dmesg output with patch applied
>> (please tell me if you need complete dmesg)
>>
>> brcmfmac: brcmf_fw_map_chip_to_name: using brcm/brcmfmac4350-pcie.bin for chip 0x004350(17232) rev 0x000008
>> brcmfmac 0000:3a:00.0: Direct firmware load for brcm/brcmfmac4350-pcie.txt failed with error -2
>> brcmfmac: brcmf_c_preinit_dcmds: Firmware version = wl0: Oct 22 2015 06:16:26 version 7.35.180.119 (r594535) FWID 01-e791c176
>> brcmfmac 0000:3a:00.0 wlp58s0: renamed from wlan0
>> brcmfmac: brcmf_p2p_create_p2pdev: enter: mac=00:00:00:00:00:00
>> brcmfmac: brcmf_p2p_create_p2pdev: set p2p_disc error
>> brcmfmac: brcmf_cfg80211_add_iface: add iface p2p-dev-wlp58s0 type 10 failed: err=-16
>>
>> ifconfig shows:
>>
>> wlp58s0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
>>          inet 192.168.18.77  netmask 255.255.255.0  broadcast 192.168.18.255
>>          inet6 fe80::3517:e1be:5bc7:899  prefixlen 64  scopeid 0x20<link>
>>          ether 30:52:cb:83:71:33  txqueuelen 1000  (Ethernet)
>>          RX packets 26557  bytes 34149730 (32.5 MiB)
>>          RX errors 0  dropped 0  overruns 0  frame 0
>>          TX packets 14965  bytes 2298777 (2.1 MiB)
>>          TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0
>
> Best,
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: brcmfmac: set p2p_disc error on BCM4350
  2018-01-15 19:48       ` Arend van Spriel
@ 2018-01-15 20:03         ` William Dauchy
  0 siblings, 0 replies; 6+ messages in thread
From: William Dauchy @ 2018-01-15 20:03 UTC (permalink / raw)
  To: Arend van Spriel; +Cc: linux-wireless

Hello Arend,

On Mon, Jan 15, 2018 at 08:48:50PM +0100, Arend van Spriel wrote:
> Now the error is only an issue if you intend to setup a P2P connection with
> some other device. For running in station or AP mode you can ignore this. So
> if you have any specific issue and suspect this to be causing it, please
> elaborate on the specific issue you see.

no specific issue, just wanted to fix issue/error and avoid other users
to report it.

Thanks,
-- 
William

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-01-15 20:04 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-07 23:49 brcmfmac: set p2p_disc error on BCM4350 William Dauchy
2018-01-08  9:31 ` Arend van Spriel
2018-01-08 10:30   ` William Dauchy
2018-01-15 10:29     ` William Dauchy
2018-01-15 19:48       ` Arend van Spriel
2018-01-15 20:03         ` William Dauchy

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.