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

Arduino.h: Add digitalPinToInterrupt #121

Merged

Conversation

Ayush1325
Copy link
Member

  • Return the same number as long as not more than number of digital pins.
  • This does not seem to be present in ArduinoCoreAPI but defined for all Arduino boards (and microblocks has started using it).
  • Also present in arduino docs 1

@Ayush1325 Ayush1325 force-pushed the digitalPinToInterrupt branch 2 times, most recently from dd45536 to 0e52b77 Compare September 2, 2024 13:13
cores/arduino/Arduino.h Outdated Show resolved Hide resolved
@Ayush1325 Ayush1325 force-pushed the digitalPinToInterrupt branch from 0e52b77 to 098ad74 Compare September 5, 2024 12:09
@Ayush1325
Copy link
Member Author

Ayush1325 commented Sep 5, 2024

I tried putting it in the anonymous namespace, but then I got undefined reference errors. So not sure if interrupts and noInterrupts are working or not.

Note: I am not a C++ dev and have never used namespaces anything newer than C++98. So my understanding here is rather limited.

@Ayush1325 Ayush1325 force-pushed the digitalPinToInterrupt branch from 098ad74 to 75846ac Compare September 5, 2024 12:13
- This does not seem to be present in ArduinoCoreAPI but defined for all
  Arduino boards (and microblocks has started using it).
- Also present in arduino docs.

Signed-off-by: Ayush Singh <[email protected]>
@DhruvaG2000 DhruvaG2000 merged commit dc24cf0 into zephyrproject-rtos:next Sep 15, 2024
3 checks passed
@Ayush1325 Ayush1325 deleted the digitalPinToInterrupt branch September 15, 2024 06:55
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