diff mbox series

[3/3] ramips: hlk-7621a-evb: rename switch ports

Message ID 20220413120554.9355-3-reinhard@m4x.de
State New
Headers show
Series [1/3] ramips: hlk-7621a-evb: remove WiFi support | expand

Commit Message

Reinhard Max April 13, 2022, 12:05 p.m. UTC
Rename the switch ports from 'lan[1..4]' and 'wan' to 'p[0..4]'. This
matches the SoC documentation and the silkscreen for the port LEDs on
the module. There is no LAN/WAN labeling on the board itself.

This eval board is so universal (many additional interfaces broken out
on pin headers, mPCIe slot, USB, etc.) that chances are high that is
being used for other things than a traditional LAN/WAN router, and then
the 'wan' naming of the 5th port might get into the way, so better stick
with the SoC naming.

Signed-off-by: Reinhard Max <reinhard@m4x.de>
---
 .../linux/ramips/dts/mt7621_hilink_hlk-7621a-evb.dts   | 10 +++++-----
 .../ramips/mt7621/base-files/etc/board.d/02_network    |  3 +++
 2 files changed, 8 insertions(+), 5 deletions(-)

Comments

Reinhard Max June 2, 2022, 5:42 p.m. UTC | #1
Hi,

I would like to ask for a discussion and/or decision on this one.

Am 13.04.2022 13:42, schrieb Reinhard Max:
> Rename the switch ports from 'lan[1..4]' and 'wan' to 'p[0..4]'. This
> matches the SoC documentation and the silkscreen for the port LEDs on
> the module. There is no LAN/WAN labeling on the board itself.
> 
> This eval board is so universal (many additional interfaces broken out
> on pin headers, mPCIe slot, USB, etc.) that chances are high that is
> being used for other things than a traditional LAN/WAN router, and then
> the 'wan' naming of the 5th port might get into the way, so better 
> stick
> with the SoC naming.

cu
Reinhard
diff mbox series

Patch

diff --git a/target/linux/ramips/dts/mt7621_hilink_hlk-7621a-evb.dts b/target/linux/ramips/dts/mt7621_hilink_hlk-7621a-evb.dts
index 20e99352f3..6b22e04b35 100644
--- a/target/linux/ramips/dts/mt7621_hilink_hlk-7621a-evb.dts
+++ b/target/linux/ramips/dts/mt7621_hilink_hlk-7621a-evb.dts
@@ -65,27 +65,27 @@ 
 	ports {
 		port@0 {
 			status = "okay";
-			label = "lan1";
+			label = "p0";
 		};
 
 		port@1 {
 			status = "okay";
-			label = "lan2";
+			label = "p1";
 		};
 
 		port@2 {
 			status = "okay";
-			label = "lan3";
+			label = "p2";
 		};
 
 		port@3 {
 			status = "okay";
-			label = "lan4";
+			label = "p3";
 		};
 
 		port@4 {
 			status = "okay";
-			label = "wan";
+			label = "p4";
 		};
 	};
 };
diff --git a/target/linux/ramips/mt7621/base-files/etc/board.d/02_network b/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
index 1fc0ccdd5b..eb931defaa 100644
--- a/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
+++ b/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
@@ -55,6 +55,9 @@  ramips_setup_interfaces()
 	gnubee,gb-pc2)
 		ucidef_set_interface_lan "lan1 lan2"
 		;;
+	hilink,hlk-7621a-evb)
+		ucidef_set_interface_lan "p0 p1 p2 p3 p4"
+		;;
 	linksys,re6500|\
 	netgear,wac104)
 		ucidef_set_interface_lan "lan1 lan2 lan3 lan4"