[PATCH 2/2] fff-hoods/fff-wireless: Reconfigure instead of delete and create

[mail at adrianschmutzler.de]

Hallo Fabian,

> Das ganze applied auf dem aktuellen master nicht wirklich schön, das ist aber ein lösbares Problem. :-) 

Ich habe einen v3 geschickt, der direkt auf den master rebased ist. Ist in der Form natürlich nicht getestet etc.

> > +             uci set wireless.w${freq}ap.disabled='1' 
> > +             uci set wireless.w${freq}mesh.disabled='1' 
> > +             uci set wireless.w${freq}config.disabled='1' 
> > +             uci set wireless.w${freq}sta.disabled='0' 
> >       done 
> Das ganze Wechseln in einen anderen Modus analog zum Kommentar von mir beim Patch 1/2 in eine Funktion bringen? 

Ich muss ehrlich sagen, dass mir das an der Stelle direkt (ohne nochmalige Funktion) besser gefällt, es ist ja schon eine Funktion und die Zahl der Codezeilen ist überschaubar.
Auch hier geht es mehr um Geschmacksfragen. Wenn du eine Funktion willst, wäre es mir lieber das so zu übernehmen und du bastelst dann nachträglich eine Funktion hin.

> > +     uci commit wireless 
> Ich würde diese temporäre Änderung gar nicht in den Flash schreiben, sondern im RAM belassen. 

Tatsächlich ist es ja keine temporäre Änderung. Es ist eine permanente Änderung, bis wir wieder ein neues Netz finden.
Bei einem reboot würde dann mit der alten (falschen) Config gebootet.

So eine Lösung im RAM kann man aber gerne mal konkret diskutieren. 
Sollte aber von diesem Patch getrennt sein.

> > -                     uci -q commit system 
> > +                     uci commit system 
> Bitte in separaten Patch, falls das tatsächlich gewollt ist, dass das hier steht. 

Ja.

> > --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless 
> > +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless 
> Die Änderungen sehen in diesem Diff ziemlich kompliziert aus, in Wirklichkeit lässt es sich aber sehr leicht auf: 
> - Alles weg 
> - Funktionen wifiListRadio() und wifiGetFreq() hinzufügen 

In der Tat. Kann ich nix dagegen machen.

> - Die Erkennung per hwmode erscheint mir sinnvoll. Ich würde allerdings noch einen Fehlerfall mit passendem Rückgabewert einführen. (Stichwort 60GHz.. :-))

OpenWrt selbst prüft scheinbar nur 2.4 GHz und 5 GHz:

https://github.com/openwrt/openwrt/blob/master/package/kernel/mac80211/files/lib/wifi/mac80211.sh#L86

Entsprechend gibt es im Moment nur diese beiden Zustände, daher würde ich da nicht vorgreifen.

Neues Patchset habe ich geschickt (v3).

Grüße

Adrian
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190616/7902f903/attachment.sig>