[PATCH 2/4] b43: set TMS to work with current band width for N-PHY

[Michael Büsch]

On Thu, 2010-12-09 at 15:13 +0100, Rafał Miłecki wrote: 
> W dniu 9 grudnia 2010 15:12 użytkownik Rafał Miłecki <zajec5 at gmail.com> napisał:
> > W dniu 7 grudnia 2010 22:08 użytkownik Michael Büsch <mb at bu3sch.de> napisał:
> >> On Tue, 2010-12-07 at 21:55 +0100, Rafał Miłecki wrote:
> >>> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
> >>> ---
> >>>  drivers/net/wireless/b43/main.c       |    6 ++++++
> >>>  drivers/net/wireless/b43/phy_common.c |    7 +++++++
> >>>  drivers/net/wireless/b43/phy_common.h |    2 ++
> >>>  drivers/net/wireless/b43/phy_n.c      |    7 -------
> >>>  4 files changed, 15 insertions(+), 7 deletions(-)
> >>>
> >>> diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
> >>> index fa48803..670fd7b 100644
> >>> --- a/drivers/net/wireless/b43/main.c
> >>> +++ b/drivers/net/wireless/b43/main.c
> >>> @@ -1150,6 +1150,12 @@ void b43_wireless_core_reset(struct b43_wldev *dev, u32 flags)
> >>>
> >>>       flags |= B43_TMSLOW_PHYCLKEN;
> >>>       flags |= B43_TMSLOW_PHYRESET;
> >>> +     if (dev->phy.type == B43_PHYTYPE_N) {
> >>> +             if (b43_channel_type_is_40mhz(dev->phy.channel_type))
> >>
> >> Is channel_type already set at this time?
> >
> > Yeah, that core switching seems to be a little tricky. I didn't figure
> > it out completely yet. We may need to fix that, or fix calls to this
> > function.
> >
> > Thanks for reviewing!
> 
> In other words: it will work for now, but we need to fix that in
> future if we want to get 40 MHz channels working.

That's exactly what I was saying. ;)
To answer my original question: No, channel_type is not set, but the
code will work for non-40mhz.

This seems a bit tricky to fix, though. Maybe we should reset the
wireless core twice. Once initially and once again after the
configuration was set. The first reset would just be to be able
to access the PHY and wireless core at all to probe it.

-- 
Greetings Michael.