On Sun, Apr 14, 2013 at 10:17:56PM +0200, Gertjan van Wingerde wrote: > >> + /* > >> + * Set back to initial state > >> + */ > >> + rt2800_bbp_write(rt2x00dev, 24, 0); > >> + > >> + rt2800_rfcsr_read(rt2x00dev, 22, &rfcsr); > >> + rt2x00_set_field8(&rfcsr, RFCSR22_BASEBAND_LOOPBACK, 0); > >> + rt2800_rfcsr_write(rt2x00dev, 22, rfcsr); > >> + > >> + /* > >> + * Set BBP back to BW20 > >> + */ > >> + rt2800_bbp_read(rt2x00dev, 4, &bbp); > >> + rt2x00_set_field8(&bbp, BBP4_BANDWIDTH, 0); > >> + rt2800_bbp_write(rt2x00dev, 4, bbp); [snip] > >> + > >> + rt2800_rx_filter_calibration(rt2x00dev); > >> } > > > > This looks a bit fishy to me, as the original code doesn't completely apply to RT3290, but now it does. On vendor DPO_RT3290_LinuxSTA_V2600_20120508 we do not do this "set back to initial state" code at all, so I'll remove rt2800_rx_filter_calibration() from 3290 procedure. Stanislaw -- 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