[PATCH] fff-support: Update PoE passthrough code for CPE 210

[Adrian Schmutzler]

Hallo,

vielleicht zur Klarstellung:
1. Im Moment wird eine veraltete Version mitgeliefert, die, wenn man sie benutzt, _nicht_ funktioniert. Entsprechend sind die Möglichkeiten meines Erachtens: Updaten oder löschen.
2. Ich bin allerdings auch der Meinung, dass das Mitliefern eines solchen Skriptes keinen Schaden anrichtet. Ich würde sogar in die andere Richtung gehen und im Wiki auf das Vorhandensein hinweisen (im Moment ist das mE nicht der Fall und ich hab das Skript nur zufällig durch eine Textsuche im GitHub gefunden).
3. Im Gegensatz zu früher muss das Skript nun _nicht_ mehr in die rc.local eingebaut werden, sondern es reicht, wenn man es einmalig ausführt. Das ist allerdings nicht upgrade-sicher, siehe https://mantis.freifunk-franken.de/view.php?id=53

Grüße

Adrian

-----Original Message-----
From: Tobias Klaus [mailto:tk+ff at meskal.net] 
Sent: Mittwoch, 5. Juli 2017 13:00
To: franken-dev at freifunk.net; Tim Niemeyer <tim at tn-x.org>
Cc: Adrian Schmutzler <freifunk at adrianschmutzler.de>
Subject: Re: [PATCH] fff-support: Update PoE passthrough code for CPE 210

Hey,

fff-support wird ja nie direkt ausgeführt, man muss es explizit selber in die rc.local_schlagmichtot einbauen und das wird explizit _nicht_ unterstützt und jeder der das tut ist _selber_ verantwortlich für das upgrade.
Ich sehe uns daher nicht in der Pflicht ein upgrade mitzuliefern.

Anderseits wird aber wohl in der aktuellen Version ein kaputtes(nie automatisch ausgeführtes!) Skript mitgeliefert. Daher bin ich schon dafür das zu fixen. Falls es grundsätzliche Bedenken gibt solche "Bequemlichkeits"- Skripte im Repo zu halten, wäre halt die Alternative sie zu löschen. Aber bis dahin finde ich den Patch als solchen gut.

Grüße
Tobias


Am Mittwoch, 5. Juli 2017, 12:26:21 CEST schrieb Tim Niemeyer:
> Hi
> 
> Ich hab da Bauchweh. Die configs/settings von lese/openwrt waren nie stabil.
> Wenn das Setting das Update über lebt muss das nicht heissen, dass es 
> danach noch geht. Dann muss man wieder ein upgrade script schreiben 
> und hat plötzlich tausend Sonderfälle.
> 
> Tim
> 
> Am 5. Juli 2017 11:28:47 MESZ schrieb Adrian Schmutzler
<freifunk at adrianschmutzler.de>:
> >Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> >---
> >.../ar71xx/usr/lib/fff-support/cpe210_activate_poe_passthrough.sh | 8
> >+++++---
> >
> > 1 file changed, 5 insertions(+), 3 deletions(-)
> >
> >mode change 100644 => 100755
> >src/packages/fff/fff-support/ar71xx/usr/lib/fff-support/cpe210_activa
> >te_poe
> >_passthrough.sh
> >
> >diff --git
> >a/src/packages/fff/fff-support/ar71xx/usr/lib/fff-support/cpe210_acti
> >vate_p
> >oe_passthrough.sh
> >b/src/packages/fff/fff-support/ar71xx/usr/lib/fff-support/cpe210_acti
> >vate_
> >poe_passthrough.sh old mode 100644
> >new mode 100755
> >index cb3508f..7351666
> >---
> >a/src/packages/fff/fff-support/ar71xx/usr/lib/fff-support/cpe210_acti
> >vate_p
> >oe_passthrough.sh +++
> >b/src/packages/fff/fff-support/ar71xx/usr/lib/fff-support/cpe210_acti
> >vate_p
> >oe_passthrough.sh @@ -1,5 +1,7 @@
> >
> > if [ "$(cat /var/sysinfo/model)" = "TP-Link CPE210 v1.1" ] ; then
> >
> >-  echo 20 > /sys/class/gpio/export
> >-  echo out > /sys/class/gpio/gpio20/direction
> >-  echo 1 > /sys/class/gpio/gpio20/value
> >+    uci set system.gpio_switch_poe_passthrough=gpio_switch
> >+    uci set system.gpio_switch_poe_passthrough.name='PoE Passthrough'
> >+    uci set system.gpio_switch_poe_passthrough.gpio_pin='20'
> >+    uci set system.gpio_switch_poe_passthrough.value='1'
> >+    uci commit system
> >
> > fi