diff options
| author | Mario Andrés Pérez | 2026-02-05 00:19:08 +0000 |
|---|---|---|
| committer | Hauke Mehrtens | 2026-02-10 23:03:05 +0000 |
| commit | 7aa1f7e814dea0627e905ef1ad92954cae51d301 (patch) | |
| tree | 3473a75d32a3aa6c064cb5fe484fdd0a7b24b099 | |
| parent | 64ec08eee1cc35417a6b0189e40b92665056da25 (diff) | |
| download | openwrt-7aa1f7e814dea0627e905ef1ad92954cae51d301.tar.gz | |
mediatek: filogic: gl-mt2500 fix compatibles PHY variants
These devices share the same "compatible" in device tree causing some
incompatibilities (sysupgrades, ASU profile identification), assign a
unique "compatible" and "model" to each variant.
Context:
Commit [1] added each variant's dts compatible to the SUPPORTED_DEVICES
field of the other variant to make easy sysupgrades between these
physically indistinguishable devices variants possible.
But there were found three issues which does not allow this:
- the sysupgrade's stricter check still used in some sysupgrade
paths(this check is being replaced(and redundant) with the newer fwtool's
SUPPORTED_DEVICES check using the info in images METADATA), this check
will fail when sysupgrading from a different board_name(compatible dts)
that the image was created for (image profile name).[2]
- ASU needs unique "dts compatible" to identify the devices profile.
- and an ASU's profile identification limitation when several devices from
a common target share SUPPORTED_DEVICES entries.[3]
There is a proposal for these issues but not yet implemented [4][3].
Until these issues are fixed we won't allow "easy" sysupgrades between
these two device variants.
Commit [5] avoided the ASU profile identification limitation but
missed the required two unique dts compatibles in order to make the two
variants fully work, although not allowing easy sysupgrade between them.
[1]: https://github.com/openwrt/openwrt/commit/8d30e07180367cdeb4affd79adead6e1025355c9
[2]: sysupgrade stricter check https://github.com/openwrt/openwrt/issues/20566#issuecomment-3583555482
[3]: ASU proposal https://github.com/openwrt/asu/pull/1533
[4]: allow easy sysupgrade proposal https://github.com/openwrt/openwrt/pull/20947
[5]: https://github.com/openwrt/openwrt/commit/b71f4665cda10c284c9460409ae30fb9b0beecf8
Fixes: b71f466 ("mediatek: filogic: fix supported_devices list for gl-mt2500")
Fixes: 8d30e07 ("mediatek: filogic: fix for new GL.iNet GL-MT2500/GL-MT2500A hardware revision")
Fixes: https://github.com/openwrt/openwrt/issues/20566
Fixes: https://github.com/openwrt/asu/issues/1525
Signed-off-by: Mario Andrés Pérez <mapb_@outlook.com>
Link: https://github.com/openwrt/openwrt/pull/21842
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
7 files changed, 15 insertions, 3 deletions
diff --git a/package/boot/uboot-tools/uboot-envtools/files/mediatek_filogic b/package/boot/uboot-tools/uboot-envtools/files/mediatek_filogic index 5972233bae..bf779c8162 100644 --- a/package/boot/uboot-tools/uboot-envtools/files/mediatek_filogic +++ b/package/boot/uboot-tools/uboot-envtools/files/mediatek_filogic @@ -58,6 +58,7 @@ acer,predator-w6|\ acer,predator-w6d|\ acer,vero-w6m|\ glinet,gl-mt2500|\ +glinet,gl-mt2500-airoha|\ glinet,gl-mt6000|\ glinet,gl-x3000|\ glinet,gl-xe3000|\ diff --git a/target/linux/mediatek/base-files/lib/preinit/05_set_preinit_iface b/target/linux/mediatek/base-files/lib/preinit/05_set_preinit_iface index 828912ca22..b43d223da6 100644 --- a/target/linux/mediatek/base-files/lib/preinit/05_set_preinit_iface +++ b/target/linux/mediatek/base-files/lib/preinit/05_set_preinit_iface @@ -6,6 +6,7 @@ set_preinit_iface() { cudy,tr3000-v1|\ cudy,tr3000-v1-ubootmod|\ glinet,gl-mt2500|\ + glinet,gl-mt2500-airoha|\ glinet,gl-mt3000|\ openembed,som7981|\ wavlink,wl-wn573hx3) diff --git a/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v1.dts b/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v1.dts index fc7524c5c0..36e83a32d0 100644 --- a/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v1.dts +++ b/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v1.dts @@ -3,6 +3,11 @@ /dts-v1/; #include "mt7981b-glinet-gl-mt2500.dtsi" +/ { + model = "GL.iNet GL-MT2500 (MaxLinear PHY)"; + compatible = "glinet,gl-mt2500", "mediatek,mt7981"; +}; + &gmac0 { phy-handle = <&phy5>; }; diff --git a/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v2.dts b/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v2.dts index 6f6c11427c..1fb69f13be 100644 --- a/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v2.dts +++ b/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500-v2.dts @@ -3,6 +3,11 @@ /dts-v1/; #include "mt7981b-glinet-gl-mt2500.dtsi" +/ { + model = "GL.iNet GL-MT2500 (Airoha PHY)"; + compatible = "glinet,gl-mt2500-airoha", "mediatek,mt7981"; +}; + &gmac0 { phy-handle = <&phy13>; }; diff --git a/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500.dtsi b/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500.dtsi index 73d254ff10..51e9309cdc 100644 --- a/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500.dtsi +++ b/target/linux/mediatek/dts/mt7981b-glinet-gl-mt2500.dtsi @@ -5,9 +5,6 @@ #include <dt-bindings/pinctrl/mt65xx.h> / { - model = "GL.iNet GL-MT2500"; - compatible = "glinet,gl-mt2500", "mediatek,mt7981"; - aliases { label-mac-device = &gmac0; led-boot = &led_sys_white; diff --git a/target/linux/mediatek/filogic/base-files/etc/board.d/02_network b/target/linux/mediatek/filogic/base-files/etc/board.d/02_network index a07b5c136f..fa206fafce 100644 --- a/target/linux/mediatek/filogic/base-files/etc/board.d/02_network +++ b/target/linux/mediatek/filogic/base-files/etc/board.d/02_network @@ -130,6 +130,7 @@ mediatek_setup_interfaces() cudy,tr3000-v1|\ cudy,tr3000-v1-ubootmod|\ glinet,gl-mt2500|\ + glinet,gl-mt2500-airoha|\ glinet,gl-mt3000|\ glinet,gl-x3000|\ glinet,gl-xe3000|\ diff --git a/target/linux/mediatek/filogic/base-files/lib/upgrade/platform.sh b/target/linux/mediatek/filogic/base-files/lib/upgrade/platform.sh index 01858dbf7c..8e7d2bdfe8 100644 --- a/target/linux/mediatek/filogic/base-files/lib/upgrade/platform.sh +++ b/target/linux/mediatek/filogic/base-files/lib/upgrade/platform.sh @@ -128,6 +128,7 @@ platform_do_upgrade() { airpi,ap3000m|\ arcadyan,mozart|\ glinet,gl-mt2500|\ + glinet,gl-mt2500-airoha|\ glinet,gl-mt6000|\ glinet,gl-x3000|\ glinet,gl-xe3000|\ @@ -352,6 +353,7 @@ platform_copy_config() { airpi,ap3000m|\ arcadyan,mozart|\ glinet,gl-mt2500|\ + glinet,gl-mt2500-airoha|\ glinet,gl-mt6000|\ glinet,gl-x3000|\ glinet,gl-xe3000|\ |