freebus.org

Hallo,
wenn ich versuche die Firmware zu übersetzen, bekomme ich eine menge Fehlermeldungen.
(AVR Studio6, RF-Board, Mega 168p oder 328p)
Die ersten beiden "APPLICATION_RUN_STATUS' undeclared here (not in a function)" und "MANUFACTORER_ADR' undeclared here (not in a function)" bekomme ich ja noch geradegebogen.
Aber spätestens bei "struct grp_addr_s' has no member named 'ga' " ist schluß.
Muss ich da noch irgendwelche libs mit einbinden?
Was mache ich falsch?

Gruß
Uwe

Hallo Uwe,

du machst überhaupt nichts falsch.
Die Applikation für den 10V Analogeingang ist noch nicht auf die neue lib umgestellt. Dies ist auch nicht mit ein paar Zeilen zu erledigen.

Mit der neue Lib habe ich einen ganz anderen Ansatz gewählt. Hier werden nicht mehr Gruppenadressen in der Applikation behandelt sondern generisch in der Lib.
D.h. die Applikation braucht sich nicht um die Gruppenadressen / Flags der Kommunikationsobjekte zu kümmern. Stattdessen bedient die Applikation direkt die Kommunikationsobjekte und kennt eigentlich überhaupt keine Gruppenadressen.

Wenn du den 10V- Analogeingang nutzen möchtest musst du entweder die alte Version verwenden oder du hilfst bei der Anpassung auf die neue Lib.

Dirk

Hallo Dirk,

jetzt wird mir schon einiges klarer. Ich weiß nicht ob ich bei der Umstellung auf die neue lib helfen kann, denke dazu reichen meine C-Kenntnisse noch nicht aus.
Ich habe mal von der App8-in aus alles imho "Unnütze" rausgelöscht um zu verstehen wie das funktioniert.

Geht prima, es lässt sich als Binäreingang Programmieren, ich kann alle Objekte auslesen und mit SetandTransmitBit() auch Telegramme senden.
Das ist ja super gelöst, für einen App-Programierer wirklich sehr komfortabel.
Jetzt in der Headerdatei COMMSTAB_ADDRESS, SOFTWARE_VERSION_NUMBER, MANUFACTORER_ADR und DEVICE_NUMBER angepasst und zack war’s ein 8-fach Aktor, prima, war einfach.
Nun mal den ain10V probiert: COMMSTAB_ADDRESS=0x3A, SOFTWARE_VERSION_NUMBER=0x01, MANUFACTORER_ADR=0x0004, DEVICE_NUMBER=0xB003
Das lässt sich zwar per ETS Parametrieren, ich kann aber keine Objekte lesen und SetandTransmitBit() lässt das ganze verm. abstürzen(Prog.Led bleibt nicht an).
Was ist falsch und wie könnte ich dann einen 8- bzw. 16-Bit Wert senden?

Gruß
Uwe

Hallo Uwe,

SetandTransmitBit() kann hier nicht funktionieren. Damit kannst du nur ein Bit verändern / übertragen. Deshalb auch der Name ...Bit().
Trotzdem dürfte die Funktion nicht abstürzen sondern einfach ohne Auswirkung zurückkehren. Intern wird geprüft ob der Datentyp zu dem Kommunikationsobjekt passt.

Für 8-16 Bit Werte oder noch mehr brauchst du SetAndTransmitObject().
SetAndTransmitBit() ist eigentlich nur ein shortcut für ein Bit. Siehe auch folgenden Code Auszug.



Bei Werten > 8 bit musst du dann aufpassen wegen dem Format. Die Kommunikationsobjekte erwarten die Daten immer im nativen KNX Format. D.h. du müsstet vorher selber die Zahlenformate konvertieren.

Gruß
Dirk

Aha ok Danke.
Aber ich sollte ja wenigstens bei leerer app_loop schonmal die Objekte bei entspr. gesetzten Flags per ETS auslesen können, was ja auch bei meinen 8-in, 8-out Versuchen geklappt hat. Deshalb mus ja noch was anderes faul sein.
Hier nochmal meine gekürzte Headerdatei:
Vielleicht springt ja jemanden von den C-Experten gleich ins Auge was da nicht stimmt. Eventuell die Commstab_Address 0x3A, das hab ich einfach aus der ain10V.h übernommen, weiß aber nicht wie man eigentlich auf diesen Wert kommt und ob das so stimmt.

Gruß
Uwe

Hallo Uwe,
so auf den ersten Blick sieht das gut aus. Habe keine Idee warum es nicht gehen sollte.
Commstab_Address 0x3A ist auch korrekt, das steht in der vd Datei.

Wenn du ein übersetzbares Projekt hast, könnte ich versuchen zu debuggen warum es nicht geht. Zumindest *.c und *.h sollten reichen.

Gruß
Dirk

Hallo Dirk,

ich hab dir gestern mal noch das Projekt gemailt, und nach dem Hinweis mit der vd Datei mich mal damit beschäftigt.
Also mal die vd vom Dimmer genommen und die entspr. Werte rausgesucht, damit hat es sofort geklappt, ich kann alle Objekte des Dimmers auslesen.
Nur der ain10V mag nicht.

Gruß
Uwe

Ja,
habe ich bekommen. Ich bräuchte nur nocht etwas mehr Zeit. Eine Kleinigkeit habe ich aber schon gefunden, aber das erklärt nicht alles.


Dirk

So, falls hier jemand mitliest, das Problem ist gelöst.
An der neuen lib lag es nicht, Dirk hat festgestellt das ein Parameter im EEPROM fehlte.
Der Zeiger auf die Assoziationstabelle wird normalerweise von der ETS beim programmieren mit übertragen.
Bei dieser Applikation nicht, deshalb muß unter:
const STRUCT_DEFPARAM defaultParam[] PROGMEM = {
noch diese Zeile eingefügt werden:
{ ASSOCTABPTR, 0x29 }, /**< ASSOCTAB Pointer */

Gruß
Uwe

Hallo,

das ist so nicht ganz richtig. Die ETS schreibt das schon mit, nur muss dort auch angewählt werden das Applikationsprogramm zu schreiben. Zuerst die PA und dann nur Parameter und Gruppenadressen reichen hier nicht aus.

erich73

Oh, wieder was gelernt.
Da scheint das aber die einzige Applikation zu sein, ich hab im ganzen Haus nur Parameter/Gruppenadressen programmiert.
Gerade noch mal bei der Dimmer Applikation geschaut, da wird "Adresse:0111 Daten:$29" mit übertragen.

Gruß
Uwe

Hallo Uwe,

das hängt damit zusammen wie die Tabellen verwaltet werden. Beim Analog_In ist die statisch (also in diesem Fall für 8GA + 8Zuordnungen), bei den Anderen wird die Größe dynamisch, je nach Anzahl der tatsächlich parametrierten GA+Zuordnungen, festgelegt.

erich73

So, ich hab mich jetzt mal an der Software versucht und sie an die neue lib angepasst.
Ich habe sie mit dem Mega328P und dem RF-Board getestet, und hoffe mal das alles halbwegs funktioniert.
Hier ist sie zu finden.

Gruß
Uwe

p.s. nicht wundern wenn der Code an manchen Stellen etwas komisch aussieht, ich übe noch.

Vielen Dank Uwe für Deine Mühe. Bin sehr glücklich, dass du mit der neuen Lib ganz gut zurecht kommst.
Ich hoffe die Bedienung ist einfach, falls du Verbesserungsvorschläge hast, kannst du dich jederzeit bei mir melden.

Ich habe nur eine Bitte (das betrifft alle Entwickler):
Legt bitte keine neuen Dateinamen an um darzustellen das es sich um neuen Code handelt oder das er neue Funktionen hat. Ändert einfach die alte Datei und schreibt das beim Commit ind die Commitmeldung mit rein. Somit kann es jeder in den Commitmeldungen lesen.
Falls es komplett experimenteller Code ist, der noch nicht funktioniert, könnte ihr auch einen neuen Branch anlegen und eure Änderungen dort machen. Diesen kann man dann später einfach in den Master Branch wieder zurück mergen.

Das hat den enormen Vorteil, das jede Änderungen auf dem Dateinamen nachvollziehbar bleibt und wenn es mal ein Problem geben sollte, ist es leichter herauszufinden welche Zeile das Problem verursachen könnte. Und jeder baut mal den ein oder anderen Bug ein
(falls ihr das nachlesen wollt, kann ich das Kapitel Branches udn Tags hier sehr empfehlen: http://git-scm.com/book/de)

Wünsche Euch allen noch wunderschöne Osterfeiertage.

_________________
Gruß
Matthias

Hallo Matthias,

ich habe mich nicht getraut den Code von jemand anderem mit meinem Machwerk zu überpinseln .
Werd mich aber in Zukunft daran halten.
Vielleicht kannst du ja den Link mit in deine Git-Anleitung aufnehmen.

Ja die neue lib ist wirklich recht einfach zu benutzen. Ich kann nur allen interessierten empfehlen sich einmal damit zu beschäftigen.
Man kann damit wirklich einfach eine freebus Applikation erstellen ohne sich um die Kommunikation mit der ETS oder um Gruppenadressen kümmern zu müssen.
Alles was man braucht sind die Objektnummern aus der ETS und die Adressen der Parameter im EEPROM, welche einem mit diesem Tool auf dem Silbertablett serviert werden.
Toll ist auch die Benutzung der Timer z.B. für das zykl. Senden gelöst. Einfach einen der zahlreich (wie viele eigentlich, ich hab hier 12 gebraucht) vorhandenen Timer anfordern, Zeit einstellen und bei Gelegenheit nachschauen ob er abgelaufen ist.
Einfacher geht’s wirklich nicht.
Also alle die gerne eine App nach ihren Wünschen hätten, probiert es aus.

Gruß
Uwe