RFD-FHEM / SIGNALDuino

System to capture digital signaldata and transfer them to another system
GNU General Public License v3.0
83 stars 35 forks source link

Bugfix FSK repeats #309

Closed elektron-bbs closed 10 months ago

elektron-bbs commented 10 months ago

No repeats are sent with FSK modulation.

Since the CC1101 automatically switched to receive mode after the first send, no repeats were sent. Now the CC1101 will first be put back into transmit mode with each repeat.

sidey79 commented 10 months ago

Sollten wir sendTransmitMode vielicht in sendFifo packen?

elektron-bbs commented 10 months ago

Mhmm, dort haben wir aber wahrscheinlich die Anzahl der Wiederholungen nicht zur Verfügung und würden beim ersten Senden zweimal setTransmitMode ausführen?

if (rep > 0) { cc1101::setTransmitMode(); }

sidey79 commented 10 months ago

Mhmm, dort haben wir aber wahrscheinlich die Anzahl der Wiederholungen nicht zur Verfügung und würden beim ersten Senden zweimal setTransmitMode ausführen?

if (rep > 0) { cc1101::setTransmitMode(); }

Das mag sein. Beim Aufruf von sendFiFo würde ich erwarten dass ich sende.

Wo wird setTransmitMode denn sonst noch ausgeführt, wenn nicht in sendFiFo.

elektron-bbs commented 10 months ago

Ein paar Zeilen vorher wird setTransmitMode ausgeführt: https://github.com/RFD-FHEM/SIGNALDuino/blob/23d2e2bd41f8144882466de2200b1b3c3ab8c07b/src/send.h#L274

sidey79 commented 10 months ago

Wir brauchen setTransmitMode vermutlich auch für send_raw und send_MC oder?

Von daher, können wir es nicht in sendFiFo verschieben, was so meine spontane Idee war.

elektron-bbs commented 10 months ago

Ja, ich hatte es an der erwähnten Stelle schon mal auskommentiert, da ging kein Senden von MU mehr. Manchester habe ich nicht probiert, aber da trifft das vermutlich auch zu.