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

[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
> >          ;;