On 05/08/2011 12:09 PM, RafaÅ MiÅecki wrote:
There is no perfect place for this workarounds, but keeping it in ssb sounds
more reasonable.
1) We already have in ssb similar workarounds for different cores
2) We do not register not working devices in system
3) We do not duplicate code in b43 and b43legacy
RafaÅ MiÅecki (3):
ssb: update list of devices supporting multiple 80211 cores
b43: drop ssb-duplicated workaround for dangling cores
b43legacy: drop ssb-duplicated workaround for dangling cores
drivers/net/wireless/b43/main.c | 17 -----------------
drivers/net/wireless/b43legacy/main.c | 18 ------------------
drivers/ssb/scan.c | 5 ++++-
3 files changed, 4 insertions(+), 36 deletions(-)
First of all, a minor grammatical mistake. It should be "this workaround" as
there is only only one workaround discussed here.
Secondly, "we do not register not working" is a little awkward. "We do not
register inactive devices..." would be better.
Finally, this commit message will never show up in the git commit logs. I think
this info is valuable, thus it should be in the real commits.
My suggestion for the [1/3] patch is something like:
"Many of the BCM43xx chips contain cores that are attached to the SSB, but are
inactive as they do not connect to the external environment. These must not be
registered. Several of these types are handled in driver ssb; however, the
specific case of an inactive 802.11 cores is now treated in b43 and b43legacy.
Although the current setup works, this minor change will place all such
workarounds in ssb, and simplify the code in drivers b43 and b43legacy."
In each of the other two patches, add a commit message something like:
"Remove the code to detect inactive 802.11 cores, as that function is now done
in ssb."
I have not yet tested the patches, but will do so soon.
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
