Return-path: Received: from mail-bw0-f219.google.com ([209.85.218.219]:47390 "EHLO mail-bw0-f219.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755444Ab0BESKL (ORCPT ); Fri, 5 Feb 2010 13:10:11 -0500 Received: by bwz19 with SMTP id 19so32840bwz.28 for ; Fri, 05 Feb 2010 10:10:00 -0800 (PST) Message-ID: <4B6C5EF3.5060006@lwfinger.net> Date: Fri, 05 Feb 2010 12:09:55 -0600 From: Larry Finger MIME-Version: 1.0 To: =?UTF-8?B?R8OhYm9yIFN0ZWZhbmlr?= CC: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , linux-wireless@vger.kernel.org, "John W. Linville" , bcm43xx-dev@lists.berlios.de Subject: Re: [RFC][PATCH] b43: LP-PHY: always adjust gain table on channel switch References: <1265317065-2922-2-git-send-email-zajec5@gmail.com> <4B6B8F8B.6020908@lwfinger.net> <69e28c911002050841x20053c6jbe768c19e983ef50@mail.gmail.com> In-Reply-To: <69e28c911002050841x20053c6jbe768c19e983ef50@mail.gmail.com> Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 02/05/2010 10:41 AM, Gábor Stefanik wrote: > On Fri, Feb 5, 2010 at 4:24 AM, Larry Finger wrote: >> On 02/04/2010 02:57 PM, Rafał Miłecki wrote: >>> --- >>> Gábor: I think you missed specs here. Could you check whole routine just for >>> sure, please? I don't understand whole radio and chanspec magic yet. >>> --- >>> drivers/net/wireless/b43/phy_lp.c | 2 +- >>> 1 files changed, 1 insertions(+), 1 deletions(-) >>> >>> diff --git a/drivers/net/wireless/b43/phy_lp.c b/drivers/net/wireless/b43/phy_lp.c >>> index 185219e..61009ee 100644 >>> --- a/drivers/net/wireless/b43/phy_lp.c >>> +++ b/drivers/net/wireless/b43/phy_lp.c >>> @@ -2655,8 +2655,8 @@ static int b43_lpphy_op_switch_channel(struct b43_wldev *dev, >>> if (err) >>> return err; >>> lpphy_set_analog_filter(dev, new_channel); >>> - lpphy_adjust_gain_table(dev, channel2freq_lp(new_channel)); >>> } >>> + lpphy_adjust_gain_table(dev, channel2freq_lp(new_channel)); >>> >>> lpphy->channel = new_channel; >>> b43_write16(dev, B43_MMIO_CHANNEL, new_channel); >> >> Both the lpphy_set_analog_filter() and lpphy_adjust_gain_table() calls should be >> outside the if statement. I changed the spec a little. It used to test "radio >> enabled", but I have found that is always true for our driver. >> >> Larry >> >> > > Isn't set_analog_filter() rev0/1-specific? It was in the 4.174.64.19 driver that I RE'd when you wrote the LP PHY code. That as changed in 5.10.56.46, which I am now doing. It will take me a while to complete the new routine "LP PHY TX Filter Init" and a routine that it calls. Certainly, there is no hurry that these changes be made. Whenever you or Rafał have time. There is no guarantee that these changes will have any effect on the LP PHY operations. Hitting a moving target is not easy. Larry