[PATCH 1/2] bsp: harmonize settings across targets

Fabian Bläse fabian at blaese.de
Mi Apr 1 14:13:41 CEST 2020


Tatsächlich scheint das alles _nicht_ der default zu sein.
Dann bin ich erstmal dafür, das so zu belassen (aka. diesen Patch so zu applien), bis wir uns dazu entschieden haben, ob und wie wir Geräte mit 4MB Flash gesondert behandeln möchten.

Hier noch der Hinweis am Rand, dass kmod-ath10k-ct und die ganzen ct Firmwares per Default auch "m" sind und daher theoretisch noch weg könnten.

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

Gruß
Fabian

On 01.04.20 13:12, mail at adrianschmutzler.de wrote:
>> sieht gut aus. Aber können wir die "is not set" Zeilen nicht weglassen? 
> 
> Ich habe das jetzt nicht konkret geprüft, bitte aber folgendes zu beachten (falls dir das nicht bewusst ist):
> 
> Die .config Dateien orientieren sich an der Kernel config. Es gibt daher für echte config parameter zwei Optionen:
> 
> enabled:
> CONFIG=y
> 
> disabled:
> # CONFIG_xy is not set
> 
> Letzteres bedeutet also _nicht_ ein no-change-use-default, sondern meint effectiv "CONFIG=n", was es in dieser Syntax nicht gibt.
> Damit schaltet man also effektiv Dinge aktiv aus.
> 
> Bei Packages gibt es dann zusätzlich noch das "m".
> 
> Ich weiß jetzt nicht, ob wir diese CONFIGs tatsächlich ausschalten müssen (da müsste man mal bei fastd etc. nachschauen, ob die standardmäßig an sind). Ich habe sie nur von ar71xx kopiert.
> Sind diese aber tatsächlich per default an, schalten wir sie mit diesen Zeilen aus. Würden wir die Zeilen weglassen, würde wir hingegen den Default verwenden.
> 
> Du kannst das auch selbst mit make menuconfig ausprobieren, du musst nur darauf achten, dass du echte Optionen erwischst. Bei Packages, die einmal selektiert wurden, kommt man in der Regel nur auf "m" zurück.
> 
> Grüße
> 
> Adrian
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200401/a230b18b/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev