[openwrt/openwrt] ath79: use "fixed-layout" for Embedded Wireless devices

LEDE Commits lede-commits at lists.infradead.org
Tue Nov 14 22:14:41 PST 2023


rmilecki pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/c60e9d183995823a52669ecae40669154614abe7

commit c60e9d183995823a52669ecae40669154614abe7
Author: Rafał Miłecki <rafal at milecki.pl>
AuthorDate: Fri Nov 10 11:25:05 2023 +0100

    ath79: use "fixed-layout" for Embedded Wireless devices
    
    Those devices have Ethernet interfaces using base MAC address increased
    by 0x40 in the 3rd indexed byte (00:00:00:FF:00:00). To describe that we
    were using a custom (downstream) "mac-address-increment-byte" property.
    
    The same result can be achieved by using "mac-base" with a properly
    adjusted offset value (0x40 << 16). It may be not pretty but it should
    work without custom property or downstream kernel patch to support it.
    
    Cc: Ansuel Smith <ansuelsmth at gmail.com>
    Cc: Catrinel Catrinescu <cc at 80211.de>
    Cc: Felix Fietkau <nbd at nbd.name>
    Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
    Reviewed-by: Rosen Penev <rosenp at gmail.com>
---
 .../ath79/dts/ar9331_embeddedwireless_dorin.dts    | 26 +++++++++----------
 .../ath79/dts/ar9344_embeddedwireless_balin.dts    | 30 ++++++++++++----------
 2 files changed, 29 insertions(+), 27 deletions(-)

diff --git a/target/linux/ath79/dts/ar9331_embeddedwireless_dorin.dts b/target/linux/ath79/dts/ar9331_embeddedwireless_dorin.dts
index de6b709b5c..6286f203ef 100644
--- a/target/linux/ath79/dts/ar9331_embeddedwireless_dorin.dts
+++ b/target/linux/ath79/dts/ar9331_embeddedwireless_dorin.dts
@@ -85,6 +85,18 @@
 				label = "art";
 				reg = <0xff0000 0x010000>;
 				read-only;
+
+				nvmem-layout {
+					compatible = "fixed-layout";
+					#address-cells = <1>;
+					#size-cells = <1>;
+
+					macaddr_art_1002: macaddr at 1002 {
+						compatible = "mac-base";
+						reg = <0x1002 0x6>;
+						#nvmem-cell-cells = <1>;
+					};
+				};
 			};
 		};
 	};
@@ -93,10 +105,8 @@
 &eth1 {
 	status = "okay";
 
-	nvmem-cells = <&macaddr_art_1002>;
+	nvmem-cells = <&macaddr_art_1002 0x400000>;
 	nvmem-cell-names = "mac-address";
-	mac-address-increment-byte = <3>;
-	mac-address-increment = <0x40>;
 };
 
 &mdio1 {
@@ -108,13 +118,3 @@
 
 	mtd-cal-data = <&art 0x1000>;
 };
-
-&art {
-	compatible = "nvmem-cells";
-	#address-cells = <1>;
-	#size-cells = <1>;
-
-	macaddr_art_1002: macaddr at 1002 {
-		reg = <0x1002 0x6>;
-	};
-};
diff --git a/target/linux/ath79/dts/ar9344_embeddedwireless_balin.dts b/target/linux/ath79/dts/ar9344_embeddedwireless_balin.dts
index a84c273f86..de13865818 100644
--- a/target/linux/ath79/dts/ar9344_embeddedwireless_balin.dts
+++ b/target/linux/ath79/dts/ar9344_embeddedwireless_balin.dts
@@ -83,16 +83,20 @@
 				reg = <0xff0000 0x010000>;
 				read-only;
 
-				compatible = "nvmem-cells";
-				#address-cells = <1>;
-				#size-cells = <1>;
-
-				calibration_art_1000: calibration_data at 1000 {
-					reg = <0x1000 0x440>;
-				};
-
-				macaddr_art_1002: macaddr at 1002 {
-					reg = <0x1002 0x6>;
+				nvmem-layout {
+					compatible = "fixed-layout";
+					#address-cells = <1>;
+					#size-cells = <1>;
+
+					calibration_art_1000: calibration_data at 1000 {
+						reg = <0x1000 0x440>;
+					};
+
+					macaddr_art_1002: macaddr at 1002 {
+						compatible = "mac-base";
+						reg = <0x1002 0x6>;
+						#nvmem-cell-cells = <1>;
+					};
 				};
 			};
 		};
@@ -106,10 +110,8 @@
 &eth1 {
 	status = "okay";
 
-	nvmem-cells = <&macaddr_art_1002>;
+	nvmem-cells = <&macaddr_art_1002 0x400000>;
 	nvmem-cell-names = "mac-address";
-	mac-address-increment-byte = <3>;
-	mac-address-increment = <0x40>;
 
 	gmac-config {
 		device = <&gmac>;
@@ -121,7 +123,7 @@
 &wmac {
 	status = "okay";
 
-	nvmem-cells = <&macaddr_art_1002>, <&calibration_art_1000>;
+	nvmem-cells = <&macaddr_art_1002 0>, <&calibration_art_1000>;
 	nvmem-cell-names = "mac-address", "calibration";
 };
 




More information about the lede-commits mailing list