diff options
author | Adrian Schmutzler <freifunk@adrianschmutzler.de> | 2019-12-22 16:51:19 +0100 |
---|---|---|
committer | Adrian Schmutzler <freifunk@adrianschmutzler.de> | 2019-12-28 02:34:41 +0100 |
commit | 62d5ece70bf6cbbdd1add3a16475a1cea1cb260d (patch) | |
tree | 4341f06d456507bda82077720e90ddb2aaa7a0ec /target/linux/ramips/dts/WR1000.dts | |
parent | 94153971bb013c5a51af64008044064aebd014bc (diff) | |
download | upstream-62d5ece70bf6cbbdd1add3a16475a1cea1cb260d.tar.gz upstream-62d5ece70bf6cbbdd1add3a16475a1cea1cb260d.tar.bz2 upstream-62d5ece70bf6cbbdd1add3a16475a1cea1cb260d.zip |
ramips: remove bogus ralink,mtd-eeprom with offset 0x4
Several devices in mt76x8 subtarget use the following line to set
up wmac in their DTS(I) files:
ralink,mtd-eeprom = <&factory 0x4>
This is strange for several reasons:
- They should use mediatek,mtd-eeprom on this SOC
- The caldata is supposed to start at 0x0
- The parent DTSI mt7628an.dtsi specifies mediatek,mtd-eeprom anyway,
starting from 0x0
- The offset coincides with the default location of the MAC address
in caldata
Based on the comment in b28e94d4bfa1 ("ramips: MiWiFi Nano fixes"),
it looks like the author for this device wanted to actually use
mtd-mac-address instead of ralink,mtd-eeprom. A check on the same
device revealed that actually the MAC address start at offset 4 there,
so the correct caldata offset is 0x0.
Based on these findings, and the fact that the expected location on
this SOC is 0x0, we remove the "ralink,mtd-eeprom = <&factory 0x4>"
statement from all devices in ramips (being only mt7628an anyway).
Thanks to Sungbo Eo for finding and researching this.
Reported-by: Sungbo Eo <mans0n@gorani.run>
Fixes: b28e94d4bfa1 ("ramips: MiWiFi Nano fixes")
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
(cherry picked from commit 09d38a3bc328d122b0d84fcf6bb53d2bce7373b6)
Diffstat (limited to 'target/linux/ramips/dts/WR1000.dts')
-rw-r--r-- | target/linux/ramips/dts/WR1000.dts | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/target/linux/ramips/dts/WR1000.dts b/target/linux/ramips/dts/WR1000.dts index ff0b9badd1..a582172aca 100644 --- a/target/linux/ramips/dts/WR1000.dts +++ b/target/linux/ramips/dts/WR1000.dts @@ -135,7 +135,6 @@ &wmac { status = "okay"; - ralink,mtd-eeprom = <&factory 0x4>; }; ðernet { |