[PATCH v2] Add support for Archer C2600

[Fabian Bläse]

Hallo Adrian,

On 23.11.19 15:01, mail at adrianschmutzler.de wrote:
> Wie schon zuvor angemerkt funktioniert hier das Entfernen von target-subtarget im buildscript nicht, da das subtarget nicht im Namen ist (bei ipq806x).
> 
> Bei 19.07 kommt es dann wieder rein.
da hast du natürlich recht, das hab ich beim rebasen (der Patch ist schon ziemlich alt) doch glatt vergessen.

> 
> Bei 18.06 funktionieren die WiFi LEDs nicht, bei 19.07 ist der Treiber dafür ergänzt.
> 
> Beides könnte man in der Commit Message erwähnen, damit es (theoretisch) für die Nachwelt nachschlagbar ist.
> 
> Noch zwei Kommentar unten.
> 
> 
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
>> Of Fabian Bläse
>> Sent: Samstag, 23. November 2019 14:25
>> To: franken-dev at freifunk.net
>> Subject: [PATCH v2] Add support for Archer C2600
>>
>> Signed-off-by: Fabian Bläse <fabian at blaese.de>
>> ---
>> Changes in v2:
>> - Remove obsolete target Variable from bsp
>> ---
>>  bsp/board_ipq806x.bsp                                |  7 +++++++
>>  bsp/ipq806x/.config                                  | 12 ++++++++++++
>>  .../fff/fff-network/ipq806x/network.tplink,c2600     |  9 +++++++++
>>  3 files changed, 28 insertions(+)
>>  create mode 100644 bsp/board_ipq806x.bsp  create mode 100644
>> bsp/ipq806x/.config  create mode 100644 src/packages/fff/fff-
>> network/ipq806x/network.tplink,c2600
>>
>> diff --git a/bsp/board_ipq806x.bsp b/bsp/board_ipq806x.bsp new file mode
>> 100644 index 0000000..cfb9b7c
>> --- /dev/null
>> +++ b/bsp/board_ipq806x.bsp
>> @@ -0,0 +1,7 @@
>> +machine=ipq806x
>> +chipset=ipq806x
>> +subtarget=generic
> 
> Ich weiß gerade nicht, wie genau das verwendet wird. Das target hat eigentlich kein subtarget, aber den generic Unterordner gibt es trotzdem. Ich glaube, wir verwenden das nur für den Ordner, also kann es wahrscheinlich bleiben.
Jo, das brauchen wir für den Unterordner und den Dateinamen. Im Dateinamen führt das jetzt natürlich zu quatsch.
Ansonsten wir die Variable für nichts verwendet.

> 
>> +
>> +images=(
>> +        "openwrt-${chipset}-tplink_c2600-squashfs-sysupgrade.bin"
>> +       )
>> diff --git a/bsp/ipq806x/.config b/bsp/ipq806x/.config new file mode 100644
>> index 0000000..707e6bd
>> --- /dev/null
>> +++ b/bsp/ipq806x/.config
>> @@ -0,0 +1,12 @@
>> +# Generated using "./buildscript config openwrt".
>> +# Do no edit manually
>> +#
>> +CONFIG_TARGET_ipq806x=y
>> +CONFIG_TARGET_MULTI_PROFILE=y
>> +CONFIG_TARGET_DEVICE_ipq806x_DEVICE_tplink_c2600=y
> 
> Hier sollte man unbedingt noch BUILD_TARGET_PER_ROOTFS (hab den genauen Namen nicht nachgesehen) hinzufügen, sonst gibt es Probleme mit den DEVICE_PACKAGES (die werden u.U. nicht mit eingebaut).
Wir verwenden bei diesem Target aktuell keine eigenen DEVICE_PACKAGES, deshalb hab ich es (analog zu wdr4900 und mt7621) nicht eingebaut.
Bei ar71xx verwenden wir das ja, um selektiv für bestimmte Geräte die Firmware zu ändern.

Ich sehe aber nichts, was gegen BUILD_TARGET_PER_ROOTFS spricht, ich würde das dann einfach bei allen in einem Patch umstellen.

> Außerdem muss man meines Wissen hier die -ct Packages selektieren, da die stock Packages so alt sind, dass sie kein mesh können.
Ist das wirklich so? Ich würde hier nach Möglichkeit immer bei dem bleiben, was bei OpenWRT verwendet wird, außer wir sind uns wirklich sicher, dass etwas anderes besser oder nötig ist.
Ich teste das bei nächster Gelegenheit.

Gruß
Fabian

-------------- 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/20191123/7dbd9e13/attachment.sig>