The sender domain has a DMARC Reject/Quarantine policy which disallows sending mailing list messages using the original "From" header.
To mitigate this problem, the original message has been wrapped automatically by the mailing list software.
--- Begin Message ---Hi, > Here, you assign eth1 to the "wan" LED, but in 02_network you assign eth1 to > "lan". > Either change 02_network or this LED shouldn't be called WAN. I think this is correct because the port is actually labeled as WAN but is used as LAN by default since LTE module is used for wan connection. > phy0tpt trigger can be moved to DTS, just have a look at other devices > recently added to ath79. Won't this disallow user to change LED function? > As stated above, this will make eth1 part of "lan" ... I don't think you can have two interfaces in one network unless you use bridge which you definitely don't want to use in this case. > What's that? You mean the "D12" comment? These are LED labels on board. But I don't have ath79 version of MR6400 so I can't tell whether LED labels on board didn't change. Anyway good job Enrico. Filip On Tue, Sep 17, 2019 at 12:07:20PM +0200, Adrian Schmutzler wrote: > Hi, > > > -----Original Message----- > > From: openwrt-devel [mailto:[email protected]] On > > Behalf Of Enrico Mioso > > Sent: Dienstag, 17. September 2019 02:21 > > To: [email protected] > > Cc: Filip Moc <[email protected]>; Piotr Dymacz <[email protected]>; Enrico Mioso > > <[email protected]> > > Subject: [OpenWrt-Devel] [PATCH] ath79: add support for TP-Link TL-MR6400 > > > > This adds support to the ath79 target for the TP-Link MR6400 router. > > > > As per original commit, hardware specifications (v1.0 EU): > > - SoC: QCA9531 > > - Flash: Winbond W25Q64FV (8MiB) > > - RAM: EtronTech EM6AB160TSE-5G (64MiB) > > - Wireless: SoC platform only (2.4GHz b/g/n, 2x internal antenna) > > - Ethernet: 2NIC (3x100M + 1x100M) > > - WWAN: TP-LINK LTE MODULE (2x external detachable antenna) > > - Power: DC 12V 1A > > > > Flashing instructions: > > You can flash via tftp recovery (serve factory image as > > /mr6400_tp_recovery.bin > > on 192.168.0.66/24, connect to any ethernet port and power on device while > > holding the reset button). Flashing via OEM web interface does not work. > > > > Working: > > - Wi-Fi > > - TP-Link LTE module does it's thing (but see Notes) > > - reset/rfkill keys > > Untested: > > - recovery via factory > > - leds > > > > Issues for which I need help: > > eth1 detects link when infact it's not there, and occasionally causes > > kernel traces due to tx timeouts. > > I promise I'll test untested items as well in final version > > > > Note: as it happened occasionally in ar71xx, during bursty flash activity, > > LTE module init will fail, with USB enumeration errors. > > > > Signed-off-by: Enrico Mioso <[email protected]> > > CC: Filip Moc <[email protected]> > > CC: Piotr Dymacz <[email protected]> > > --- > > .../ath79/base-files/etc/board.d/01_leds | 6 + > > .../ath79/base-files/etc/board.d/02_network | 5 + > > .../ath79/dts/qca9531_tplink_tl-mr6400-v1.dts | 165 ++++++++++++++++++ > > target/linux/ath79/image/generic-tp-link.mk | 10 ++ > > 4 files changed, 186 insertions(+) > > create mode 100644 target/linux/ath79/dts/qca9531_tplink_tl-mr6400-v1.dts > > > > diff --git a/target/linux/ath79/base-files/etc/board.d/01_leds > > b/target/linux/ath79/base-files/etc/board.d/01_leds > > index 778316e450..5728aeb491 100755 > > --- a/target/linux/ath79/base-files/etc/board.d/01_leds > > +++ b/target/linux/ath79/base-files/etc/board.d/01_leds > > @@ -130,6 +130,12 @@ tplink,archer-c7-v5) > > ucidef_set_led_switch "lan3" "LAN3" "tp-link:green:lan3" "switch0" > > "0x10" > > ucidef_set_led_switch "lan4" "LAN4" "tp-link:green:lan4" "switch0" > > "0x20" > > ;; > > +tplink,tl-mr6400-v1) > > + ucidef_set_led_netdev "lan" "LAN" "tp-link:white:lan" "eth0" > > + ucidef_set_led_netdev "wan" "WAN" "tp-link:white:wan" "eth1" > > Here, you assign eth1 to the "wan" LED, but in 02_network you assign eth1 to > "lan". > Either change 02_network or this LED shouldn't be called WAN. > > > + ucidef_set_led_netdev "4g" "4G" "tp-link:white:4g" "usb0" > > + ucidef_set_led_wlan "wlan" "WLAN" "tp-link:white:wlan" "phy0tpt" > > phy0tpt trigger can be moved to DTS, just have a look at other devices > recently added to ath79. > > > + ;; > > tplink,archer-c2-v3|\ > > tplink,tl-wr1043nd-v4|\ > > tplink,tl-wr1043n-v5) > > diff --git a/target/linux/ath79/base-files/etc/board.d/02_network > > b/target/linux/ath79/base-files/etc/board.d/02_network > > index 5b47af8ef7..2b0e1ce400 100755 > > --- a/target/linux/ath79/base-files/etc/board.d/02_network > > +++ b/target/linux/ath79/base-files/etc/board.d/02_network > > @@ -56,6 +56,11 @@ ath79_setup_interfaces() > > winchannel,wb2000) > > ucidef_set_interface_lan "eth0" > > ;; > > + tplink,tl-mr6400-v1) > > + ucidef_set_interfaces_lan_wan "eth0.1 eth1" "usb0" > > As stated above, this will make eth1 part of "lan" ... > > > + ucidef_add_switch "switch0" \ > > + "0@eth0" "1:lan" "2:lan" "3:lan" > > + ;; > > avm,fritz4020|\ > > pcs,cr3000|\ > > tplink,archer-c58-v1|\ > > diff --git a/target/linux/ath79/dts/qca9531_tplink_tl-mr6400-v1.dts > > b/target/linux/ath79/dts/qca9531_tplink_tl-mr6400-v1.dts > > new file mode 100644 > > index 0000000000..4acb1a02a4 > > --- /dev/null > > +++ b/target/linux/ath79/dts/qca9531_tplink_tl-mr6400-v1.dts > > @@ -0,0 +1,165 @@ > > +// SPDX-License-Identifier: GPL-2.0-or-later > > +/dts-v1/; > > + > > +#include <dt-bindings/gpio/gpio.h> > > +#include <dt-bindings/input/input.h> > > + > > +#include "qca953x.dtsi" > > + > > +/ { > > + compatible = "tplink,tl-mr6400-v1", "qca,qca9531"; > > + model = "TP-Link TL-MR6400 v1.0"; > > I would remove the ".0" here, as with TP-Link the sub-revisions typically > refer only to their firmware. So just "v1" should be better. > > > + > > + aliases { > > + led-boot = &led_power; > > + led-failsafe = &led_power; > > + led-running = &led_power; > > + led-upgrade = &led_power; > > + }; > > + > > + gpio_leds: leds { > > + compatible = "gpio-leds"; > > + > > + /* D12 */ > > What's that? > > > + led_wan: wan { > > + label = "tp-link:white:wan"; > > + gpios = <&gpio 0 GPIO_ACTIVE_HIGH>; > > + }; > > IMO those labels only make sense if you use them, as for led_power. I would > remove all of the other ones, i.e. only "wan {" here instead of "led_wan: wan > {" ... > > > + > > + /* D11 */ > > + led_4g: 4g { > > + label = "tp-link:white:4g"; > > + gpios = <&gpio 1 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + /* D5 */ > > + led_wps: wps { > > + label = "tp-link:white:wps"; > > + gpios = <&gpio 3 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + /* D3 */ > > + led_wlan: wlan { > > + label = "tp-link:white:wlan"; > > + gpios = <&gpio 11 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + /* D2 */ > > + led_power: power { > > + label = "tp-link:white:power"; > > + gpios = <&gpio 13 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + /* D4 */ > > + led_lan: lan { > > + label = "tp-link:white:lan"; > > + gpios = <&gpio 16 GPIO_ACTIVE_HIGH>; > > + }; > > + }; > > + > > + gpio-export { > > + compatible = "gpio-export"; > > + #size-cells = <0>; > > + > > + gpio_usb_power { > > + gpio-export,name = "tp-link:power:LTE"; > > + gpio-export,output = <0>; > > Sure about the 0? > > > + gpios = <&gpio 4 GPIO_ACTIVE_LOW>; > > + }; > > + }; > > Please replace gpio-export by a gpio-hog. Either look at recently added > devices or at https://patchwork.ozlabs.org/patch/1141057/ > > Note that in this case I would call the node "lte_power" or "power_lte" > instead of referring to the USB, but that's a matter of taste. > > > + > > + keys { > > + compatible = "gpio-keys"; > > + > > + /* SW2 */ > > + reset { > > + label = "Reset button"; > > + linux,code = <KEY_RESTART>; > > + gpios = <&gpio 12 GPIO_ACTIVE_LOW>; > > + debounce-interval = <60>; > > + }; > > + > > + /* SW3 */ > > + rfkill { > > + label = "RFKill button"; > > + linux,code = <KEY_RFKILL>; > > + gpios = <&gpio 14 GPIO_ACTIVE_LOW>; > > + debounce-interval = <60>; > > + }; > > + }; > > +}; > > + > > +&uart { > > + status = "okay"; > > +}; > > + > > +&spi { > > + status = "okay"; > > + num-cs = <1>; > > + > > + flash@0 { > > + #address-cells = <1>; > > + #size-cells = <1>; > > + compatible = "jedec,spi-nor"; > > + reg = <0>; > > + spi-max-frequency = <10000000>; > > + > > + partitions { > > + compatible = "fixed-partitions"; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + > > + uboot: partition@0 { > > There is a tab between uboot and partition. Replace by space ... > > > + label = "u-boot"; > > + reg = <0x000000 0x020000>; > > + read-only; > > + }; > > + > > + partition@20000 { > > + compatible = "tplink,firmware"; > > + label = "firmware"; > > + reg = <0x020000 0x7d0000>; > > + }; > > + > > + art: partition@7f0000 { > > + label = "art"; > > + reg = <0x7f0000 0x010000>; > > + read-only; > > + }; > > + }; > > + }; > > +}; > > + > > +ð1 { > > + mtd-mac-address = <&uboot 0x1fc00>; > > + mtd-mac-address-increment = <(-1)>; > > +}; > > + > > +ð0 { > > + status = "okay"; > > + phy-handle = <&swphy4>; > > + > > + mtd-mac-address = <&uboot 0x1fc00>; > > + mtd-mac-address-increment = <1>; > > +}; > > + > > +&wmac { > > + status = "okay"; > > + mtd-cal-data = <&art 0x1000>; > > + mtd-mac-address = <&uboot 0x1fc00>; > > +}; > > + > > +&usb0 { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + status = "okay"; > > + > > + hub_port: port@1 { > > + reg = <1>; > > + #trigger-source-cells = <0>; > > + }; > > +}; > > + > > +&usb_phy { > > + status = "okay"; > > +}; > > diff --git a/target/linux/ath79/image/generic-tp-link.mk > > b/target/linux/ath79/image/generic-tp-link.mk > > index 5519e9c960..78d7810f29 100644 > > --- a/target/linux/ath79/image/generic-tp-link.mk > > +++ b/target/linux/ath79/image/generic-tp-link.mk > > @@ -393,6 +393,16 @@ define Device/tplink_tl-wr810n-v1 > > endef > > TARGET_DEVICES += tplink_tl-wr810n-v1 > > > > +define Device/tplink_tl-mr6400-v1 > > + $(Device/tplink-8mlzma) > > + ATH_SOC := qca9531 > > + DEVICE_MODEL := TL-MR6400 > > + DEVICE_VARIANT := v1 > > + TPLINK_HWID := 0x64000001 > > + DEVICE_PACKAGES := kmod-usb-core kmod-usb2 kmod-usb-net > > kmod-usb-net-rndis kmod-usb-serial kmod-usb-serial-option adb > > You can remove kmod-usb-core, kmod-usb-net and kmod-usb-serial, as those are > dependencies of the other packages. > > Since this is a port from ar71xx, you should add a SUPPORTED_DEVICES entry. > > Best > > Adrian > > > +endef > > +TARGET_DEVICES += tplink_tl-mr6400-v1 > > + > > define Device/tplink_tl-wr810n-v2 > > $(Device/tplink-8mlzma) > > ATH_SOC := qca9533 > > -- > > 2.23.0 > > > > > > _______________________________________________ > > openwrt-devel mailing list > > [email protected] > > https://lists.openwrt.org/mailman/listinfo/openwrt-devel
--- End Message ---
_______________________________________________ openwrt-devel mailing list [email protected] https://lists.openwrt.org/mailman/listinfo/openwrt-devel
