[PATCH v2 2/2] Add support for TP-Link C50 v3

[mail at adrianschmutzler.de]

Hallo Fabian,

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

Ich würde noch ein "ramips:" oder "ramips/mt76x8:" vor den Commit title schreiben.

Die Port-Zuordnung zu Batman/Client kann ich logischerweise nicht überprüfen.

Ich frage mich gerade, ob man hier vielleicht noch eine sinnvolle Commit Message ergänzen kann.
Ich habe dann meistens auch nur generische Sachen hingeschrieben, und ggf. die MAC Adressen noch mal erwähnt, wenn ich sie getestet habe:
https://github.com/adrianschmutzler/fff-firmware/commit/c29417c0cee52549f021e5a0592ffa9b3281f27a
https://github.com/adrianschmutzler/fff-firmware/commit/3192247b0b85bf7aa54730cb4d9acab1fdc002b9
https://github.com/adrianschmutzler/fff-firmware/commit/7a214dd41aed06d66b8ddb2de78fa35b11e2d1f2

Man sollte aber in jedem Fall in der Commit Message erwähnen, dass man als Folge des Geräte-Supports das Target ramips/mt76x8 neu erstellt.

Review gilt erstmal nur für die 2/2. Im Prinzip sind die Patches aber ja eigentlich unabhängig, ohne 1/2 gibt’s halt erstmal nur sysupgrade.

Kannst du gerne zeitnah einfüllen.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Donnerstag, 2. April 2020 14:54
> To: franken-dev at freifunk.net
> Subject: [PATCH v2 2/2] Add support for TP-Link C50 v3
> 
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
> Changes in v2:
> - Add ETHMESHMAC
> ---
>  bsp/board_mt76x8.bsp                          |  5 ++++
>  bsp/mt76x8/.config                            | 27 +++++++++++++++++++
>  .../fff-network/ramips/network.tplink,c50-v3  | 10 +++++++
>  3 files changed, 42 insertions(+)
>  create mode 100644 bsp/board_mt76x8.bsp  create mode 100644
> bsp/mt76x8/.config  create mode 100644 src/packages/fff/fff-
> network/ramips/network.tplink,c50-v3
> 
> diff --git a/bsp/board_mt76x8.bsp b/bsp/board_mt76x8.bsp new file mode
> 100644 index 0000000..812456c
> --- /dev/null
> +++ b/bsp/board_mt76x8.bsp
> @@ -0,0 +1,5 @@
> +machine=mt76x8
> +chipset=ramips
> +subtarget=mt76x8
> +images=("openwrt-${chipset}-${subtarget}-tplink_c50-v3-squashfs-
> sysupgrade.bin"
> +        )
> diff --git a/bsp/mt76x8/.config b/bsp/mt76x8/.config new file mode 100644
> index 0000000..9d76ed5
> --- /dev/null
> +++ b/bsp/mt76x8/.config
> @@ -0,0 +1,27 @@
> +# Generated using "./buildscript config openwrt".
> +# Do no edit manually
> +#
> +CONFIG_TARGET_ramips=y
> +CONFIG_TARGET_ramips_mt76x8=y
> +CONFIG_TARGET_MULTI_PROFILE=y
> +CONFIG_TARGET_DEVICE_ramips_mt76x8_DEVICE_tplink_c50-v3=y
> +CONFIG_TARGET_DEVICE_PACKAGES_ramips_mt76x8_DEVICE_tplink_c50-
> v3=""
> +CONFIG_BUSYBOX_CUSTOM=y
> +CONFIG_TARGET_PER_DEVICE_ROOTFS=y
> +# CONFIG_BUSYBOX_CONFIG_BRCTL is not set #
> CONFIG_BUSYBOX_CONFIG_CROND
> +is not set # CONFIG_BUSYBOX_CONFIG_CRONTAB is not set #
> +CONFIG_BUSYBOX_CONFIG_FEATURE_FAST_TOP is not set #
> +CONFIG_BUSYBOX_CONFIG_FEATURE_NTPD_SERVER is not set
> +CONFIG_CLEAN_IPKG=y # CONFIG_DROPBEAR_CURVE25519 is not set #
> +CONFIG_FASTD_ENABLE_CIPHER_SALSA2012 is not set #
> +CONFIG_FASTD_ENABLE_MAC_GHASH is not set #
> +CONFIG_FASTD_ENABLE_MAC_UHASH is not set #
> +CONFIG_FASTD_ENABLE_METHOD_COMPOSED_GMAC is not set #
> +CONFIG_FASTD_ENABLE_METHOD_COMPOSED_UMAC is not set #
> +CONFIG_FASTD_ENABLE_METHOD_GENERIC_GMAC is not set #
> +CONFIG_FASTD_ENABLE_METHOD_GENERIC_UMAC is not set #
> +CONFIG_PACKAGE_ALFRED_VIS is not set CONFIG_PACKAGE_opkg=m
> +CONFIG_STRIP_KERNEL_EXPORTS=y
> diff --git a/src/packages/fff/fff-network/ramips/network.tplink,c50-v3
> b/src/packages/fff/fff-network/ramips/network.tplink,c50-v3
> new file mode 100644
> index 0000000..a779a71
> --- /dev/null
> +++ b/src/packages/fff/fff-network/ramips/network.tplink,c50-v3
> @@ -0,0 +1,10 @@
> +. /lib/functions/fff/network
> +
> +WANDEV=eth0
> +SWITCHDEV=eth0
> +CLIENT_PORTS="6t 3 4"
> +WAN_PORTS="6t 0"
> +BATMAN_PORTS="6t 1 2"
> +
> +ROUTERMAC=$(cat /sys/class/net/eth0/address)
> +ETHMESHMAC=$(macFlipLocalBit "$(cat /sys/class/net/eth0/address)")
> --
> 2.26.0
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200402/6ab48389/attachment.sig>