realtek: set up L2 table entries properly
authorJan Hoffmann <jan@3e8.eu>
Tue, 25 Oct 2022 22:20:04 +0000 (00:20 +0200)
committerSander Vanheule <sander@svanheule.net>
Wed, 26 Oct 2022 07:59:24 +0000 (09:59 +0200)
commit2088e440b1325ca87e4aa30162e7baa03850a37b
tree54d3c78a62cd1198bd9e3a2c5de5b13a610e2b61
parent80a62a675d2ef57a1885e1829926dd2ea1682612
realtek: set up L2 table entries properly

Initialize the data structure using memset to avoid the possibility of
writing garbage values to the hardware.

Always set a valid entry type, which should fix writing unicast entries
on RTL930x.

For unicast entries, set the is_static flag to prevent the switch from
aging them out.

Also set the rvid field for unicast entries. This is not strictly
necessary, as the switch fills it in automatically from a non-zero vid.
However, this makes the code consistent with multicast entry setup.

While at it, reorder the statements and fix some style issues (double
space, comma instead of semicolon at end of statement). Also remove the
unneeded priv parameter and debug print for the multicast entry setup
function.

Fixes: cde31976e37 ("realtek: Add support for Layer 2 Multicast")
Signed-off-by: Jan Hoffmann <jan@3e8.eu>
target/linux/realtek/files-5.10/drivers/net/dsa/rtl83xx/dsa.c