[PATCH 1/4] b43: N-PHY: update init tables

[Rafał Miłecki]

W dniu 10 grudnia 2010 20:09 użytkownik  <ikorot at earthlink.net> napisał:
> Rafal,
>
>
> -----Original Message-----
>>From: Rafał Miłecki <zajec5 at gmail.com>
>>Sent: Dec 9, 2010 10:36 PM
>>To: ikorot at earthlink.net
>>Cc: Larry Finger <Larry.Finger at lwfinger.net>, Hauke Mehrtens <hauke at hauke-m.de>, linux-wireless at vger.kernel.org, b43-dev at lists.infradead.org
>>Subject: Re: [PATCH 1/4] b43: N-PHY: update init tables
>>
>>W dniu 10 grudnia 2010 03:26 użytkownik  <ikorot at earthlink.net> napisał:
>>> -----Original Message-----
>>>>From: Rafał Miłecki <zajec5 at gmail.com>
>>>>Sent: Dec 9, 2010 6:06 AM
>>>>To: ikorot at earthlink.net
>>>>Cc: Larry Finger <Larry.Finger at lwfinger.net>, Hauke Mehrtens <hauke at hauke-m.de>, linux-wireless at vger.kernel.org, b43-dev at lists.infradead.org
>>>>Subject: Re: [PATCH 1/4] b43: N-PHY: update init tables
>>>>
>>>>2010/12/8  <ikorot at earthlink.net>:
>>>>> Larry,
>>>>>
>>>>>
>>>>> -----Original Message-----
>>>>>>From: Larry Finger <Larry.Finger at lwfinger.net>
>>>>>>Sent: Dec 7, 2010 12:00 PM
>>>>>>To: ikorot at earthlink.net
>>>>>>Cc: Hauke Mehrtens <hauke at hauke-m.de>, linux-wireless at vger.kernel.org, b43-dev at lists.infradead.org
>>>>>>Subject: Re: [PATCH 1/4] b43: N-PHY: update init tables
>>>>>>
>>>>>>On 12/07/2010 01:49 PM, ikorot at earthlink.net wrote:
>>>>>>>> One reason is that these tables changed from non-zero to zero values between
>>>>>>>> Broadcom driver 4.174.64.19 and 5.10.56.46. As they might change again, I think
>>>>>>>> we should retain the full version.
>>>>>>>
>>>>>>> Is there a way to check for version of the driver?
>>>>>>> This way whoever uses old one won't be screwed...
>>>>>>
>>>>>>That will not be a problem as no older version of b43 works at all with b43. To
>>>>>>help you understand this change, when the 4.174.64.19 Broadcom driver was
>>>>>>reverse-engineered, the tables had non-zero values. Rafel later compared the
>>>>>>trace dumps of b43 with the latest version of wl and found that the values are
>>>>>>now zero. When I rechecked driver 5.10.56.46, I found them to be zero there as
>>>>>>well and changed the specs.
>>>>>
>>>>> Well, it maybe feasible to keep the old values and check
>>>>> the version of the driver, since you are saying yourself
>>>>> that the value might be changing in the future.
>>>>> Maybe for couple of releases only?
>>>>
>>>>Sorry, but I don't understand that at all. What do you mean by version
>>>>of the driver? What driver?
>>>
>>> #if B43_VERSION <= x.x.x.x
>>> // table initialized to some values
>>> #else
>>> //table initialized to 0's
>>> #endif
>>
>>You want to put condition checking b43 version in... b43 driver? I
>>can't follow you.
>
> Yes.
> And what you can't follow?
>
> For historical purposes and to better understand the changes and ease the
> debugging I proposed this addition.
> Besides if for some reason in future versions Broadcom will decide to return to
> the same values for init tables, the changes will be very simple.

So I guess you meant using "#if 0" for commenting old values?

I guess we could do that instead deleting old values, but fortunately
it's git so we always can browse history and restore old values
without problems :)

-- 
Rafał