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

[PHY, modules, LoRaWAN] Generalize IRQ checks #1188

Merged
merged 9 commits into from
Aug 18, 2024
Merged

[PHY, modules, LoRaWAN] Generalize IRQ checks #1188

merged 9 commits into from
Aug 18, 2024

Conversation

StevenCellist
Copy link
Collaborator

Instead of using a specific isRxTimeout() function, this PR adds a general function isIrqSet() for most common IRQ values on a range of modules (SX126x, SX127x, SX128x and LR11x0). This makes it easy to retrieve low-level IRQ status flags while retaining the radio abstraction. I am not sure if / how to implement this for other classes; they seem to work in a different way.

As the LoRaWAN stack used the function isRxTimeout(), it has been modified accordingly and is checked to run as usual.

Copy link
Owner

@jgromes jgromes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you - I left some comments, but nothing major.

PS: I also pushed an update to master to fix ESP-IDF CI and merged it here.

src/protocols/PhysicalLayer/PhysicalLayer.h Outdated Show resolved Hide resolved
src/protocols/PhysicalLayer/PhysicalLayer.h Outdated Show resolved Hide resolved
@jgromes jgromes merged commit 0abb751 into master Aug 18, 2024
30 checks passed
@jgromes jgromes deleted the irqFlags branch August 18, 2024 07:57
@jgromes
Copy link
Owner

jgromes commented Aug 18, 2024

All looks good now - merged, thanks!

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