Commit 473baeab authored by Rafał Miłecki's avatar Rafał Miłecki Committed by Florian Fainelli
Browse files

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: ba4aebce ("ARM: dts: BCM5301X: Describe switch ports in the main DTS")
Signed-off-by: default avatarRafał Miłecki <rafal@milecki.pl>
Link: https://lore.kernel.org/r/20231013103314.10306-1-zajec5@gmail.com


Signed-off-by: default avatarFlorian Fainelli <florian.fainelli@broadcom.com>
parent b8d4f7c1
Loading
Loading
Loading
Loading
+8 −0
Original line number Diff line number Diff line
@@ -181,5 +181,13 @@ port@4 {
		port@5 {
			label = "cpu";
		};

		port@7 {
			status = "disabled";
		};

		port@8 {
			status = "disabled";
		};
	};
};
+8 −0
Original line number Diff line number Diff line
@@ -85,5 +85,13 @@ port@4 {
		port@5 {
			label = "cpu";
		};

		port@7 {
			status = "disabled";
		};

		port@8 {
			status = "disabled";
		};
	};
};
+8 −0
Original line number Diff line number Diff line
@@ -88,5 +88,13 @@ port@4 {
		port@5 {
			label = "cpu";
		};

		port@7 {
			status = "disabled";
		};

		port@8 {
			status = "disabled";
		};
	};
};
+8 −0
Original line number Diff line number Diff line
@@ -122,5 +122,13 @@ port@4 {
		port@5 {
			label = "cpu";
		};

		port@7 {
			status = "disabled";
		};

		port@8 {
			status = "disabled";
		};
	};
};
+8 −0
Original line number Diff line number Diff line
@@ -145,6 +145,14 @@ port@4 {
		port@5 {
			label = "cpu";
		};

		port@7 {
			status = "disabled";
		};

		port@8 {
			status = "disabled";
		};
	};
};

Loading