diff options
| author | Mario Andrés Pérez | 2026-02-05 00:19:08 +0000 |
|---|---|---|
| committer | Hauke Mehrtens | 2026-02-10 23:09:30 +0000 |
| commit | 3ebb157b2d3bd2dc7cfacff6651993e8b0c2cf7f (patch) | |
| tree | 4c8ef8a250efed22f1007c330ffee74b878b04ed | |
| parent | 82fec218e1a179c043f858aa58276100d281f436 (diff) | |
| download | openwrt-3ebb157b2d3bd2dc7cfacff6651993e8b0c2cf7f.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>
(cherry picked from commit 7aa1f7e814dea0627e905ef1ad92954cae51d301)
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 5d3a3319fc..125648b076 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 @@ -5,6 +5,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 fd7c159aab..684ab1aa8e 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 @@ -129,6 +129,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 dde54a053d..16c387a879 100644 --- a/target/linux/mediatek/filogic/base-files/lib/upgrade/platform.sh +++ b/target/linux/mediatek/filogic/base-files/lib/upgrade/platform.sh @@ -127,6 +127,7 @@ platform_do_upgrade() { airpi,ap3000m|\ arcadyan,mozart|\ glinet,gl-mt2500|\ + glinet,gl-mt2500-airoha|\ glinet,gl-mt6000|\ glinet,gl-x3000|\ glinet,gl-xe3000|\ @@ -350,6 +351,7 @@ platform_copy_config() { airpi,ap3000m|\ arcadyan,mozart|\ glinet,gl-mt2500|\ + glinet,gl-mt2500-airoha|\ glinet,gl-mt6000|\ glinet,gl-x3000|\ glinet,gl-xe3000|\ |