Search Linux Wireless

Re: About the patch: "staging: brcm80211: only enable brcmsmac if bcma is not set"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2011/9/8 Francis Moreau <francis.moro@xxxxxxxxx>:
> Hello,
>
> I have a question regarding the following patch:
> http://permalink.gmane.org/gmane.linux.kernel.wireless.general/75638
>
> The commit message says:
>
>   brcmsmac doesn't yet use bcma, but both drivers attempt to claim the same
>   device.  For now, turn of brcmsmac if bcma is enabled.
>
> I think it's not quite complete:  both driver claim the same devices
> (actually that's not entirely true since b43/bcma can drive on more
> card) but in the case of b43/bcma not all cards are actually working.
> For example I'm using BCM4313 card with an unsupported phy
> (B43_PHYTYPE_LCN) so it's basically not supported by b43/bcma.
>
> So I'm wondering if b43/bcma should be simply disabled until it got
> the same level of support as brcmsmac ?

I'm working on LCN support in b43 (BCM4313), static initialization is
already done. Should see more results soon.

The problem is that brcmsmac duplicates code present in the bcma
driver. It doesn't use bcma bus architecture.

The real solution is to add bcma driver support in brcmsmac and ssb
driver support in brcmfmac. Then you can always use bcma and:
1) If b43 will load and detect unsupported PHY, it will release the BCMA's core
2) brcmsmac will be loaded to let it support the core

-- 
Rafał
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux