summaryrefslogtreecommitdiff
path: root/arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts
diff options
context:
space:
mode:
authorRafał Miłecki <rafal@milecki.pl>2023-10-13 13:33:13 +0300
committerFlorian Fainelli <florian.fainelli@broadcom.com>2023-10-24 01:30:57 +0300
commit473baeab929444295b0530f8766e4becb6a08973 (patch)
tree4f9952d08b889e8a7b3f92cd4c56e6c057117382 /arch/arm/boot/dts/broadcom/bcm53016-meraki-mr32.dts
parentb8d4f7c1be04d66c37c119c501c87bccc4197694 (diff)
downloadlinux-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.dts8
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";
+ };
};
};