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

Bugfix FSK repeats #309

Merged
merged 1 commit into from
Nov 25, 2023
Merged

Bugfix FSK repeats #309

merged 1 commit into from
Nov 25, 2023

Conversation

elektron-bbs
Copy link
Contributor

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.

No repeats are sent with FSK modulation.
@elektron-bbs elektron-bbs requested a review from sidey79 November 20, 2023 12:42
@sidey79
Copy link
Contributor

sidey79 commented Nov 20, 2023

Sollten wir sendTransmitMode vielicht in sendFifo packen?

@elektron-bbs
Copy link
Contributor Author

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
Copy link
Contributor

sidey79 commented Nov 20, 2023

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
Copy link
Contributor Author

Ein paar Zeilen vorher wird setTransmitMode ausgeführt:

#ifdef CMP_CC1101

@sidey79
Copy link
Contributor

sidey79 commented Nov 25, 2023

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
Copy link
Contributor Author

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.

@elektron-bbs elektron-bbs merged commit 8140b4e into master Nov 25, 2023
34 checks passed
@elektron-bbs elektron-bbs deleted the master_bugfix_FSK_repeats branch November 1, 2024 16:16
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