Michael Buesch wrote:
> On Tuesday 13 February 2007 23:54, Larry Finger wrote:
>> The specifications for the bcm43xx driver have been modified. This patch
>> incorporates these changes in the code, which results in the BCM4311 and
>> BCM4312 working. The name of one of the PHY parameters, previously known
>> as "version", has been changed to "analog", short for "analog core version" .
>>
>> Signed-off-by: Larry Finger<Larry.Finger@xxxxxxxxxxxx>
>
>> @@ -750,7 +757,7 @@ static void bcm43xx_phy_initb5(struct bc
>> if (radio->version == 0x2050)
>> bcm43xx_phy_write(bcm, 0x0038, 0x0667);
>>
>> - if (phy->connected) {
>> + if (phy->type == BCM43xx_PHYTYPE_G) {
>
> I don't think this is correct.
> I think (didn't verify) the specs talk about the "gmode" here.
> I think you have been misleaded by that. The "gmode" bit is
> the "connected" bit in the old softmac driver.
>
The specs definitely say "If this is a GPHY" is step 5 of http://bcm-specs.sipsolutions.net/B5PHY.
Larry
-
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
