Support of UP1-8xTH

[henfri]

Now PR against devel.
UNTESTED.

[ab-tools]

Hallo henfri,

sorry, dass es etwas gedauert hat, aber ich musste zunächst noch Power-Control für den Fingerprint für meine Hardware implementieren, was doch nochmal einen gewissen Teil des Codes geändert hat.

Währenddessen habe ich jedoch bereits einige Änderungen von dir direkt übernommen, insbesondere alle noch fehlenden Definitionen in die hardware.h ergänzt.

Zudem habe ich soeben noch ein paar Änderungen beim Fingerprint, aber auch beim Schaltaktor gepusht, die sicherstellen, dass alles ohne den Fingerprint-PWR-PIN sowie auch mit 0 Schaltaktor-Kanälen kompiliert.

Daraufhin habe ich nun einen neuen Branch devel-up1 angelegt und dort die neue Hardware-und Build-Definition hinzugefügt.

Ich würde dich nun bitten das alles gründlich (mit besagtem, neuen Branch) zu testen, damit sicher gestellt ist, dass die von dir gewählte Hardware-Konfiguration (= Pin-Out) mit allen definierten PINs auch korrekt funktionieren. Ansonsten ist mir noch aufgefallen, dass du weder eine Prog- noch Info-LED und auch keinen Prog-Button für die UP1-Hardware definiert hast. Hat der UP1 denn das wirklich alles nicht?

Wie gesagt, bitte einmal alles gründlich testen.
Wenn du dann bestätigst, dass damit alles funktioniert, würde ich mit dem nächsten Fingerprint Firmware-Release auch ein Release für den UP1 veröffentlichen.

Viele Grüße
Andreas

[henfri]

Hallo Andreas,

ich muss für die Tests erst neue Hardware bekommen, denn der Fingerprint ist schon verbaut. Das wird etwas dauern - vermutlich bis Mitte Dezember.

Gruß,
Hendrik

[ab-tools]

Hallo Hendrik,

kein Problem, dann meld' dich einfach, wenn du soweit bist.

Viele Grüße
Andreas

[henfri]

Hallo,

ich habe mir den Code im Develop-Branch einmal angesehen. Sieht soweit gut aus
Nur das hier habe ich vermisst:

platformio.custom.ini

[env:debug_RP2040]
extends = RP2040_custom_develop
upload_protocol = mbed
build_flags = 
  ${RP2040_custom_releases.build_flags}
  -D OKNXHW_UP1_CONTROLLER2040
;upload_port = D:\

Gruß,
Hendrik

[henfri]

Hallo Andreas,

hast du meinen Hinweis oben gesehen?
Zusätzlich wäre der Pinout noch gut in /hardware/ siehe
https://github.com/henfri/OAM-FingerprintUP1/tree/main/hardware

Viele Grüße,
Hendrik

[henfri]

Hallo,

noch ein Hinweis:
der branch kompiliert nicht. CRC16 fehlt, obwohl
[custom]
lib_deps= ...

in der platformio.ini die Library installieren soll.

Es gibt keine Fehlermeldung. Ich vermute aber es funktioniert bei dir, da du die Library manuell vorher schon installiert hattest?

Gruß,
Hendrik

[ab-tools]

Hallo Hendrik,

das kann gerade tatsächlich sein, weil ich die neuesten Änderungen in den Branch noch nicht gemergt habe - kann ich aber gleich machen.

Ich bin ja ohnehin aktuell recht viel am Fingerprint dran, da er ja gerade die NFC-Erweiterung bekommt. :-)

Viele Grüße
Andreas

[ab-tools]

Hallo Hendrik,

erledigt: Neuster Stand ist gemerged und zwei zusätzliche Defines hinzugefügt - jetzt kompiliert der devel-up1-Branch für mich. Ansonsten aber ungetestet.

Viele Grüße
Andreas

[henfri]

Hi Andreas,

vielen Dank!
Kann es sein, dass da noch eine Abhängigkeit fehlt?

In file included from src\main.cpp:2:
lib\OFM-SwitchActuator\src/SwitchActuatorModule.h:3:10: fatal error: GPIOModule.h: No such file or directory

Ich habe vorher restore-dependencies ausgeführt, dann kompiliert
Testweise auch einmal restore-dependencies-branch (da bin ich mir immer nicht sicher...

Gruß,
Hendrik

[ab-tools]

Hallo Hendrik,

das kann gut sein, die Dependencies-Datei baut er glaube ich automatisch nur beim Release neu.

Vermutlich fehlt das OFM-GPIOModule. Bis zum Release wird das aber vermutlich ohnehin nicht mehr benötigt, da Dominik das ja gerade nach Common migriert.

Viele Grüße
Andreas

[henfri]

Hallo Andreas,

vielen Dank!
Das habe ich eingefügt und komme weiter
2b8c0a8 v1 lib/OFM-GPIOModule https://github.com/OpenKNX/OFM-GPIOModule

Allerdings, kommt der Code nicht damit klar, wenn OPENKNX_SWA_SET_PINS nicht gesetzt ist.
In FingerprintModule.cpp wird ja ein
#ifdef OPENKNX_SWA_SET_PINS verwendet
Ich denke, besser wäre da doch ein Prüfen auf
#define OPENKNX_SWA_CHANNEL_COUNT 0
oder?

Wie auch immer:
An anderer Stelle fehlt diese Prüfung:

Compiling .pio\build\release_UP1\libdb1\knx\knx\rf_medium_object.cpp.o
In file included from src\main.cpp:2:
lib\OFM-SwitchActuator\src/SwitchActuatorModule.h:16:60: error: 'OPENKNX_SWA_SET_PINS' was not declared in this scope
   16 | const uint8_t RELAY_SET_PINS[OPENKNX_SWA_CHANNEL_COUNT] = {OPENKNX_SWA_SET_PINS};
      |                                                            ^~~~~~~~~~~~~~~~~~~~
lib\OFM-SwitchActuator\src/SwitchActuatorModule.h:17:62: error: 'OPENKNX_SWA_RESET_PINS' was not declared in this scope; did you mean 'OPENKNX_BI_GPIO_PINS'?
   17 | const uint8_t RELAY_RESET_PINS[OPENKNX_SWA_CHANNEL_COUNT] = {OPENKNX_SWA_RESET_PINS};
      |                                                              ^~~~~~~~~~~~~~~~~~~~~~
      |                                                              OPENKNX_BI_GPIO_PINS

Gruß,
Hendrik

[ab-tools]

Hallo Hendrik,

dann musst du eine andere Version der Module bei dir haben als ich, denn bei mir hat er vorhin kompiliert.

Am besten nochmal alle Module auf den neuesten Stand aus GIT updaten.

Viele Grüße
Andreas

[henfri]

Hallo Andreas,

sorry, wenn ich mich blöd anstelle...
Welche der Dateien macht das?

-a----        09.03.2025     14:54           1237 Restore-Checkout-Branch.ps1
-a----        09.03.2025     14:54           1072 Restore-Checkout-Hash.ps1
-a----        09.03.2025     14:54           1512 Restore-Dependencies-Branch.ps1
-a----        09.03.2025     14:54          36755 Restore-Dependencies.ps1
-a----        09.03.2025     14:54           2237 Restore-Project.ps1

Restore-Dependencies.ps1
und
Restore-Dependencies-Branch.ps1
haben jedenfalls nicht geholfen.

Ein git pull im OFM-SwitchActuator sagt "already up to date".
Laut dependencies.txt müsste ich da auch im develop sein:
cfaf665 devel lib/OFM-SwitchActuator
Kann es sein, dass du Änderungen noch nicht comitted hast?

Gruß,
Hendrik