Commit c3cf059a authored by Álvaro Fernández Rojas's avatar Álvaro Fernández Rojas Committed by Jakub Kicinski
Browse files

net: dsa: b53: support legacy FCS tags 



Commit 46c5176c ("net: dsa: b53: support legacy tags") introduced
support for legacy tags, but it turns out that BCM5325 and BCM5365
switches require the original FCS value and length, so they have to be
treated differently.

Reviewed-by: default avatarFlorian Fainelli <florian.fainelli@broadcom.com>
Signed-off-by: default avatarÁlvaro Fernández Rojas <noltari@gmail.com>
Link: https://patch.msgid.link/20250614080000.1884236-4-noltari@gmail.com


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

drivers/net/dsa/b53/Kconfig

+1 −0
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ menuconfig B53 
	select NET_DSA_TAG_NONE

	select NET_DSA_TAG_BRCM

	select NET_DSA_TAG_BRCM_LEGACY

	select NET_DSA_TAG_BRCM_LEGACY_FCS

	select NET_DSA_TAG_BRCM_PREPEND

	help

	  This driver adds support for Broadcom managed switch chips. It supports

drivers/net/dsa/b53/b53_common.c

+5 −2
Original line number Diff line number Diff line
@@ -2241,8 +2241,11 @@ enum dsa_tag_protocol b53_get_tag_protocol(struct dsa_switch *ds, int port, 
		goto out;

	}



	/* Older models require a different 6 byte tag */

	if (is5325(dev) || is5365(dev) || is63xx(dev)) {

	/* Older models require different 6 byte tags */

	if (is5325(dev) || is5365(dev)) {

		dev->tag_protocol = DSA_TAG_PROTO_BRCM_LEGACY_FCS;

		goto out;

	} else if (is63xx(dev)) {

		dev->tag_protocol = DSA_TAG_PROTO_BRCM_LEGACY;

		goto out;

	}