[v4,1/2] fff-network: Provide script with CPUPORT

This provides the CPUPORT which is needed for the gateway firmware.
The script is supposed to be sourced.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
Reviewed-by: Robert Langhammer <rlanghammer@web.de>
---
 .../fff-network/files/lib/functions/fff/cpuport    | 34 ++++++++++++++++++++++
 1 file changed, 34 insertions(+)
 create mode 100644 src/packages/fff/fff-network/files/lib/functions/fff/cpuport

Hi

Ich denke das kann man so machen. Wobei ich persönlich vermutlich lieber eine Funktion gebaut hätte. Das hält den Code übersichtlicher, denn so wird eine Datei gesourced, aber man weiss gar nicht was sie alles tut und welche globale Variable gesetzt wird. Globale Variablen in zu sourcenden Dateien .. Klingt erstmal nicht gut.

Wegen mir soll es aber erstmal rein, am liebsten aber gleich mit nem Ticket, dass das noch umgebaut werden soll.

Tim

Am 24. April 2019 14:05:44 MESZ schrieb Adrian Schmutzler <freifunk@adrianschmutzler.de>:
>This provides the CPUPORT which is needed for the gateway firmware.
>The script is supposed to be sourced.
>
>Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>Reviewed-by: Robert Langhammer <rlanghammer@web.de>
>---
>.../fff-network/files/lib/functions/fff/cpuport    | 34
>++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
>create mode 100644
>src/packages/fff/fff-network/files/lib/functions/fff/cpuport
>
>diff --git
>a/src/packages/fff/fff-network/files/lib/functions/fff/cpuport
>b/src/packages/fff/fff-network/files/lib/functions/fff/cpuport
>new file mode 100644
>index 00000000..b27b775f
>--- /dev/null
>+++ b/src/packages/fff/fff-network/files/lib/functions/fff/cpuport
>@@ -0,0 +1,34 @@
>+# Copyright 2019 Adrian Schmutzler
>+# License GPLv3
>+
>+BOARD="$(uci get board.model.name)"
>+
>+case "$BOARD" in
>+	tl-wr1043nd-v1)
>+		CPUPORT="5t"
>+		;;
>+	tl-wr1043nd-v2|\
>+	tl-wr1043nd-v3)
>+		CPUPORT="6t"
>+		;;
>+	cpe210-v2|\
>+	cpe210-v3|\
>+	tl-mr3020-v1|\
>+	tl-wa850re-v1|\
>+	tl-wa860re-v1|\
>+	tl-wa901nd-v2|\
>+	ubnt-bullet-m|\
>+	ubnt-loco-m|\
>+	ubnt-loco-m-xw|\
>+	ubnt-nano-m|\
>+	ubnt-pico-m|\
>+	ubnt-power-m-xw|\
>+	ubnt-unifi|\
>+	ubnt-unifiac-lite|\
>+	ubnt-unifiac-mesh)
>+		CPUPORT=""
>+		;;
>+	*)
>+		CPUPORT="0t"
>+		;;
>+esac

Hallo zusammen,

Dann würde ich vorschlagen, dass wir das hier so wie es ist übernehmen.
Ich bin noch nicht ganz mit dem Paket glücklich, da man hier aktuell nicht zwischen node und gw unterscheiden kann.

Da sich der Rest meiner Konfigurationspatches aber auch noch auf Dinge verlässt, die aktuell im fff-network Paket sind, würde ich das jetzt erstmal so lassen und dann später sauber voneinander trennen.

Reviewed-by: Fabian Bläse <fabian@blaese.de>

On 09.05.19 17:52, Tim Niemeyer wrote:
> Wegen mir soll es aber erstmal rein, am liebsten aber gleich mit nem Ticket, dass das noch umgebaut werden soll.
Können wir dann anlegen. Ich würde es jetzt aber nicht in diesem Patch ändern, da das gateway-network Skript ja derzeit CPUPORT als globale Variable verwenden und dann auch angepasst werden müsste.

Gruß
Fabian

Moin

Am 9. Mai 2019 19:29:45 MESZ schrieb "Fabian Bläse" <fabian@blaese.de>:
>Hallo zusammen,
>
>Dann würde ich vorschlagen, dass wir das hier so wie es ist übernehmen.
>Ich bin noch nicht ganz mit dem Paket glücklich, da man hier aktuell
>nicht zwischen node und gw unterscheiden kann.
>
>Da sich der Rest meiner Konfigurationspatches aber auch noch auf Dinge
>verlässt, die aktuell im fff-network Paket sind, würde ich das jetzt
>erstmal so lassen und dann später sauber voneinander trennen.
>
>Reviewed-by: Fabian Bläse <fabian@blaese.de>
>
>On 09.05.19 17:52, Tim Niemeyer wrote:
>> Wegen mir soll es aber erstmal rein, am liebsten aber gleich mit nem
>Ticket, dass das noch umgebaut werden soll.
>Können wir dann anlegen. Ich würde es jetzt aber nicht in diesem Patch
>ändern, da das gateway-network Skript ja derzeit CPUPORT als globale
>Variable verwenden und dann auch angepasst werden müsste.

Ja. Das machen wir dann in einem rutsch, wenn das Zeug erstmal drauf ist.

Tim


>Gruß
>Fabian

Wegen mir könnt ihr das auch gerne in eine Funktion umbauen. Mach ich bei mir ggf. auch.

 

From: Fabian Bläse [mailto:fabian@blaese.de] 
Sent: Donnerstag, 9. Mai 2019 19:30
To: Tim Niemeyer <tim@tn-x.org>; franken-dev@freifunk.net; Adrian Schmutzler <freifunk@adrianschmutzler.de>
Subject: Re: [PATCH v4 1/2] fff-network: Provide script with CPUPORT

 

Hallo zusammen, 

Dann würde ich vorschlagen, dass wir das hier so wie es ist übernehmen. 
Ich bin noch nicht ganz mit dem Paket glücklich, da man hier aktuell nicht zwischen node und gw unterscheiden kann. 

Da sich der Rest meiner Konfigurationspatches aber auch noch auf Dinge verlässt, die aktuell im fff-network Paket sind, würde ich das jetzt erstmal so lassen und dann später sauber voneinander trennen.

Reviewed-by: Fabian Bläse <fabian@blaese.de <mailto:fabian@blaese.de> > 

On 09.05.19 17:52, Tim Niemeyer wrote: 
> Wegen mir soll es aber erstmal rein, am liebsten aber gleich mit nem Ticket, dass das noch umgebaut werden soll. 
Können wir dann anlegen. Ich würde es jetzt aber nicht in diesem Patch ändern, da das gateway-network Skript ja derzeit CPUPORT als globale Variable verwenden und dann auch angepasst werden müsste.

Gruß 
Fabian

Applied.

On 24.04.19 14:05, Adrian Schmutzler wrote:
> This provides the CPUPORT which is needed for the gateway firmware.
> The script is supposed to be sourced.
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> Reviewed-by: Robert Langhammer <rlanghammer@web.de>
> ---
>  .../fff-network/files/lib/functions/fff/cpuport    | 34 ++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
>  create mode 100644 src/packages/fff/fff-network/files/lib/functions/fff/cpuport
> 
> diff --git a/src/packages/fff/fff-network/files/lib/functions/fff/cpuport b/src/packages/fff/fff-network/files/lib/functions/fff/cpuport
> new file mode 100644
> index 00000000..b27b775f
> --- /dev/null
> +++ b/src/packages/fff/fff-network/files/lib/functions/fff/cpuport
> @@ -0,0 +1,34 @@
> +# Copyright 2019 Adrian Schmutzler
> +# License GPLv3
> +
> +BOARD="$(uci get board.model.name)"
> +
> +case "$BOARD" in
> +	tl-wr1043nd-v1)
> +		CPUPORT="5t"
> +		;;
> +	tl-wr1043nd-v2|\
> +	tl-wr1043nd-v3)
> +		CPUPORT="6t"
> +		;;
> +	cpe210-v2|\
> +	cpe210-v3|\
> +	tl-mr3020-v1|\
> +	tl-wa850re-v1|\
> +	tl-wa860re-v1|\
> +	tl-wa901nd-v2|\
> +	ubnt-bullet-m|\
> +	ubnt-loco-m|\
> +	ubnt-loco-m-xw|\
> +	ubnt-nano-m|\
> +	ubnt-pico-m|\
> +	ubnt-power-m-xw|\
> +	ubnt-unifi|\
> +	ubnt-unifiac-lite|\
> +	ubnt-unifiac-mesh)
> +		CPUPORT=""
> +		;;
> +	*)
> +		CPUPORT="0t"
> +		;;
> +esac
>