it's 6352 dude

On 10/10/2018, Stanislaw Gruszka <sgrus...@redhat.com> wrote:
> We do not need separate lines for calculating register values.
> Also add comment that value is different than in vendor driver.
>
> Suggested-by: Daniel Golle <dan...@makrotopia.org>
> Signed-off-by: Stanislaw Gruszka <sgrus...@redhat.com>
> ---
>  drivers/net/wireless/ralink/rt2x00/rt2800lib.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> index a2cdd3a5034a..7b6effaa0740 100644
> --- a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> +++ b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> @@ -3986,9 +3986,12 @@ static void rt2800_config_channel(struct rt2x00_dev
> *rt2x00dev,
>               rt2800_bbp_write(rt2x00dev, 195, 141);
>               rt2800_bbp_write(rt2x00dev, 196, reg);
>
> -             /* AGC init */
> -             reg = rf->channel <= 14 ? 0x1c : 0x24;
> -             reg += 2 * rt2x00dev->lna_gain;
> +             /* AGC init.
> +              * Despite the vendor driver using different values here for
> +              * RT6362 chip, we use 0x1c for now. This may have to be changed
> +              * once TSSI got implemented.
> +              */
> +             reg = (rf->channel <= 14 ? 0x1c : 0x24) + 2*rt2x00dev->lna_gain;
>               rt2800_bbp_write_with_rx_chain(rt2x00dev, 66, reg);
>
>               rt2800_iq_calibrate(rt2x00dev, rf->channel);
> --
> 2.7.5
>
>

Reply via email to