On Thu, Nov 12, 2009 at 12:22:26AM +0100, Gertjan van Wingerde wrote: > On 11/11/09 23:07, John W. Linville wrote: > > On Mon, Nov 09, 2009 at 10:59:58PM +0100, Gertjan van Wingerde wrote: > >> Mirror the legacy Ralink driver with respect to rt2800 RF register programming. Execute > >> rt2800_config_channel_rt3x for all RF2020, RF3020, RF3021 & RF3022 chipsets when operating on RT3070 > >> devices. > >> > >> Signed-off-by: Gertjan van Wingerde <gwingerde@xxxxxxxxx> > >> --- > >> drivers/net/wireless/rt2x00/rt2800lib.c | 10 +++++++--- > >> 1 files changed, 7 insertions(+), 3 deletions(-) > >> > >> diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c > >> index 2c91c44..446087e 100644 > >> --- a/drivers/net/wireless/rt2x00/rt2800lib.c > >> +++ b/drivers/net/wireless/rt2x00/rt2800lib.c > >> @@ -806,10 +806,14 @@ static void rt2800_config_channel(struct rt2x00_dev *rt2x00dev, > >> unsigned int tx_pin; > >> u8 bbp; > >> > >> - if (rt2x00_rev(&rt2x00dev->chip) != RT3070_VERSION) > >> - rt2800_config_channel_rt2x(rt2x00dev, conf, rf, info); > >> - else > >> + if (rt2x00_rt(&rt2x00dev->chip, RT3070) && > >> + (rt2x00_rf(&rt2x00dev->chip, RF2020) || > >> + rt2x00_rf(&rt2x00dev->chip, RF3020) || > >> + rt2x00_rf(&rt2x00dev->chip, RF3021) || > >> + rt2x00_rf(&rt2x00dev->chip, RF3022))) > >> rt2800_config_channel_rt3x(rt2x00dev, conf, rf, info); > >> + else > >> + rt2800_config_channel_rt2x(rt2x00dev, conf, rf, info); > >> > >> /* > >> * Change BBP settings > > > > CC [M] drivers/net/wireless/rt2x00/rt2800lib.o > > drivers/net/wireless/rt2x00/rt2800lib.c: In function ‘rt2800_config_channel’: > > drivers/net/wireless/rt2x00/rt2800lib.c:809: error: ‘RF3070’ undeclared (first use in this function) > > drivers/net/wireless/rt2x00/rt2800lib.c:809: error: (Each undeclared identifier is reported only once > > drivers/net/wireless/rt2x00/rt2800lib.c:809: error: for each function it appears in.) > > > > Me very confused here. This patch doesn't even introduce the RF3070 symbol, nor can I see where it is used. > It does use symbol RT3070 on line 809, but that one is introduced in an earlier patch. > Ah, did you apply the RT3070 USB chipset detection patch before this one? > > Still wouldn't explain the compile error on symbol RF3070 though. RF3070 is a copy-n-paste error, sorry...the symbol is RT3070. What patch introduces it? John -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- 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