Message ID | 20200402125343.128007-2-fabian@blaese.de |
---|---|
State | Superseded |
Headers | show |
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)")
Hallo Fabian, Reviewed-by: Adrian Schmutzler <freifunk@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@freifunk.net] On Behalf > Of Fabian Bläse > Sent: Donnerstag, 2. April 2020 14:54 > To: franken-dev@freifunk.net > Subject: [PATCH v2 2/2] Add support for TP-Link C50 v3 > > Signed-off-by: Fabian Bläse <fabian@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
Hallo Fabian, Nachtrag: Wie beim 841 v13 gerade angemerkt, macht es Sinn, in der Commit Message die Flashing-Instruktionen zu ergänzen. So kann jemand gleich beim Support sehen, wie die Images benutzt werden (und wird ggf. gewarnt, wenn etwas "ungewohnt" ist). Ich würde hierfür einfach ganz stumpf den entsprechenden Teil des OpenWrt-Commits kopieren: https://github.com/openwrt/openwrt/commit/14951e8f8e09ba6e2cff140769189a66155a5345 Bei allen neueren Geräten in OpenWrt wird dort eine entsprechende Erklärung verlangt, in meinen Augen ist es also erstrebenswert, das auch in Zukunft so zu machen. Beste Grüße Adrian
Mit ergänzter Commit Message (v3) applied.
Signed-off-by: Fabian Bläse <fabian@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