[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