[OpenWrt-Devel] [PATCH] ath79: rename EEPROM to art

Adrian Schmutzler mail at adrianschmutzler.de
Wed Nov 27 10:17:22 EST 2019


Hi David,

> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org] On
> Behalf Of David Bauer
> Sent: Freitag, 22. November 2019 22:40
> To: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> Cc: openwrt-devel at lists.openwrt.org
> Subject: Re: [OpenWrt-Devel] [PATCH] ath79: rename EEPROM to art
> 
> Hello Adrian,
> 
> On 11/22/19 2:45 PM, Adrian Schmutzler wrote:
> > This renames all remaining occurrences of "EEPROM" to "art" to
> > further harmonize the partition labelling in ath79.
> 
> Hmm, I'm not sure if/why we should to these changes.
> 
> Ubiquiti names these partitions "EEPROM" in their firmware
> and it contains more information (e.g. board-id / sub-revision)
> than just the calibration data.

similar to what Piotr stated, I just give the unification slightly higher priority than the consistency with the vendor names in this case.

Despite, note that for XM the partition is already named "art" in ath79, too, so we are also increasing consistency from that angle.

Best

Adrian

> 
> Best wishes
> David
> 
> >
> > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > ---
> >  .../linux/ath79/dts/ar9342_ubnt_lap-120.dts   |  2 +-
> >  .../ath79/dts/ar9342_ubnt_nanobeam-ac.dts     |  2 +-
> >  .../dts/ar9342_ubnt_nanostation-ac-loco.dts   |  2 +-
> >  .../ath79/dts/ar9342_ubnt_nanostation-ac.dts  |  4 ++--
> >  target/linux/ath79/dts/ar9342_ubnt_wa.dtsi    |  6 +++---
> >  target/linux/ath79/dts/ar9342_ubnt_xw.dtsi    |  8 ++++----
> >  .../linux/ath79/dts/qca9533_ubnt_acb-isp.dts  | 10 +++++-----
> >  .../ath79/dts/qca9563_ubnt_unifiac-lite.dtsi  |  2 +-
> >  .../ath79/dts/qca9563_ubnt_unifiac-pro.dtsi   |  2 +-
> >  .../linux/ath79/dts/qca9563_ubnt_unifiac.dtsi |  6 +++---
> >  .../etc/hotplug.d/firmware/11-ath10k-caldata  | 20 +++++++++----------
> >  11 files changed, 31 insertions(+), 33 deletions(-)
> >
> > diff --git a/target/linux/ath79/dts/ar9342_ubnt_lap-120.dts
> b/target/linux/ath79/dts/ar9342_ubnt_lap-120.dts
> > index 82f864b8e3..757654eaee 100644
> > --- a/target/linux/ath79/dts/ar9342_ubnt_lap-120.dts
> > +++ b/target/linux/ath79/dts/ar9342_ubnt_lap-120.dts
> > @@ -26,7 +26,7 @@
> >  	/* default for ar934x, except for 1000M and 10M */
> >  	pll-data = <0x06000000 0x00000101 0x00001313>;
> >
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >
> >  	phy-mode = "rgmii";
> >  	phy-handle = <&phy4>;
> > diff --git a/target/linux/ath79/dts/ar9342_ubnt_nanobeam-ac.dts
> b/target/linux/ath79/dts/ar9342_ubnt_nanobeam-ac.dts
> > index 6e64c7faad..30d054dfed 100644
> > --- a/target/linux/ath79/dts/ar9342_ubnt_nanobeam-ac.dts
> > +++ b/target/linux/ath79/dts/ar9342_ubnt_nanobeam-ac.dts
> > @@ -51,7 +51,7 @@
> >  	/* default for ar934x, except for 1000M and 10M */
> >  	pll-data = <0x06000000 0x00000101 0x00001313>;
> >
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >
> >  	phy-mode = "rgmii";
> >  	phy-handle = <&phy4>;
> > diff --git a/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac-loco.dts
> b/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac-loco.dts
> > index 9b26d1a628..89904721c0 100644
> > --- a/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac-loco.dts
> > +++ b/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac-loco.dts
> > @@ -27,7 +27,7 @@
> >  	/* default for ar934x, except for 1000M and 10M */
> >  	pll-data = <0x06000000 0x00000101 0x00001313>;
> >
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >
> >  	phy-mode = "rgmii";
> >  	phy-handle = <&phy4>;
> > diff --git a/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac.dts
> b/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac.dts
> > index b591925154..97597e5f15 100644
> > --- a/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac.dts
> > +++ b/target/linux/ath79/dts/ar9342_ubnt_nanostation-ac.dts
> > @@ -58,7 +58,7 @@
> >  	/* default for ar934x, except for 1000M and 10M */
> >  	pll-data = <0x06000000 0x00000101 0x00001313>;
> >
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >
> >  	phy-mode = "rgmii";
> >  	phy-handle = <&phy0>;
> > @@ -73,5 +73,5 @@
> >  &wmac {
> >  	status = "okay";
> >
> > -	mtd-cal-data = <&eeprom 0x1000>;
> > +	mtd-cal-data = <&art 0x1000>;
> >  };
> > diff --git a/target/linux/ath79/dts/ar9342_ubnt_wa.dtsi
> b/target/linux/ath79/dts/ar9342_ubnt_wa.dtsi
> > index 30fa299638..2847d4098c 100644
> > --- a/target/linux/ath79/dts/ar9342_ubnt_wa.dtsi
> > +++ b/target/linux/ath79/dts/ar9342_ubnt_wa.dtsi
> > @@ -75,8 +75,8 @@
> >  				read-only;
> >  			};
> >
> > -			eeprom: partition at ff0000 {
> > -				label = "EEPROM";
> > +			art: partition at ff0000 {
> > +				label = "art";
> >  				reg = <0xff0000 0x010000>;
> >  				read-only;
> >  			};
> > @@ -88,5 +88,5 @@
> >  	status = "okay";
> >
> >  	qca,disable-5ghz;
> > -	mtd-cal-data = <&eeprom 0x1000>;
> > +	mtd-cal-data = <&art 0x1000>;
> >  };
> > diff --git a/target/linux/ath79/dts/ar9342_ubnt_xw.dtsi
> b/target/linux/ath79/dts/ar9342_ubnt_xw.dtsi
> > index ee42498af1..cf24aba5aa 100644
> > --- a/target/linux/ath79/dts/ar9342_ubnt_xw.dtsi
> > +++ b/target/linux/ath79/dts/ar9342_ubnt_xw.dtsi
> > @@ -103,8 +103,8 @@
> >  				read-only;
> >  			};
> >
> > -			eeprom: partition at 7f0000 {
> > -				label = "EEPROM";
> > +			art: partition at 7f0000 {
> > +				label = "art";
> >  				reg = <0x7f0000 0x010000>;
> >  				read-only;
> >  			};
> > @@ -115,9 +115,9 @@
> >  &wmac {
> >  	status = "okay";
> >
> > -	mtd-cal-data = <&eeprom 0x1000>;
> > +	mtd-cal-data = <&art 0x1000>;
> >  };
> >
> >  &eth0 {
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >  };
> > diff --git a/target/linux/ath79/dts/qca9533_ubnt_acb-isp.dts
> b/target/linux/ath79/dts/qca9533_ubnt_acb-isp.dts
> > index 629899e1a2..ded95f3a18 100644
> > --- a/target/linux/ath79/dts/qca9533_ubnt_acb-isp.dts
> > +++ b/target/linux/ath79/dts/qca9533_ubnt_acb-isp.dts
> > @@ -64,8 +64,8 @@
> >  				read-only;
> >  			};
> >
> > -			eeprom: partition at ff0000 {
> > -				label = "EEPROM";
> > +			art: partition at ff0000 {
> > +				label = "art";
> >  				reg = <0xff0000 0x010000>;
> >  				read-only;
> >  			};
> > @@ -79,13 +79,13 @@
> >
> >  &eth0 {
> >  	status = "okay";
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >  	phy-handle = <&swphy4>;
> >  };
> >
> >  &eth1 {
> >  	status = "okay";
> > -	mtd-mac-address = <&eeprom 0x6>;
> > +	mtd-mac-address = <&art 0x6>;
> >
> >  	gmac-config {
> >  		device = <&gmac>;
> > @@ -94,5 +94,5 @@
> >
> >  &wmac {
> >  	status = "okay";
> > -	mtd-cal-data = <&eeprom 0x1000>;
> > +	mtd-cal-data = <&art 0x1000>;
> >  };
> > diff --git a/target/linux/ath79/dts/qca9563_ubnt_unifiac-lite.dtsi
> b/target/linux/ath79/dts/qca9563_ubnt_unifiac-lite.dtsi
> > index 4f77661fbd..8248e060c5 100644
> > --- a/target/linux/ath79/dts/qca9563_ubnt_unifiac-lite.dtsi
> > +++ b/target/linux/ath79/dts/qca9563_ubnt_unifiac-lite.dtsi
> > @@ -21,6 +21,6 @@
> >  &eth0 {
> >  	status = "okay";
> >
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >  	phy-handle = <&phy4>;
> >  };
> > diff --git a/target/linux/ath79/dts/qca9563_ubnt_unifiac-pro.dtsi
> b/target/linux/ath79/dts/qca9563_ubnt_unifiac-pro.dtsi
> > index 3a6c0cd3fa..5a0cf4bade 100644
> > --- a/target/linux/ath79/dts/qca9563_ubnt_unifiac-pro.dtsi
> > +++ b/target/linux/ath79/dts/qca9563_ubnt_unifiac-pro.dtsi
> > @@ -25,6 +25,6 @@
> >  &eth0 {
> >  	status = "okay";
> >
> > -	mtd-mac-address = <&eeprom 0x0>;
> > +	mtd-mac-address = <&art 0x0>;
> >  	phy-handle = <&phy0>;
> >  };
> > diff --git a/target/linux/ath79/dts/qca9563_ubnt_unifiac.dtsi
> b/target/linux/ath79/dts/qca9563_ubnt_unifiac.dtsi
> > index bb20caba47..bfad3ff306 100644
> > --- a/target/linux/ath79/dts/qca9563_ubnt_unifiac.dtsi
> > +++ b/target/linux/ath79/dts/qca9563_ubnt_unifiac.dtsi
> > @@ -104,8 +104,8 @@
> >  				read-only;
> >  			};
> >
> > -			eeprom: partition at ff0000 {
> > -				label = "EEPROM";
> > +			art: partition at ff0000 {
> > +				label = "art";
> >  				reg = <0xff0000 0x010000>;
> >  				read-only;
> >  			};
> > @@ -116,5 +116,5 @@
> >  &wmac {
> >  	status = "okay";
> >
> > -	mtd-cal-data = <&eeprom 0x1000>;
> > +	mtd-cal-data = <&art 0x1000>;
> >  };
> > diff --git a/target/linux/ath79/generic/base-files/etc/hotplug.d/firmware/11-
> ath10k-caldata b/target/linux/ath79/generic/base-
> files/etc/hotplug.d/firmware/11-ath10k-caldata
> > index f128e8759f..258efc3bf4 100644
> > --- a/target/linux/ath79/generic/base-files/etc/hotplug.d/firmware/11-ath10k-
> caldata
> > +++ b/target/linux/ath79/generic/base-files/etc/hotplug.d/firmware/11-
> ath10k-caldata
> > @@ -12,6 +12,15 @@ case "$FIRMWARE" in
> >  	case $board in
> >  	comfast,cf-wr650ac-v1|\
> >  	comfast,cf-wr650ac-v2|\
> > +	ubnt,unifiac-lite|\
> > +	ubnt,unifiac-lr|\
> > +	ubnt,unifiac-mesh|\
> > +	ubnt,unifiac-mesh-pro|\
> > +	ubnt,lap-120|\
> > +	ubnt,nanobeam-ac|\
> > +	ubnt,nanostation-ac|\
> > +	ubnt,nanostation-ac-loco|\
> > +	ubnt,unifiac-pro|\
> >  	yuncore,a770)
> >  		caldata_extract "art" 0x5000 0x844
> >  		;;
> > @@ -101,17 +110,6 @@ case "$FIRMWARE" in
> >  		caldata_extract "art" 0x5000 0x844
> >  		ath10k_patch_mac $(macaddr_add $(mtd_get_mac_binary info
> 0x8) +1)
> >  		;;
> > -	ubnt,unifiac-lite|\
> > -	ubnt,unifiac-lr|\
> > -	ubnt,unifiac-mesh|\
> > -	ubnt,unifiac-mesh-pro|\
> > -	ubnt,lap-120|\
> > -	ubnt,nanobeam-ac|\
> > -	ubnt,nanostation-ac|\
> > -	ubnt,nanostation-ac-loco|\
> > -	ubnt,unifiac-pro)
> > -		caldata_extract "EEPROM" 0x5000 0x844
> > -		;;
> >  	esac
> >  	;;
> >  "ath10k/pre-cal-pci-0000:00:00.0.bin")
> >
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20191127/57b376b0/attachment.sig>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list