Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support of UP1-8xTH #2

Open
wants to merge 6 commits into
base: devel
Choose a base branch
from
Open

Support of UP1-8xTH #2

wants to merge 6 commits into from

Conversation

henfri
Copy link

@henfri henfri commented Sep 18, 2024

Now PR against devel.
UNTESTED.

@ab-tools
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants