| Message ID | 20180804143427.18488-5-freifunk@adrianschmutzler.de |
|---|---|
| State | Rejected |
| Headers | show |
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) + BOARD=tl-wdr4900-v1 + ;; tl-mr3020) BOARD=tl-mr3020-v1 ;;
Hi Am Samstag, den 04.08.2018, 16:34 +0200 schrieb Adrian Schmutzler: > Signed-off-by: Adrian Schmutzler <freifunk@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 > ;;
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@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Sonntag, 5. August 2018 17:23 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@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@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 > > ;;
Hi Okay, dann passt das. Reviewed-by: Tim Niemeyer <tim@tn-x.org> Tim Am Sonntag, den 05.08.2018, 23:05 +0200 schrieb mail@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@freifunk.net] On > > Behalf > > Of Tim Niemeyer > > Sent: Sonntag, 5. August 2018 17:23 > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > > dev@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@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 > > > ;; > >
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-boardname/files/etc/uci-defaults/50-fff-boardname | 3 +++ 1 file changed, 3 insertions(+)