Stanislaw Gruszka <[email protected]> writes:

> There is duplicated 'if (rt2x00_rt(rt2x00dev, RT6352))' entry that
> causes we do not perform register initialization for RT6352 (MT7620
> SOCs) in correct branch. Fix this and disable registers initialization
> that is specific to particular MT7620 revision.
>
> Signed-off-by: Stanislaw Gruszka <[email protected]>

[...]

> @@ -5466,6 +5465,10 @@ static int rt2800_init_registers(struct rt2x00_dev 
> *rt2x00dev)
>               rt2800_register_write(rt2x00dev, TX_SW_CFG0, 0x00000401);
>               rt2800_register_write(rt2x00dev, TX_SW_CFG1, 0x000C0000);
>               rt2800_register_write(rt2x00dev, TX_SW_CFG2, 0x000C0408);
> +             /* TODO add chip version support and init registers
> +              * according to the version.
> +              */
> +#if 0
>               rt2800_register_write(rt2x00dev, MIMO_PS_CFG, 0x00000002);
>               rt2800_register_write(rt2x00dev, TX_PIN_CFG, 0x00150F0F);
>               rt2800_register_write(rt2x00dev, TX_ALC_VGA3, 0x06060606);
> @@ -5480,6 +5483,7 @@ static int rt2800_init_registers(struct rt2x00_dev 
> *rt2x00dev)
>               reg = rt2800_register_read(rt2x00dev, TX_ALC_CFG_1);
>               rt2x00_set_field32(&reg, TX_ALC_CFG_1_ROS_BUSY_EN, 0);
>               rt2800_register_write(rt2x00dev, TX_ALC_CFG_1, reg);
> +#endif

No '#if 0', please. If the code is not needed you can remove it, it's
available from git history anyway if it's needed later.

-- 
Kalle Valo

Reply via email to