Re: [PATCH net 2/2] net: dsa: microchip: Fix LAN937X set_ageing_time function

From: Andrew Lunn
Date: Thu Dec 19 2024 - 11:59:03 EST


On Tue, Dec 17, 2024 at 06:02:24PM -0800, Tristram.Ha@xxxxxxxxxxxxx wrote:
> From: Tristram Ha <tristram.ha@xxxxxxxxxxxxx>
>
> The aging count is not a simple 20-bit value but comprises a 3-bit
> multiplier and a 20-bit second time. The code tries to use the
> original multiplier which is 4 as the second count is still 300 seconds
> by default.
>
> As the 20-bit number is now too large for practical use there is an option
> to interpret it as microseconds instead of seconds.
>
> Fixes: 2c119d9982b1 ("net: dsa: microchip: add the support for set_ageing_time")
> Signed-off-by: Tristram Ha <tristram.ha@xxxxxxxxxxxxx>

Reviewed-by: Andrew Lunn <andrew@xxxxxxx>

Andrew