Re: [RFC net-next v2 3/4] net: dsa: mt7530: set STP state also on filter ID 1
From: DENG Qingfang <dqfext@gmail.com>
Date: 2021-08-02 15:58:23
Also in:
linux-arm-kernel, linux-mediatek, lkml
On Mon, Aug 02, 2021 at 06:42:26PM +0300, Vladimir Oltean wrote:
On Mon, Aug 02, 2021 at 11:31:29PM +0800, DENG Qingfang wrote:quoted
The current code only sets FID 0's STP state. This patch sets both 0's and 1's states. The *5 part is binary magic. [1:0] is FID 0's state, [3:2] is FID 1's state and so on. Since 5 == 4'b0101, the value in [1:0] is copied to [3:2] after the multiplication. Perhaps I should only change FID 1's state.Keep the patches dumb for us mortals please. If you only change FID 1's state, I am concerned that the driver no longer initializes FID 0's port state, and might leave that to the default set by other pre-kernel initialization stage (bootloader?). So even if you might assume that standalone ports are FORWARDING, they might not be.
The default value is forwarding, and the switch is reset by the driver so any pre-kernel initialization stage is no more.