net: dsa: b53: Deny enslaving port 7 for 7278 into a bridge
authorFlorian Fainelli <f.fainelli@gmail.com>
Mon, 30 Mar 2020 21:38:49 +0000 (14:38 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 31 Mar 2020 02:55:42 +0000 (19:55 -0700)
On 7278, port 7 connects to the ASP which should only receive frames
through the use of CFP rules, it is not desirable to have it be part of
a bridge at all since that would make it pick up unwanted traffic that
it may not even be able to filter or sustain.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/b53/b53_common.c

index 42c41b0916827578d9250bd8e32e6d7832be13e9..68e2381694b9ebd217a8e31debddcc7097ef6caa 100644 (file)
@@ -1728,6 +1728,12 @@ int b53_br_join(struct dsa_switch *ds, int port, struct net_device *br)
        u16 pvlan, reg;
        unsigned int i;
 
+       /* On 7278, port 7 which connects to the ASP should only receive
+        * traffic from matching CFP rules.
+        */
+       if (dev->chip_id == BCM7278_DEVICE_ID && port == 7)
+               return -EINVAL;
+
        /* Make this port leave the all VLANs join since we will have proper
         * VLAN entries from now on
         */