2015-03-22 20:14:47

by Jürgen Bausa

[permalink] [raw]
Subject: Re: Broadcom 43340

QXJlbmQgdmFuIFNwcmllbCA8YXJlbmRALi4uPiB3cml0ZXM6Cgo+IAo+IE9uIDAzLzIxLzE1IDIxOjM4LCBKw7xyZ2VuIEJhdXNhIHdyb3RlOgo+ID4gMS4gdGhlIG52cmFtIGZpbGUgdW5kZXIgL3N5cy9maXJtd2FyZS9lZmkvZWZpdmFycy8gaGFzIGEgd3JvbmcgTUFDIGFkZHJlc3MuCj4gPiBJIGNoYW5nZWQgaXQgdG8gdGhlIGNvcnJlY3Qgb25lLiBIb3dldmVyLCBJIGFtIG5vdCBzdXJlIGlmIHRoaXMgaXMgcmVhbGx5Cj4gPiBuZWNhc3NhcnkuCj4gCj4gVGhhdCBNQUMgYWRkcmVzcyBpcyBvbmx5IHVzZWQgZm9yIGRldmljZSB0aGF0IGRvIGhhdmUgbWFjIGFkZHJlc3MgaW4gdGhlIAo+IGRldmljZSBzbyB5b3UgY2FuIGlnbm9yZSB0aGF0IG9uZS4KCllvdSBhcmUgcmlnaHQsIGZvdW5kIGl0IG91dCBtYXlzZWxmIGluIHRoZSBtZWFudGltZS4gVGhlcmUgaXMgbm8gbmVlZCB0bwpjaGFuZ2UgdGhlIE1BQyBhZGRyZXNzLgoKPiAKPiA+IDIuIEZvdW5kIHRoZSBmb2xsb3dpbmcgb24gdGhlIHdlYjoKPiA+Cj4gPj4gcm1tb2QgYnJjbWZtYWMKPiA+PiBybW1vZCBicmNtdXRpbAo+ID4+IGVjaG8gb24+ICAvc3lzL2J1cy9wbGF0Zm9ybS9kcml2ZXJzL3NkaGNpLWFjcGkvSU5UMzNCQlw6MDAvcG93ZXIvY29udHJvbAo+ID4+IG1vZHByb2JlIGJyY21mbWFjCj4gPgo+ID4gQWZ0ZXIgdGhpcywgSSBjYW4gY29ubmVjdCB0byB3bGFuIQo+IAo+IEFoLCBJIHdhcyB3b25kZXJpbmcgd2hhdCB0eXBlIG9mIHNkaW8gaG9zdCBjb250cm9sbGVyIHdhcyB1c2VkLiBXaGVuIAo+IHNkaW8gaG9zdCBjb250cm9sbGVyIHVzZXMgcnVudGltZS1wbSBpdCBhZmZlY3QgY29tbXVuaWNhdGlvbiB3aXRoIHdsYW4gCj4gZGV2aWNlLiBJIHJlY2VudGx5IHBvc3RlZCBhIHBhdGNoIGZvciB0aGF0IFsxXS4KClNvLCBkbyBJIHVuZGVyc3RhbmQgY29ycmVjdGx5LCB0aGF0IHdpdGggeW91ciBwYXRjaCB0aGUgYWJvdmUgd29ya2Fyb3VuZAp3aWxsIG5vIGxvbmdlciBiZSBuZWVkZWQ/IEluIHdoaWNoIGtlcm5lbCB3aWxsIHlvdXIgcGF0Y2ggYmUgaW5jbHVkZWQ/IEFzCndpZmkgaXMgd29ya2luZyBub3csIEkgd291bGQgYXZvaWQgaW5zdGFsbGluZyB0aGUgcGF0Y2ggYW5kIGluc3RlYWQgd2FpdCBmb3IKdGhlIG5leHQga2VybmVsLgoKPiAKPiA+IERvZXMgdGhpcyBnaXZlIHlvdSBlbm91Z2ggaW5mb3JtYXRpb24gdG8gZmluZCBvdXQsIHdoYXQgdGhlIHByb2JsZW0gaXM/IE9yCj4gPiBkbyB5b3Ugc3RpbGwgbmVlZCBEQkcgb3V0cHV0IGZyb20gdGhlIGRyaXZlcj8gaWYgc28sIGp1c3QgdGVsbCBtZSBhbmQgSSB3aWxsCj4gPiB0cnkgdG8gY29tcGlsZSBhbmQgaW5zdGFsbCB0aGUga2VybmVsIHdpdGggREJHIHR1cm5kZWQgb24uIEl0cyBub3QgYSBiaWcKPiA+IGRlYWwgYnV0IEkgc3RvcHBlZCB3b3JraW5nIG9uIGl0IGFzIEkgZm91bmQgdGhpcyB3b3JrYXJvdW5kLgo+ID4KPiA+IEF0IHRoZSBtb21tZW50IEkgYW0gdHJ5aW5nIHRvIHB1dCB0aGVzZSBjb21tYW5kcyB0byByYyxsb2NhbC4gV2l0aCBsaW1pdGVkCj4gPiBzdWNjZXNzIHVudGlsIG5vdywgQnV0IHRoaXMgbWF5IGJlIGNhdXNlZCBieSB0aGUgZmFjdCwgdGhhdCB0aGV5IGFyZQo+ID4gZXhlY3V0ZWQgdG9vIGVhcmx5LiBXaWxsIGFkZCBzb21lIHNsZWVwLgo+IAo+IE1pZ2h0IGJlIHRoZSBkcml2ZXIgaXMgYWxyZWFkeSBsb2FkZWQgYmVmb3JlIHJvb3QgZmlsZSBzeXN0ZW0gaXMgbG9hZGVkIAo+IChmcm9tIGluaXRyYW1mcykuCgpJIG5vdyBnb3QgaXQgd29ya2luZy4gSnVzdCBwdXQgJ3NsZWVlcCAyMCcgYmVmb3JlIHRoZSBjb21tYW5kcyBhbmQgbm93IGl0CndvcmtzIHBlcmZlY3RseS4KCj4gCj4gPiBBbm90aGVyIHRoaW5nIEkgZm91bmQgb3V0IGlzLCB0aGF0IHRoZSBzeXN0ZW0gc29tZXRpbWVzIGNvbWVzIHVwIHdpdGhvdXQKPiA+IHdsYW4wLiBJdCBzZWVtcyBJIGhhdmUgaXQgb25seSBvbiBldmVyeSBzZWNvbmQgYm9vdC4gSG93ZXZlciwgdGhpcyBjb3VsZCBiZQo+ID4ganVzdCBieSBpbmNpZGVudCBhbmQgaXQgaXMgbm90IGV4YWN0bHkgZXZlcnkgc2Vjb25kIGJvb3QgYnV0IGFib3V0IDUwICUgb2YKPiA+IGFsbCBib290cy4KPiAKPiBJcyB0aGF0IGFmdGVyIHJlYm9vdCBvciByZWFsbHkgYWZ0ZXIgcG93ZXJ1cC4gQ3VyaW91cyB3aGV0aGVyIHRoZXJlIGlzIGEgCj4gZGlmZmVyZW5jZSBiZXR3ZWVuIHRoZSB0d28gcmVnYXJkaW5nIHRoaXMgaXNzdWUuCj4gCgpJdCB3YXMgYWZ0ZXIgcmVib290IGFuZCBhdCBjb2xkIGJvb3QuCgpIb3dldmVyLCB0aGlzIHByb2JsZW0gc2VlbXMgdG8gYmUgZ29uZS4gd2xhbjAgaXMgdXAgb24gZXZlcnkgYm9vdC4gSW4gdGhlCm1lYW50aW1lIGkgdXBncmFkZWQgdG8gcmM0LiBDb3VsZCB0aGlzIGJlIHRoZSByZWFzb24/CgpEbyB5b3Ugc3RpbGwgbmVlZCBzb21lIGRiZyBpbmZvcm1hdGlvbiBmcm9tIG15IG1hY2hpbmU/IElmIHNvLCBqdXN0IHRlbGwgbWUKYW5kIEkgd2lsbCBpbnN0YWxsIGEga2VybmVsIHdpdGggdGhlIG5lZWRlZCBvcHRpb25zLiBPdGhlcndpc2UsIEkgd2lsbCBzdGF5CndpdGggdGhlIGN1cnJlbnQgc2l0dWF0aW9uLgoKCj4+IE9rLCB0aGFua3MuIFdpbGwgdGhlIGRyaXZlciBhbHNvIGVuYWJsZSBibHVldG9vdGg/IEFzIEkgcmVhZCwgdGhlCj4+IGJjbTQzMzQxIGFsc28gaGFzIEJsdWV0b290aCBjYXBhYmlsaXRpZXMuCj4KPkl0IGhhcywgYnV0IG5vIGNsdWUgd2hhdCBob3N0IGludGVyZmFjZSBpcyB1c2VkLiBBbnl3YXksIGl0IHdpbGwgaGF2ZSAKPmEgc2VwYXJhdGUgZHJpdmVyLgoKRG8geW91IG5vdywgaWYgdGhlcmUgaXMgc29tZSBkZXZlbG9wbWVudCBnb2luZyBvbiBmb3IgdGhlIGJsdWV0b290aCBkcml2ZXI/CldobyBjb3VsZCBJIGNvbmFjdD8KCkp1ZXJnZW4KCg==



2015-03-23 09:52:43

by Arend van Spriel

[permalink] [raw]
Subject: Re: Broadcom 43340

On 03/22/15 21:14, Jürgen Bausa wrote:
> Arend van Spriel<arend@...> writes:
>
>>
>> On 03/21/15 21:38, Jürgen Bausa wrote:
>>> 1. the nvram file under /sys/firmware/efi/efivars/ has a wrong MAC address.
>>> I changed it to the correct one. However, I am not sure if this is really
>>> necassary.
>>
>> That MAC address is only used for device that do have mac address in the
>> device so you can ignore that one.
>
> You are right, found it out mayself in the meantime. There is no need to
> change the MAC address.
>
>>
>>> 2. Found the following on the web:
>>>
>>>> rmmod brcmfmac
>>>> rmmod brcmutil
>>>> echo on> /sys/bus/platform/drivers/sdhci-acpi/INT33BB\:00/power/control
>>>> modprobe brcmfmac
>>>
>>> After this, I can connect to wlan!
>>
>> Ah, I was wondering what type of sdio host controller was used. When
>> sdio host controller uses runtime-pm it affect communication with wlan
>> device. I recently posted a patch for that [1].
>
> So, do I understand correctly, that with your patch the above workaround
> will no longer be needed? In which kernel will your patch be included? As
> wifi is working now, I would avoid installing the patch and instead wait for
> the next kernel.

Indeed. The patch basically does the same thing as the workaround. It is
applied to wireless-drivers-next tree so it will be in 4.1 kernel. So it
will take a while before that gets released by Linus. 4.1-rc1 will
probably take approx. 6-7 wks.

>>
>>> Does this give you enough information to find out, what the problem is? Or
>>> do you still need DBG output from the driver? if so, just tell me and I will
>>> try to compile and install the kernel with DBG turnded on. Its not a big
>>> deal but I stopped working on it as I found this workaround.
>>>
>>> At the momment I am trying to put these commands to rc,local. With limited
>>> success until now, But this may be caused by the fact, that they are
>>> executed too early. Will add some sleep.
>>
>> Might be the driver is already loaded before root file system is loaded
>> (from initramfs).
>
> I now got it working. Just put 'sleeep 20' before the commands and now it
> works perfectly.

Holding the boot for 20s is steep, but I guess a working device trumps a
fast boot ;-)

>>
>>> Another thing I found out is, that the system sometimes comes up without
>>> wlan0. It seems I have it only on every second boot. However, this could be
>>> just by incident and it is not exactly every second boot but about 50 % of
>>> all boots.
>>
>> Is that after reboot or really after powerup. Curious whether there is a
>> difference between the two regarding this issue.
>>
>
> It was after reboot and at cold boot.
>
> However, this problem seems to be gone. wlan0 is up on every boot. In the
> meantime i upgraded to rc4. Could this be the reason?

Nothing in brcmfmac is changed that would explain this. Maybe something
was fixed in sdio host controller, but it is all speculation.

> Do you still need some dbg information from my machine? If so, just tell me
> and I will install a kernel with the needed options. Otherwise, I will stay
> with the current situation.

If you are happy and the issue seems resolved there is no need. Thanks.

>>> Ok, thanks. Will the driver also enable bluetooth? As I read, the
>>> bcm43341 also has Bluetooth capabilities.
>>
>> It has, but no clue what host interface is used. Anyway, it will have
>> a separate driver.
>
> Do you now, if there is some development going on for the bluetooth driver?
> Who could I conact?

I am not sure what host-interface is used to bt part. Is it usb or uart?
I know there are plans to mainline the bt uart driver, but not sure when
that will make its way into the kernel. If it is usb you should be able
to use btusb driver, but you need firmware. They added support for
handling firmware in btusb recently.

Regards,
Arend