Hi On 2020-03-09, Adrian Schmutzler wrote: > > -----Original Message----- > > From: Stefan Lippers-Hollmann [mailto:[email protected]] [...] > > On 2020-02-28, John Crispin wrote: > > > On 28.02.20 21:30, Stefan Lippers-Hollmann wrote: > > > > On 2020-02-28, [email protected] wrote: > > > >>> On 2020-02-28, Koen Vandeputte wrote: > > [...] > > > > ath79 with kernel 4.14 has already been (mostly) broken by "ath79: add > > > > new ar934x spi driver" (ebf0d8dadeca443121f4f597c51bf6591e341caf), which > > > > does break the (shared between all kernel versions) DTS definitions for > > > > NAND based ath79 devices (breaking compilation with kernel 4.14). > > > > > > > > Because of "FS#2524 - ath79-tiny: TP-Link TL-WR941ND v2.4: Marvel > > > > MV88E6060 regression between kernel v4.14 and v4.19" > > > > https://bugs.openwrt.org/index.php?do=details&task_id=2524 I still do > > > > regularly build ath79 with kernel 4.14, but in order to finish the build > > > > I do need to comment out the affected devices: > > [...] > > > > > > > > Admittedly, the affected TL-WR941ND v2 (4/32) is barely worth the effort > > > > anymore. [...] > > Neither, nor. It's merely a report that kernel 4.14 support (albeit > > technically present in source) for ath79 isn't functional anymore (and > > hasn't been for a few weeks) anyways, regardless of the introduction of > > kernel 5.4 and its changes to ag71xx. Kind of answering Adrian's previous > > question "By moving ag71xx to files-4.19 on ath79, I suspect 4.14 is broken > > now on this target." [...]
> thanks for you detailed explanation. > > Is there a specific reason why you build based on master and do not use the > 19.07 stable branch with working 4.14 support? [...] As mentioned previously, the device in question isn't really in active use anymore, I mostly do semi-regular regression testing with it. Therefore my pain threshold for it is limited, - enough to patch it back into the generic subtarget and build it (thanks to TARGET_MULTI_PROFILE && TARGET_PER_DEVICE_ROOTFS) as part of the build for actually useful devices, - enough to rebase the patchset to bring it back into generic, - enough to fix the build if the image got oversized again, - enough to spend some efforts on debugging new issues, but I probably wouldn't start a dedicated build just for this particular device (as part of dedicated debugging, sure, but not regularly). If it can't keep up with master builds, it's fate will be entertaining the dust bunnies, before finally meeting its maker. If this regression with ag71xx and Marvell MV88E6060 can't be fixed, it would be better to stop generating images for this particular device, as it's hard to recover. Therefore I'd suggest to add either of these patches to master: https://github.com/pkgadd/openwrt/commits/tl-wr941ndv2-deactivate -- From 3aacb4b374ee76bf2b0d2e43fc450a6bab50ffd9 Mon Sep 17 00:00:00 2001 From: Stefan Lippers-Hollmann <[email protected]> Date: Mon, 9 Mar 2020 23:56:34 +0100 Subject: [PATCH] ath79-tiny: disable image generation for the tl-wr941nd v2/ v3 Since kernel v4.19 support for the Marvell MV88E6060 switch used in this devices has been broken, leading to an inaccesible device without an enduser compatible recovery method (no push-button tftp recovery, requiring serial console access and adding quite tiny 0-ohm SMD resistors to fix the cut serial rx traces). Disable image generation for this device until this gets fixed. Signed-off-by: Stefan Lippers-Hollmann <[email protected]> --- target/linux/ath79/image/tiny-tp-link.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/ath79/image/tiny-tp-link.mk b/target/linux/ath79/image/tiny-tp-link.mk index 29fdfec1e58..f0eaefbee41 100644 --- a/target/linux/ath79/image/tiny-tp-link.mk +++ b/target/linux/ath79/image/tiny-tp-link.mk @@ -338,7 +338,7 @@ define Device/tplink_tl-wr941-v2 TPLINK_HWID := 0x09410002 TPLINK_HWREV := 2 endef -TARGET_DEVICES += tplink_tl-wr941-v2 +#TARGET_DEVICES += tplink_tl-wr941-v2 define Device/tplink_tl-wr941-v4 $(Device/tplink-4m) -- or https://github.com/pkgadd/openwrt/commits/tl-wr941ndv2-nuke -- From 8dbdd50dc732d429e4650d375b56fc72587c5f42 Mon Sep 17 00:00:00 2001 From: Stefan Lippers-Hollmann <[email protected]> Date: Tue, 10 Mar 2020 00:05:16 +0100 Subject: [PATCH] ath79-tiny: drop support for the tl-wr941nd v2/ v3 Since kernel v4.19 support for the Marvell MV88E6060 switch used in this devices has been broken, leading to an inaccesible device without an enduser compatible recovery method (no push-button tftp recovery, requiring serial console access and adding quite tiny 0-ohm SMD resistors to fix the cut serial rx traces). Drop support for this device. Signed-off-by: Stefan Lippers-Hollmann <[email protected]> --- .../ath79/dts/ar9132_tplink_tl-wr941-v2.dts | 168 ------------------ target/linux/ath79/image/tiny-tp-link.mk | 13 -- target/linux/ath79/tiny/config-default | 5 - 3 files changed, 186 deletions(-) delete mode 100644 target/linux/ath79/dts/ar9132_tplink_tl-wr941-v2.dts diff --git a/target/linux/ath79/dts/ar9132_tplink_tl-wr941-v2.dts b/target/linux/ath79/dts/ar9132_tplink_tl-wr941-v2.dts deleted file mode 100644 index 1be4d6183ac..00000000000 --- a/target/linux/ath79/dts/ar9132_tplink_tl-wr941-v2.dts +++ /dev/null @@ -1,168 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 -/dts-v1/; - -#include <dt-bindings/gpio/gpio.h> -#include <dt-bindings/input/input.h> - -#include "ar9132.dtsi" - -/ { - compatible = "tplink,tl-wr941-v2", "qca,ar9132"; - model = "TP-Link TL-WR941N/ND v2/v3"; - - aliases { - led-boot = &led_system; - led-failsafe = &led_system; - led-running = &led_system; - led-upgrade = &led_system; - label-mac-device = ð0; - }; - - keys { - compatible = "gpio-keys"; - - reset { - label = "reset"; - linux,code = <KEY_RESTART>; - gpios = <&gpio 3 GPIO_ACTIVE_LOW>; - debounce-interval = <60>; - }; - - qss { - label = "qss"; - linux,code = <KEY_WPS_BUTTON>; - gpios = <&gpio 7 GPIO_ACTIVE_LOW>; - debounce-interval = <60>; - }; - }; - - leds { - compatible = "gpio-leds"; - - led_system: system { - label = "tp-link:green:system"; - gpios = <&gpio 2 GPIO_ACTIVE_LOW>; - }; - - qss_r { - label = "tp-link:red:qss"; - gpios = <&gpio 4 GPIO_ACTIVE_HIGH>; - }; - - qss_g { - label = "tp-link:green:qss"; - gpios = <&gpio 5 GPIO_ACTIVE_HIGH>; - }; - - wlan { - label = "tp-link:green:wlan"; - gpios = <&gpio 9 GPIO_ACTIVE_LOW>; - linux,default-trigger = "phy0tpt"; - }; - }; - - dsa { - compatible = "marvell,dsa"; - #address-cells = <1>; - #size-cells = <0>; - - dsa,mii-bus = <&mdio0>; - dsa,ethernet = <ð0>; - - switch@0 { - #address-cells = <1>; - #size-cells = <0>; - reg = <0>; - - port@0 { - reg = <0>; - label = "wan"; - }; - - port@1 { - reg = <1>; - label = "lan1"; - }; - - port@2 { - reg = <2>; - label = "lan2"; - }; - - port@3 { - reg = <3>; - label = "lan3"; - }; - - port@4 { - reg = <4>; - label = "lan4"; - }; - - port@5 { - reg = <5>; - label = "cpu"; - }; - }; - }; -}; - -&uart { - status = "okay"; -}; - -&spi { - status = "okay"; - - flash@0 { - compatible = "jedec,spi-nor"; - reg = <0>; - spi-max-frequency = <25000000>; - - partitions { - compatible = "fixed-partitions"; - #address-cells = <1>; - #size-cells = <1>; - - uboot: partition@0 { - label = "u-boot"; - reg = <0x000000 0x20000>; - read-only; - }; - - partition@20000 { - compatible = "tplink,firmware"; - label = "firmware"; - reg = <0x020000 0x3d0000>; - }; - - art: partition@3f0000 { - label = "art"; - reg = <0x3f0000 0x10000>; - read-only; - }; - }; - }; -}; - -&mdio0 { - status = "okay"; -}; - -ð0 { - status = "okay"; - - phy-mode = "rmii"; - mtd-mac-address = <&uboot 0x1fc00>; - - fixed-link { - speed = <100>; - full-duplex; - }; -}; - -&wmac { - status = "okay"; - mtd-cal-data = <&art 0x1000>; - mtd-mac-address = <&uboot 0x1fc00>; -}; diff --git a/target/linux/ath79/image/tiny-tp-link.mk b/target/linux/ath79/image/tiny-tp-link.mk index 29fdfec1e58..3428e5c11db 100644 --- a/target/linux/ath79/image/tiny-tp-link.mk +++ b/target/linux/ath79/image/tiny-tp-link.mk @@ -327,19 +327,6 @@ define Device/tplink_tl-wr940n-v6 endef TARGET_DEVICES += tplink_tl-wr940n-v6 -define Device/tplink_tl-wr941-v2 - $(Device/tplink-4m) - SOC := ar9132 - DEVICE_MODEL := TL-WR941ND - DEVICE_VARIANT := v2/v3 - DEVICE_ALT0_VENDOR := TP-Link - DEVICE_ALT0_MODEL := TL-WR941N - DEVICE_ALT0_VARIANT := v2/v3 - TPLINK_HWID := 0x09410002 - TPLINK_HWREV := 2 -endef -TARGET_DEVICES += tplink_tl-wr941-v2 - define Device/tplink_tl-wr941-v4 $(Device/tplink-4m) SOC := ar7240 diff --git a/target/linux/ath79/tiny/config-default b/target/linux/ath79/tiny/config-default index 42243cfc481..9b8ed872a2b 100644 --- a/target/linux/ath79/tiny/config-default +++ b/target/linux/ath79/tiny/config-default @@ -1,11 +1,6 @@ CONFIG_LEDS_RESET=y CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=y CONFIG_NET_DEVLINK=y -CONFIG_NET_DSA=y -CONFIG_NET_DSA_MV88E6060=y -# CONFIG_NET_DSA_TAG_QCA is not set -CONFIG_NET_DSA_TAG_TRAILER=y -CONFIG_NET_SWITCHDEV=y CONFIG_PHYLINK=y CONFIG_PHY_AR7100_USB=y CONFIG_PHY_AR7200_USB=y -- Regards Stefan Lippers-Hollmann
pgpxDpUpB1bgi.pgp
Description: Digitale Signatur von OpenPGP
_______________________________________________ openwrt-devel mailing list [email protected] https://lists.openwrt.org/mailman/listinfo/openwrt-devel
