[PATCH 3/3] Add support for WR841N v12

Tobias Klaus tk+ff at meskal.net
Di Mai 30 00:02:18 CEST 2017


Hey Adrian,

Danke für deine Arbeit an der Unterstützung dieses doch recht wichtigen 
Modells!

Ein paar kleine(inline) Anmerkungen hab ich noch. Bei entsprechender 
Rückmeldung deinerseits könnte ich die auch beim Einspielen noch übernehmen.

Am Montag, 22. Mai 2017, 16:33:11 CEST schrieb Adrian Schmutzler:
> Dieser Patch fügt den Support für den WR841N v12 hinzu. Hierfür wird
> der LEDE Commit 3dbc4175a82c0cfe6c8f435b49bd9370ab442a4f (lede-17.01)
> benötigt.
Ich denke dieser Kommentar sollte nicht in der git-Geschichte landen. 
Einerseits hatten wir uns mal geeinigt Commit-Nachrichten auf Englisch zu 
schreiben und andererseits handelt es sich hier ja eher um einen Hinweis an 
die Maintainer um die Einspielreihenfolge klar zu machen. Dafür würde sich 
vermutlich der cover-letter (git format-patch --cover-letter) anbieten. Falls 
für dich OK, würde ich ihn deswegen beim Einspielen anpassen.

> 
> Für TFTP-Recovery ist zu beachten, dass der Image-Name
> wr841nv11_tp_recovery.bin lautet (mit 11, nicht 12).
Hierfür gilt zunächst dasselbe. Zusätzlich sollten wir diese Info aber 
vermutlich im Wiki festhalten?

> ---
>  bsp/ar71xx/.config                                        |  2 ++
>  bsp/board_ar71xx.bsp                                      |  1 +
>  src/packages/fff/fff-network/ar71xx/network.tl-wr841n-v12 | 10 ++++++++++
>  3 files changed, 13 insertions(+)
>  create mode 100644
> src/packages/fff/fff-network/ar71xx/network.tl-wr841n-v12
> 
> diff --git a/bsp/ar71xx/.config b/bsp/ar71xx/.config
> index cb550ee..bb20393 100644
> --- a/bsp/ar71xx/.config
> +++ b/bsp/ar71xx/.config
> @@ -44,6 +44,8 @@ CONFIG_TARGET_DEVICE_ar71xx_generic_DEVICE_tl-wr841-v10=y
>  CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_generic_DEVICE_tl-wr841-v10=""
>  CONFIG_TARGET_DEVICE_ar71xx_generic_DEVICE_tl-wr841-v11=y
>  CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_generic_DEVICE_tl-wr841-v11=""
> +CONFIG_TARGET_DEVICE_ar71xx_generic_DEVICE_tl-wr841-v12=y
> +CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_generic_DEVICE_tl-wr841-v12=""
>  CONFIG_TARGET_DEVICE_ar71xx_generic_DEVICE_tl-wr841-v7=y
>  CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_generic_DEVICE_tl-wr841-v7=""
>  CONFIG_TARGET_DEVICE_ar71xx_generic_DEVICE_tl-wr841-v8=y
> diff --git a/bsp/board_ar71xx.bsp b/bsp/board_ar71xx.bsp
> index 6ef9b80..fdf221c 100644
> --- a/bsp/board_ar71xx.bsp
> +++ b/bsp/board_ar71xx.bsp
> @@ -13,6 +13,7 @@
> images=("lede-ar71xx-generic-cpe210-220-squashfs-sysupgrade.bin"
> "lede-ar71xx-generic-tl-wr841-v9-squashfs-sysupgrade.bin"
> "lede-ar71xx-generic-tl-wr841-v10-squashfs-sysupgrade.bin"
> "lede-ar71xx-generic-tl-wr841-v11-squashfs-sysupgrade.bin" +       
> "lede-ar71xx-generic-tl-wr841-v12-squashfs-sysupgrade.bin"
> "lede-ar71xx-generic-tl-wr842n-v2-squashfs-sysupgrade.bin"
> "lede-ar71xx-generic-tl-wr1043nd-v1-squashfs-sysupgrade.bin"
> "lede-ar71xx-generic-tl-wr1043nd-v2-squashfs-sysupgrade.bin" diff --git
> a/src/packages/fff/fff-network/ar71xx/network.tl-wr841n-v12
> b/src/packages/fff/fff-network/ar71xx/network.tl-wr841n-v12 new file mode
> 100644
> index 0000000..81b406c
> --- /dev/null
> +++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841n-v12
> @@ -0,0 +1,10 @@
> +
> +WANDEV=eth1
> +# WANDEV=eth0
Diese auskommentierte Zeile sollte nicht ins Repository gelangen. Da auch dies 
rein kosmetisch ist, würde ich (oder entsprechende Commiter) sie einfach 
entfernen, falls OK?



> +SWITCHDEV=eth0
> +CLIENT_PORTS="1 2 0t"
> +WAN_PORTS=
> +BATMAN_PORTS="3 4 0t"
> +
> +ETHMESHMAC=eth1
> +ROUTERMAC=eth0

Viele Grüße
Tobias


Mehr Informationen über die Mailingliste franken-dev