From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael =?ISO-8859-1?Q?B=FCsch?= Date: Thu, 09 Dec 2010 15:19:44 +0100 Subject: [PATCH 2/4] b43: set TMS to work with current band width for N-PHY In-Reply-To: (sfid-20101209_151338_363925_536AF3B3) References: <1291755360-21570-1-git-send-email-zajec5@gmail.com> <1291755360-21570-3-git-send-email-zajec5@gmail.com> <1291756117.27497.101.camel@maggie> (sfid-20101209_151338_363925_536AF3B3) Message-ID: <1291904384.2905.2.camel@maggie> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= Cc: linux-wireless@vger.kernel.org, "John W. Linville" , b43-dev@lists.infradead.org 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 napisa?: > > W dniu 7 grudnia 2010 22:08 u?ytkownik Michael B?sch napisa?: > >> On Tue, 2010-12-07 at 21:55 +0100, Rafa? Mi?ecki wrote: > >>> Signed-off-by: Rafa? Mi?ecki > >>> --- > >>> 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.