On Wed, Apr 10, 2013 at 09:12:22AM +0200, Gabor Juhos wrote:
> > - if (rt2x00_rt(rt2x00dev, RT3290) ||
> > - rt2x00_rt(rt2x00dev, RT5390) ||
> > - rt2x00_rt(rt2x00dev, RT5392)) {
> > - rt2800_rfcsr_read(rt2x00dev, 2, &rfcsr);
> > - rt2x00_set_field8(&rfcsr, RFCSR2_RESCAL_EN, 1);
> > - rt2800_rfcsr_write(rt2x00dev, 2, rfcsr);
> > - msleep(1);
> > - rt2x00_set_field8(&rfcsr, RFCSR2_RESCAL_EN, 0);
> > - rt2800_rfcsr_write(rt2x00dev, 2, rfcsr);
> > - } else {
> > - rt2800_rfcsr_read(rt2x00dev, 30, &rfcsr);
> > - rt2x00_set_field8(&rfcsr, RFCSR30_RF_CALIBRATION, 1);
> > - rt2800_rfcsr_write(rt2x00dev, 30, rfcsr);
> > - msleep(1);
> > - rt2x00_set_field8(&rfcsr, RFCSR30_RF_CALIBRATION, 0);
> > - rt2800_rfcsr_write(rt2x00dev, 30, rfcsr);
> > - }
>
> The code in the else branch is required by the 305x_soc devices. However the
> change moves it into the 'rt2800_init_rfcsr_30xx' function and that is only
> called for RT3070, RT3071, RT3090.
>
> Either add the code into the 'rt2800_init_rfcsr_305x_soc function' as well or
> create a helper function and call that from 'rt2800_init_rfcsr_30xx' and from
> 'rt2800_init_rfcsr_305x_soc'.
Right. 3352 and 3572 need that too. I'll post v2 shortly.
Thanks
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
