[PATCH 1/9] net: gmac_rockchip: Fix misuse of GENMASK macro

David Wu david.wu at rock-chips.com
Thu Apr 7 14:28:26 CEST 2022


Hi Pierre-Clément,

Thanks for your correction, there was wrong mask here.
Reviewed-by: David Wu <david.wu at rock-chips.com>

在 2022/4/6 23:08, Kever Yang 写道:
> Add David,
> 
> Hi David,
> 
>      Could you help to check this patch?
> 
> 
> Thanks,
> - Kever
> On 2022/3/16 23:39, Pierre-Clément Tosi wrote:
>> Swap the arguments as that seems to have been the author's intention.
>>
>> Note: This fix wasn't tested on hardware and will result in more bits
>>        being set by the underlying writel() in rk_clrsetreg(), which
>>        might bring unexpected behavioural changes.
>>
>> Fixes: b07911840025 ("net: gmac_rockchip: add support for px30")
>> Signed-off-by: Pierre-Clément Tosi <ptosi at google.com>
>> Cc: Joe Hershberger <joe.hershberger at ni.com>
>> Cc: Heiko Stuebner <heiko at sntech.de>
>> Cc: Kever Yang <kever.yang at rock-chips.com>
>> Cc: Tom Rini <trini at konsulko.com>
>> ---
>>   drivers/net/gmac_rockchip.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/gmac_rockchip.c b/drivers/net/gmac_rockchip.c
>> index 04008d2b19..0ecbcdf641 100644
>> --- a/drivers/net/gmac_rockchip.c
>> +++ b/drivers/net/gmac_rockchip.c
>> @@ -350,7 +350,7 @@ static void px30_gmac_set_to_rmii(struct 
>> gmac_rockchip_plat *pdata)
>>       struct px30_grf *grf;
>>       enum {
>>           PX30_GMAC_PHY_INTF_SEL_SHIFT = 4,
>> -        PX30_GMAC_PHY_INTF_SEL_MASK  = GENMASK(4, 6),
>> +        PX30_GMAC_PHY_INTF_SEL_MASK  = GENMASK(6, 4),
>>           PX30_GMAC_PHY_INTF_SEL_RMII  = BIT(6),
>>       };


More information about the U-Boot mailing list