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

[Adrian Schmutzler]

Hallo,

1. also im Moment überlebt es ein Update ja nicht. Mir ist allerdings nicht so wirklich klar, warum.

2. Da in der neuen Firmware das direkte Bearbeiten der /sys/class/gpio/* Dateien nicht möglich ist (zumindest mir), ist die einzige mir bekannte Möglichkeit für den PoE Passthrough das Setzen der system.* Einträge. Vor dem Patch habe ich das halt von Hand in die Konsole eingegeben, das Skript fast das nur zusammen. Das heißt für mich, das Setzen des system.* Einträge lässt sich schlussendlich nicht vermeiden, wenn ich PoE-Passthrough will. Mit dem Skript habe ich jetzt halt nur eine Zeile statt fünf und ich muss nicht jedes Mal im Wiki nachschauen.

Quelle ist übrigens hier (nachdem die alte Methode nicht mehr funktioniert hat, hab ich das gegoogelt):
https://wiki.freifunk.net/TP-Link_CPE210#PoE_Passthrough

Wenn jemand weiß, wie es anders geht, raus damit. Den Teil mit den verschiedenen Configs habe ich glaube ich nicht vollständig verstanden, aber würde das wirklich für den o.g. Fall helfen?

Grüße

Adrian



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

Hi

Am 5. Juli 2017 13:36:57 MESZ schrieb Adrian Schmutzler <mail at adrianschmutzler.de>:
>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.

Das könnte man fixen.

>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

Ich bleibe bei meinem Veto, weil du mit den Script Dinge in eine Datei schreibst die das update überleben wird.
Dabei ist aber nicht gesichert, dass der Mechanismus nach dem Update noch geht. Das müsste man dann jeweils beim hochziehen von lede prüfen. Dann ggfs ein upgrade script schreiben und dabei die verschiedenen alten Versionen berücksichtigen. Das ist alles viel zu aufwändig.

Besser geht es so:
Du machst eine eigene config. Die wird beim (ersten)booten von einem script gelesen und je nach dem ob an oder aus das passende setting in der system config (oder wo auch immer) gesetzt. Die system config darf das update nicht ueberleben. Wir haben dann die Kontrolle der config und webig Pflegearbeiten.

Tim


>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