diff options
author | Rafał Miłecki <rafal@milecki.pl> | 2023-10-13 13:33:13 +0300 |
---|---|---|
committer | Florian Fainelli <florian.fainelli@broadcom.com> | 2023-10-24 01:30:57 +0300 |
commit | 473baeab929444295b0530f8766e4becb6a08973 (patch) | |
tree | 4f9952d08b889e8a7b3f92cd4c56e6c057117382 /arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts | |
parent | b8d4f7c1be04d66c37c119c501c87bccc4197694 (diff) | |
download | linux-473baeab929444295b0530f8766e4becb6a08973.tar.xz |
ARM: dts: BCM5301X: Explicitly disable unused switch CPU ports
When redescribing ports I assumed that missing "label" (like "cpu")
means switch port isn't used. That was incorrect and I realized my
change made Linux always use the first (5) CPU port (there are 3 of
them).
While above should technically be possible it often isn't correct:
1. Non-default switch ports are often connected to Ethernet interfaces
not fully covered by vendor setup (they may miss MACs)
2. On some devices non-default ports require specifying fixed link
This fixes network connectivity for some devices. It was reported &
tested for Netgear R8000. It also affects Linksys EA9200 with its
downstream DTS.
Fixes: ba4aebce23b2 ("ARM: dts: BCM5301X: Describe switch ports in the main DTS")
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
Link: https://lore.kernel.org/r/20231013103314.10306-1-zajec5@gmail.com
Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com>
Diffstat (limited to 'arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts')
-rw-r--r-- | arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts b/arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts index 26c12bfb0bdd..25eeacf6a248 100644 --- a/arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts +++ b/arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts @@ -185,6 +185,14 @@ full-duplex; }; }; + + port@7 { + status = "disabled"; + }; + + port@8 { + status = "disabled"; + }; }; }; |