Commit 1833d8a2 authored by Peter Rashleigh's avatar Peter Rashleigh Committed by Jakub Kicinski
Browse files

net: dsa: mv88e6xxx: Fix the max_vid definition for the MV88E6361 



According to the Marvell datasheet the 88E6361 has two VTU pages
(4k VIDs per page) so the max_vid should be 8191, not 4095.

In the current implementation mv88e6xxx_vtu_walk() gives unexpected
results because of this error. I verified that mv88e6xxx_vtu_walk()
works correctly on the MV88E6361 with this patch in place.

Fixes: 12899f29 ("net: dsa: mv88e6xxx: enable support for 88E6361 switch")
Signed-off-by: default avatarPeter Rashleigh <peter@rashleigh.ca>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/20241014204342.5852-1-peter@rashleigh.ca


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent e8c526f2

drivers/net/dsa/mv88e6xxx/chip.c

+1 −1
Original line number Diff line number Diff line
@@ -6347,7 +6347,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = { 
		.invalid_port_mask = BIT(1) | BIT(2) | BIT(8),

		.num_internal_phys = 5,

		.internal_phys_offset = 3,

		.max_vid = 4095,

		.max_vid = 8191,

		.max_sid = 63,

		.port_base_addr = 0x0,

		.phy_base_addr = 0x0,