[PATCH 1/4] Add support for Archer C60 v1

[Adrian Schmutzler]

Hallo Fabian,

 

dann mache ich einen eigenen Commit für das Umordnen, aber VOR dem Patch für den C60.

 

Das mit dem –ct ist so ein permanenter Konflikt bei den OpenWrt Leuten. –ct ist etwas besser maintained (eine Person statt gar keine ;-)), ansonsten geht je nach Gerät entweder der eine Treiber oder der andere oder beide.

Ich erinnere mich auch dunkel, dass es im –ct irgendeine Funktion gibt/gab, die wir für Freifunk spezifisch gebraucht haben (IBSS?). Im master hat OpenWrt vor nicht allzu langer Zeit auf die –ct Variante als default gewechselt. In der Praxis probiert man bei ath10k einfach, mit welchen Konfigurationen alles funktioniert und hofft dann, dass es so bleibt. Das war bei mir für die genannte Kombination so. Ich würde das auch gerne so lassen. Dies stellt aber natürlich einen nennenswerten Unterschied zu deiner „getesteten“ Variante dar.

 

CPUPORT ist Copy-Paste aus meiner FW, das sollte tatsächlich weg.

 

Grüße

 

Adrian

 

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Montag, 28. Januar 2019 14:12
To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-dev at freifunk.net
Subject: Re: [PATCH 1/4] Add support for Archer C60 v1

 

Hallo Adrian, 

bei mir läuft genau dieser Patch auf einem C60v1 mit meiner Gatewayfirmware einwandfrei. 

Das Umordnen der anderen Zeilen in diesem Patch finde ich aber ehrlichgesagt nicht wirklich hübsch. Ich würde so etwas in einen eigenen Patch/Commit packen, der Übersichtlichkeit halber.

On 28.01.19 12:55, Adrian Schmutzler wrote: 
> --- a/bsp/ar71xx/.config 
> +++ b/bsp/ar71xx/.config 
> @@ -4,16 +4,18 @@ 
>  CONFIG_TARGET_ar71xx=y 
>  CONFIG_TARGET_ar71xx_tiny=y 
>  CONFIG_TARGET_MULTI_PROFILE=y 
> -CONFIG_TARGET_DEVICE_ar71xx_tiny_DEVICE_gl-ar150=y 
> -CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_tiny_DEVICE_gl-ar150="" 
>  CONFIG_TARGET_DEVICE_ar71xx_tiny_DEVICE_archer-c25-v1=y 
>  CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_tiny_DEVICE_archer-c25-v1="-kmod-ath10k kmod-ath10k-ct" 
> +CONFIG_TARGET_DEVICE_ar71xx_tiny_DEVICE_archer-c60-v1=y 
> +CONFIG_TARGET_DEVICE_PACKAGES_ar71xx_tiny_DEVICE_archer-c60-v1="-kmod-ath10k kmod-ath10k-ct -ath10k-firmware-qca9888 ath10k-firmware-qca9888-ct"

Bei mir habe ich weder Firmware noch Kerneltreiber verändert. Wofür genau war das nochmal gut? 

> --- /dev/null 
> +++ b/src/packages/fff/fff-network/ar71xx/network.archer-c60-v1 
> @@ -0,0 +1,10 @@ 
> +PORTORDER="4 3 2 1" 
> + 
> +WANDEV=eth0 
> +SWITCHDEV=eth1 
> +CLIENT_PORTS="1 2 0t" 
> +BATMAN_PORTS="3 4 0t" 
> +CPUPORT="0t" 
Eine CPUPORT Variable gibt es Upstream eigentlich noch gar nicht. 
Ich bin mir auch noch nicht so ganz sicher, ob das so die optimalste Möglichkeit ist. 

Da ich an der ganzen Gatewaykonfiguration noch relativ viel basteln werde, bis dafür ein Patch kommt, würde ich das hier erstmal weglassen.

Mein C60v1 ist im Betrieb, ich würde den nur ungern aus der Decke rauskramen, um den Patch zu testen. 
Entsprechend kann ich auch ETHMESHMAC und ROUTERMAC nicht so richtig testen. Sieht aber richtig aus, vorausgesetzt die wXmesh Interfaces bekommen die erste jeweils vom Treiber zugewiesene MAC.

Zu den restlichen Geräten kann ich mangels Hardware nichts sagen. 

Gruß 
Fabian 

 
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190128/401d2e8b/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190128/401d2e8b/attachment.sig>