| Message ID | 20180804143427.18488-7-freifunk@adrianschmutzler.de |
|---|---|
| State | Rejected |
| Headers | show |
diff --git a/src/packages/fff/fff-sysupgrade/Makefile b/src/packages/fff/fff-sysupgrade/Makefile index 0be55abd..1ed3d46c 100644 --- a/src/packages/fff/fff-sysupgrade/Makefile +++ b/src/packages/fff/fff-sysupgrade/Makefile @@ -1,8 +1,8 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fff-sysupgrade -PKG_VERSION:=7 -PKG_RELEASE:=4 +PKG_VERSION:=8 +PKG_RELEASE:=1 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) diff --git a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh index 8ef4949b..72a10053 100755 --- a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh +++ b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh @@ -17,9 +17,23 @@ BOARD=$(uci get board.model.name) #decide SOC case $BOARD in tl-wdr4900-v1 ) - SOC="mpc85xx" ;; + SOC="mpc85xx-g" ;; + tl-mr3020-v1 |\ + tl-wa850re-v1 |\ + tl-wa860re-v1 |\ + tl-wa901nd-v2 |\ + tl-wr740n-v4 |\ + tl-wr741nd-v2 |\ + tl-wr741nd-v4 |\ + tl-wr841-v7 |\ + tl-wr841-v8 |\ + tl-wr841-v9 |\ + tl-wr841-v10 |\ + tl-wr841-v11 |\ + tl-wr841-v12 ) + SOC="ar71xx-tiny" ;; * ) - SOC="ar71xx" ;; + SOC="ar71xx-g" ;; esac echo "" echo "Hardware: $BOARD" @@ -27,9 +41,9 @@ echo "Hardware: $BOARD" #rewrite BOARD case $BOARD in cpe210 ) - BOARD="cpe210-220" ;; + BOARD="cpe210-220-v1" ;; cpe510 ) - BOARD="cpe510-520" ;; + BOARD="cpe510-520-v1" ;; esac wget "${UPGRADE_PATH}/release.nfo" @@ -67,11 +81,7 @@ if [ "$VERSION" -eq "$FIRMWARE_VERSION" ]; then done fi -if [ "$FIRMWARE_COMMUNITY" = "franken" ]; then - FIRMWARE_COMMUNITY="fff" -fi - -FILE="${FIRMWARE_COMMUNITY}-${VERSION}-${SOC}-g-${BOARD}-squashfs-sysupgrade.bin" +FILE="fff-${VERSION}-${SOC}-${BOARD}-squashfs-sysupgrade.bin" echo "Downloading $FILE" echo "" wget "${UPGRADE_PATH}/${FILE}"
Hi In diesem Patch sehe ich noch mehr generve, warum es nicht gut ist für die 4/8 MB Kiste ein neues BSP zu bauen. Denn wenn sich der Name ändert, dann müssen wir das am Server berücksichtigen, weil die alte Firmware ja nach dem alten Namen sucht. Dieser würde nicht gefunden werden. Das kann man schon machen, aber ich würde das lieber vermeiden wollen, wenn es nicht wirklich notwendig ist und die Notwendigkeit kann ich nicht sehen. Am Samstag, den 04.08.2018, 16:34 +0200 schrieb Adrian Schmutzler: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-sysupgrade/Makefile | 4 ++-- > .../fff/fff-sysupgrade/files/etc/sysupgrade.sh | 28 > +++++++++++++++------- > 2 files changed, 21 insertions(+), 11 deletions(-) > > diff --git a/src/packages/fff/fff-sysupgrade/Makefile > b/src/packages/fff/fff-sysupgrade/Makefile > index 0be55abd..1ed3d46c 100644 > --- a/src/packages/fff/fff-sysupgrade/Makefile > +++ b/src/packages/fff/fff-sysupgrade/Makefile > @@ -1,8 +1,8 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-sysupgrade > -PKG_VERSION:=7 > -PKG_RELEASE:=4 > +PKG_VERSION:=8 > +PKG_RELEASE:=1 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > index 8ef4949b..72a10053 100755 > --- a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > +++ b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > @@ -17,9 +17,23 @@ BOARD=$(uci get board.model.name) > #decide SOC > case $BOARD in > tl-wdr4900-v1 ) > - SOC="mpc85xx" ;; > + SOC="mpc85xx-g" ;; > + tl-mr3020-v1 |\ > + tl-wa850re-v1 |\ > + tl-wa860re-v1 |\ > + tl-wa901nd-v2 |\ > + tl-wr740n-v4 |\ > + tl-wr741nd-v2 |\ > + tl-wr741nd-v4 |\ > + tl-wr841-v7 |\ > + tl-wr841-v8 |\ > + tl-wr841-v9 |\ > + tl-wr841-v10 |\ > + tl-wr841-v11 |\ > + tl-wr841-v12 ) > + SOC="ar71xx-tiny" ;; Da fällt auf.. -tiny sollte -t sein, sonst könnten die Dateinamen wieder zu lang werden. Tim > * ) > - SOC="ar71xx" ;; > + SOC="ar71xx-g" ;; > esac > echo "" > echo "Hardware: $BOARD" > @@ -27,9 +41,9 @@ echo "Hardware: $BOARD" > #rewrite BOARD > case $BOARD in > cpe210 ) > - BOARD="cpe210-220" ;; > + BOARD="cpe210-220-v1" ;; > cpe510 ) > - BOARD="cpe510-520" ;; > + BOARD="cpe510-520-v1" ;; > esac > > wget "${UPGRADE_PATH}/release.nfo" > @@ -67,11 +81,7 @@ if [ "$VERSION" -eq "$FIRMWARE_VERSION" ]; then > done > fi > > -if [ "$FIRMWARE_COMMUNITY" = "franken" ]; then > - FIRMWARE_COMMUNITY="fff" > -fi > - > -FILE="${FIRMWARE_COMMUNITY}-${VERSION}-${SOC}-g-${BOARD}-squashfs- > sysupgrade.bin" > +FILE="fff-${VERSION}-${SOC}-${BOARD}-squashfs-sysupgrade.bin" > echo "Downloading $FILE" > echo "" > wget "${UPGRADE_PATH}/${FILE}"
Hallo Tim, das hier hängt von der Diskussion für den bsp Patch ab. Theoretisch ist natürlich denkbar, dass man einfach das generic oder tiny ganz raus nimmt und das den Server machen lässt. Ich habe den Patch so gebaut, dass sich möglichst wenig ändert. Das muss natürlich nicht so sein. Für meine eigene Firmware habe ich das inzwischen so gemacht, dass das target und subtarget mit in die /etc/config/board geschrieben wird (man kann ja je nach subtarget ein anderes File einbinden). Dann muss ich kein switch-case nach devices machen. Ist halt ne größere Änderung, von der ich nicht ausgehe, dass die mehrheitsfähig ist. Das Prinzip kann man hier in dem Patch sehen: https://github.com/adrianschmutzler/fff-firmware/commit/25ab742322c7cb3191a8cbd8861a7eb96663484e und hier dann die Änderungen für sysupgrade https://github.com/adrianschmutzler/fff-firmware/commit/33f4e4f7eb200343fac4199aa44a55a5c13d539b Ist bei mir besonders praktisch, weil ich ar71xx und ath79 da parallel drin habe ... Falls das jemandem gefällt, kann ich das auch schicken ... Was man hier im Endeffekt macht, ist aber wahrscheinlich relativ egal. Grüße Adrian > -----Original Message----- > From: Tim Niemeyer [mailto:tim@tn-x.org] > Sent: Sonntag, 5. August 2018 17:31 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@freifunk.net > Subject: Re: [PATCH v3 6/8] fff-sysupgrade: Update sysupgrade.sh to > support openwrt-18.06 > > Hi > > In diesem Patch sehe ich noch mehr generve, warum es nicht gut ist für die > 4/8 MB Kiste ein neues BSP zu bauen. > > Denn wenn sich der Name ändert, dann müssen wir das am Server > berücksichtigen, weil die alte Firmware ja nach dem alten Namen sucht. > Dieser würde nicht gefunden werden. > > Das kann man schon machen, aber ich würde das lieber vermeiden wollen, > wenn es nicht wirklich notwendig ist und die Notwendigkeit kann ich nicht > sehen. > > > Am Samstag, den 04.08.2018, 16:34 +0200 schrieb Adrian Schmutzler: > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > src/packages/fff/fff-sysupgrade/Makefile | 4 ++-- > > .../fff/fff-sysupgrade/files/etc/sysupgrade.sh | 28 > > +++++++++++++++------- > > 2 files changed, 21 insertions(+), 11 deletions(-) > > > > diff --git a/src/packages/fff/fff-sysupgrade/Makefile > > b/src/packages/fff/fff-sysupgrade/Makefile > > index 0be55abd..1ed3d46c 100644 > > --- a/src/packages/fff/fff-sysupgrade/Makefile > > +++ b/src/packages/fff/fff-sysupgrade/Makefile > > @@ -1,8 +1,8 @@ > > include $(TOPDIR)/rules.mk > > > > PKG_NAME:=fff-sysupgrade > > -PKG_VERSION:=7 > > -PKG_RELEASE:=4 > > +PKG_VERSION:=8 > > +PKG_RELEASE:=1 > > > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > > > diff --git a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > > b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > > index 8ef4949b..72a10053 100755 > > --- a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > > +++ b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh > > @@ -17,9 +17,23 @@ BOARD=$(uci get board.model.name) > > #decide SOC > > case $BOARD in > > tl-wdr4900-v1 ) > > - SOC="mpc85xx" ;; > > + SOC="mpc85xx-g" ;; > > + tl-mr3020-v1 |\ > > + tl-wa850re-v1 |\ > > + tl-wa860re-v1 |\ > > + tl-wa901nd-v2 |\ > > + tl-wr740n-v4 |\ > > + tl-wr741nd-v2 |\ > > + tl-wr741nd-v4 |\ > > + tl-wr841-v7 |\ > > + tl-wr841-v8 |\ > > + tl-wr841-v9 |\ > > + tl-wr841-v10 |\ > > + tl-wr841-v11 |\ > > + tl-wr841-v12 ) > > + SOC="ar71xx-tiny" ;; > Da fällt auf.. -tiny sollte -t sein, sonst könnten die Dateinamen wieder zu lang > werden. > > Tim > > > * ) > > - SOC="ar71xx" ;; > > + SOC="ar71xx-g" ;; > > esac > > echo "" > > echo "Hardware: $BOARD" > > @@ -27,9 +41,9 @@ echo "Hardware: $BOARD" > > #rewrite BOARD > > case $BOARD in > > cpe210 ) > > - BOARD="cpe210-220" ;; > > + BOARD="cpe210-220-v1" ;; > > cpe510 ) > > - BOARD="cpe510-520" ;; > > + BOARD="cpe510-520-v1" ;; > > esac > > > > wget "${UPGRADE_PATH}/release.nfo" > > @@ -67,11 +81,7 @@ if [ "$VERSION" -eq "$FIRMWARE_VERSION" ]; then > > done > > fi > > > > -if [ "$FIRMWARE_COMMUNITY" = "franken" ]; then > > - FIRMWARE_COMMUNITY="fff" > > -fi > > - > > -FILE="${FIRMWARE_COMMUNITY}-${VERSION}-${SOC}-g-${BOARD}- > squashfs- > > sysupgrade.bin" > > +FILE="fff-${VERSION}-${SOC}-${BOARD}-squashfs-sysupgrade.bin" > > echo "Downloading $FILE" > > echo "" > > wget "${UPGRADE_PATH}/${FILE}"
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-sysupgrade/Makefile | 4 ++-- .../fff/fff-sysupgrade/files/etc/sysupgrade.sh | 28 +++++++++++++++------- 2 files changed, 21 insertions(+), 11 deletions(-)