[PATCH v3 4/8] fff-boardname: Fix changed board name of WDR4900v1

Tim Niemeyer tim at tn-x.org
Mo Aug 6 06:55:08 CEST 2018


Hi

Okay, dann passt das.

Reviewed-by: Tim Niemeyer <tim at tn-x.org>

Tim

Am Sonntag, den 05.08.2018, 23:05 +0200 schrieb
mail at adrianschmutzler.de:
> Hallo Tim,
> 
> OpenWRT hat sich entschieden, grundsätzlich für die compatible-
> strings der syntax
> 
> "vendor,model"
> 
> zu folgen.
> 
> Also z.B.
> 
> tp-link,tl-wdr-4900
> 
> Im neuen ath79-Target (Kernel 4.14), dass das ar71xx Target
> mittelfristig ersetzen soll, ist dies auch schon für die
> unterstützten Geräte umgesetzt (ich habe eine experimentelle Firmware
> für ath79, dort musste ich das auch entsprechend im boardname
> anpassen, daher kenne ich mich aus). Dort heißen dann die Geräte alle
> entsprechend, z.B.
> 
> tp-link,archer-c7-v2
> tp-link,tl-wr841-v12
> ubnt,nanostation-m
> d-link,weißkeinmodell
> 
> Beim WDR4900 im mpcdings Target haben die das mal umgestellt, im
> ar71xx ist der Stand im Moment so, dass man das alte Zeug lässt und
> es für den noch halb fertigen ath79 dann ordentlich neu macht. Sofern
> wir uns also im openwrt-18.06 Branch bewegen, der nur ar71xx enthält,
> macht es Sinn hier den wdr-4900 als Sonderfall wie in meinem Patch
> abzuhandeln. Wenn wir irgendwann auf ath79 umsteigen, kann man das
> entweder generell wegschnippeln oder mit den neuen Strings arbeiten,
> aber das ist Zukunftsmusik.
> 
> Grüße
> 
> Adrian
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > Behalf
> > Of Tim Niemeyer
> > Sent: Sonntag, 5. August 2018 17:23
> > To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-
> > dev at freifunk.net
> > Subject: Re: [PATCH v3 4/8] fff-boardname: Fix changed board name
> > of
> > WDR4900v1
> > 
> > Hi
> > 
> > Am Samstag, den 04.08.2018, 16:34 +0200 schrieb Adrian Schmutzler:
> > > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > > ---
> > >  src/packages/fff/fff-boardname/files/etc/uci-defaults/50-fff-
> > > boardname | 3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/src/packages/fff/fff-boardname/files/etc/uci-
> > > defaults/50-fff-boardname b/src/packages/fff/fff-
> > > boardname/files/etc/uci-defaults/50-fff-boardname
> > > index 7d4cdb6a..cb9edd50 100644
> > > --- a/src/packages/fff/fff-boardname/files/etc/uci-defaults/50-
> > > fff-
> > > boardname
> > > +++ b/src/packages/fff/fff-boardname/files/etc/uci-defaults/50-
> > > fff-
> > > boardname
> > > @@ -58,6 +58,9 @@ case "$BOARD" in
> > >          grep "4300" /var/sysinfo/model && BOARD=tl-wdr4300-v1
> > >          grep "4310" /var/sysinfo/model && BOARD=tl-wdr4310-v1
> > >          ;;
> > > +    tplink,tl-wdr4900-v1)
> > 
> > Da leuten sofort die Alarm-Glocken, aber ich glaube das ist nur
> > schwer zu
> > lesen.
> > 
> > Ich dachte erst das Komma steht für ODER. Aber in
> > /var/sysinfo/board_name
> > steht wirklich dieser ganze String "tplink,tl- wdr4900-v1", oder?
> > 
> > Wenn du mir das kurz bestätigen könntest, würde ich mein Review
> > geben.
> > Sonst müsste man da noch irgendwie mehr Sicherheit rein tun.
> > 
> > Tim
> > 
> > > +        BOARD=tl-wdr4900-v1
> > > +        ;;
> > >      tl-mr3020)
> > >          BOARD=tl-mr3020-v1
> > >          ;;
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180806/6fa24a1f/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev