]> www.infradead.org Git - users/dwmw2/linux.git/commit
net: sparx5: fix default value of monitor ports
authorDaniel Machon <daniel.machon@microchip.com>
Thu, 5 Dec 2024 13:54:27 +0000 (14:54 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 9 Dec 2024 13:30:16 +0000 (13:30 +0000)
commite4d505fda6c81baf9b92a7f32f89912984654983
tree77954c543501cbc5800797982233e8edd049a8c7
parentf004f2e535e2b66ccbf5ac35f8eaadeac70ad7b7
net: sparx5: fix default value of monitor ports

When doing port mirroring, the physical port to send the frame to, is
written to the FRMC_PORT_VAL field of the QFWD_FRAME_COPY_CFG register.
This field is 7 bits wide on sparx5 and 6 bits wide on lan969x, and has
a default value of 65 and 30, respectively (the number of front ports).

On mirror deletion, we set the default value of the monitor port to
65 for this field, in case no more ports exists for the mirror. Needless
to say, this will not fit the 6 bits on lan969x.

Fix this by correctly using the n_ports constant instead.

Fixes: 3f9e46347a46 ("net: sparx5: use SPX5_CONST for constants which already have a symbol")
Signed-off-by: Daniel Machon <daniel.machon@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/microchip/sparx5/sparx5_mirror.c