John W. Linville wrote:
> BCM43xx_LOCALE_USA_CANADA_ANZ a correct grouping?

John,

Those categories were copied from the data in the SPROM on the BCM43xx chips, 
which are obviously 
out of date. That said, it is likely that the code will do the right thing in 
Australia and New 
Zealand as most of the cards have a 0 in the locale field of the SPROM, which 
falls through to the 
default case of the switch.

Larry

> John
> 
>>
>> Hi John,
>>
>> Apols for unsolicted email but I was looking through your patch and this
>> caught my eye:
>>> +/* set the maximum channel based on locale set in sprom or witle locale 
>>> option */
>>> +   switch (bcm->sprom.locale) {
>>> +   case BCM43xx_LOCALE_THAILAND:
>>> +   case BCM43xx_LOCALE_ISRAEL:
>>> +   case BCM43xx_LOCALE_JORDAN:
>>> +   case BCM43xx_LOCALE_USA_CANADA_ANZ:
>>> +   case BCM43xx_LOCALE_USA_LOW:
>>> +           max_bg_channel = 11;
>>> +           break;
>>> +   case BCM43xx_LOCALE_JAPAN:
>>> +   case BCM43xx_LOCALE_JAPAN_HIGH:
>>> +           max_bg_channel = 14;
>>> +           break;
>>> +   default:
>>> +           max_bg_channel = 13;
>>> +   }
>>> +
>>>   
>> Specifically BCM43xx_LOCALE_USA_CANADA_ANZ. A/NZ is certainly not the
>> same as USA/Canada - in Australia we get 13 channels not 11 of US/Canada.
>>
>> I'm sure this is because I don't know your driver but just in case you
>> do need to split out an ANZ definition I thought I'd shout up.
>>
>> Steve
> 
> ----- End forwarded message -----
> 

_______________________________________________
Bcm43xx-dev mailing list
[EMAIL PROTECTED]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to