[PATCH] FHT TF Fensterkontakt-Daten senden

Begonnen von Matscher, 27 November 2014, 20:53:05

Vorheriges Thema - Nächstes Thema

Matscher

Hallo Zusammen,

anbei ein Patch für die Erweiterung der culfw zum aussenden von FHT TF Fensterkontakt-Daten. Weiterhin habe ich gleich die commandref und culfw.html angepasst,  falls es auf den Branch einfließt. :)

Test- und Diskussionsthread zum: http://forum.fhem.de/index.php/topic,27465.0.html

Im Moment sind 4 TFs möglich. Das kann bei belieben auch per #define erhöht werden. Bis jetzt sind keine Probleme bekannt. Somit ist es möglich, verschiedenste Sensoren per virtuellen FHT TF an die FHTs anzumelden.

Was meint ihr dazu?

Danke und Grüße.


Auszug aus der commandRef.html:

The 80b is designed to handle up to 4 FHT80TF window sensors. A housecode of FHT80TF starts at the first byte above 0x69.

To setup one or more window sensors, use the commands below. Set the FHT80b in syncing mode for sensors and start with a sync, followed by a finished command. The default value after this procedure is window closed (0x02). Now, you are be able to set the right value. Use the following list of commands:

    T8630A00C - start sync
    T8630A00F - finished sync
    T8630A001 - window open
    T8630A002 - window closed
    T011234 (set HC to 1234) - clear the buffers
    T12 check FHT80TF buffer, could tell 00:8630A001, if empty -> N/A


(8630A0 -> example address of FHT80TF)
Rasp 3
CUL V3 868Mhz + nanoCUL 868Mhz als RFR + nanoCUL 868Mhz für Homematic + SIGNALduino
Zigbee CC2531 - Aquara TempSensor
MySensors Ethernet Gateway, Water meter, Gas meter
Modul: 09_CUL_FHTTK.pm (assumed), culfw part HAS_FHT_TF

rudolfkoenig

Da es um clock.c und fht.c geht, fuehle ich mich zustaendig.

Wenn ich das Patch uebernehmen soll, dann bitte:
- die Formatierung an dem in dieser Datei ueblichen anpassen, d.h. unter anderem: auf 80-Zeichen Breite formatieren, keine Tabs verwenden, einruecktiefe 2 Stellen.
- das Patch sollte commandref.html enthalten
- die Dateien Devices/*/board.h passend ergenzen.

Matscher

Okay danke. Ich korrigiere das gleich heute Abend.
Rasp 3
CUL V3 868Mhz + nanoCUL 868Mhz als RFR + nanoCUL 868Mhz für Homematic + SIGNALduino
Zigbee CC2531 - Aquara TempSensor
MySensors Ethernet Gateway, Water meter, Gas meter
Modul: 09_CUL_FHTTK.pm (assumed), culfw part HAS_FHT_TF

Matscher

#3
So jetzt sollte alles dabei sein und die Formatierung passen
Rasp 3
CUL V3 868Mhz + nanoCUL 868Mhz als RFR + nanoCUL 868Mhz für Homematic + SIGNALduino
Zigbee CC2531 - Aquara TempSensor
MySensors Ethernet Gateway, Water meter, Gas meter
Modul: 09_CUL_FHTTK.pm (assumed), culfw part HAS_FHT_TF

Matscher

Hallo Rudolf,

passt der Patch so oder habe ich noch etwas wichtiges vergessen? Ich habe die board.h der Devices, welche schon die FHT80 und FHT8V benutzen, um fht80TF erweitert. Weiterhin habe ich die commandref.html ergänzt und denke das das so jeder verstehen wird. Und natürlich die Formatierung :)

Danke und Grüße,
Steve
Rasp 3
CUL V3 868Mhz + nanoCUL 868Mhz als RFR + nanoCUL 868Mhz für Homematic + SIGNALduino
Zigbee CC2531 - Aquara TempSensor
MySensors Ethernet Gateway, Water meter, Gas meter
Modul: 09_CUL_FHTTK.pm (assumed), culfw part HAS_FHT_TF

rudolfkoenig

Zitatpasst der Patch so

Geht so.
- patch beschwert sich wg. CR/NL, die Dateien haben wohl ein DOS-Editor kennengelernt.
- das Patchen von Devices/rpiaddon/board.h geht schief.
- beim Uebersetzten beschwert sich der Compiler:
../../clib/clock.c: In function '__vector_19':
../../clib/clock.c:80:35: warning: operation on 'fht_tf_timeout_Array[(int)i * 3]' may be undefined [-Wsequence-point]
       fht_tf_timeout_Array[3 * i] = fht_tf_timeout_Array[3 * i]--;

../../clib/fht.c: In function 'fht_tf_timer':
../../clib/fht.c:363:33: warning: operation on 'fht_tf_timeout_Array[(int)ind * 3 + 2]' may be undefined [-Wsequence-point]
   fht_tf_timeout_Array[3*ind+2] = fht_tf_timeout_Array[3*ind+2]++;

und da hat er schwer Recht. Habs gefixt, aber nicht getestet.
- Wg. den RFMBUS Patches kann man CUNO2 nicht uebersetzen (das ist wohl nicht deine Schuld)
- die CHANGED Datei wurde nicht geaendert.
- habe einige Formulierungen in den Kommentaren leicht angepasst.

Habs eingecheckt.

Matscher

Rasp 3
CUL V3 868Mhz + nanoCUL 868Mhz als RFR + nanoCUL 868Mhz für Homematic + SIGNALduino
Zigbee CC2531 - Aquara TempSensor
MySensors Ethernet Gateway, Water meter, Gas meter
Modul: 09_CUL_FHTTK.pm (assumed), culfw part HAS_FHT_TF