On 03/01/2012 10:26 PM, Arend van Spriel wrote:
> On 02/27/2012 11:12 AM, Arend van Spriel wrote:
>> On 02/25/2012 01:52 PM, Hauke Mehrtens wrote:
>>> On 02/23/2012 10:52 PM, Arend van Spriel wrote:
>>>> Wireless Broadcom chips can have either their SPROM data stored
>>>> on either external SPROM or on-chip OTP memory. Both are accessed
>>>> through the same register space. This patch adds support for the
>>>> on-chip OTP memory.
>>>>
>>>> Tested with:
>>>> BCM43224 OTP and SPROM
>>>> BCM4331 SPROM
>>>> BCM4313 OTP
>>>
>>> Does bcma now support the same features regarding sprom and otp as
>>> brcmsamc expect it does not read out all the attributes brcmsmac reads
>>> out or are there any features still missing? I am just asking because
>>> the code used in brcmsmac is ~4 times longer.
>>
>> I started on using bcma sprom content in brcmsmac and indeed there are
>> some entries missing. The change in this patch only provides read-access
>> to the srom data. As the chip comes up for read-access there is not much
>> programming need to accomplish that. The only feature that is not there
>> is that on some chips OTP can be powered down for power-saving. The
>> current chips supported by BCMA don't have that.
>
> With BCMA retrieving the SPROM data correctly, regardless how/where it
> is stored I went to change brcmsmac to make use of it. However, it
> turned out several attributes were missing.
>
> Now I could go and extend the ssb_sprom structure, but there I got
> confused. The structure is filled by BCMA in the function
> bcma_sprom_extract_r8(). The name suggest it deals with SROM revision 8
> attributes, but:
>
> for (i = 0; i < 3; i++) {
> v = sprom[SPOFF(SSB_SPROM8_IL0MAC) + i];
> *(((__be16 *)bus->sprom.il0mac) + i) = cpu_to_be16(v);
> }
>
> SPEX(board_rev, SSB_SPROM8_BOARDREV, ~0, 0);
>
> SPEX(txpid2g[0], SSB_SPROM4_TXPID2G01, SSB_SPROM4_TXPID2G0,
> SSB_SPROM4_TXPID2G0_SHIFT);
> SPEX(txpid2g[1], SSB_SPROM4_TXPID2G01, SSB_SPROM4_TXPID2G1,
> SSB_SPROM4_TXPID2G1_SHIFT);
>
> The attributes txpid2g that are filled here are deprecated for srom
> revision 8. Is b43 using these fields for boards with srom rev 8?
The code filling txpid2g is probably wrong or dated and should be
removed. It could be that the spec b43 is written on was based on an old
version of the Broadcom wl driver and they used txpid2g in n-phy code.
txpid2g is used by b43 (b43_nphy_tx_power_fix()), but brcmsmac
(wlc_phy_txpwr_fixpower_nphy()) uses 0 where b43 used the sprom values.
> I propose to move to BCMA having its own sprom structure definition.
> Srom revision 10 is upcoming and it does only apply to BCMA-based cards.
Doesn't this also apply for srom rev 9, or are there any ssb based
devices with srom rev 9 out there?
>
> Gr. AvS
>
--
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
