10000 Add PDM library for Arduino Nano RP2040 Connect by earlephilhower · Pull Request #213 · earlephilhower/arduino-pico · GitHub
[go: up one dir, main page]
Skip to content

Add PDM library for Arduino Nano RP2040 Connect #213

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 16, 2021
Merged

Add PDM library for Arduino Nano RP2040 Connect #213

merged 2 commits into from
Jun 16, 2021

Conversation

earlephilhower
Copy link
Owner

Thanks @Claupio

@earlephilhower
Copy link
Owner Author

@Claupio, where is the PDM object instantiated in the main MBED version? Does the app need to make a PDM object, or is it part of the core? Looks like the library probably needs a little patch to instantiate it and make the PDM example pass.

@Claupio
Copy link
Claupio commented Jun 16, 2021

@Claupio, where is the PDM object instantiated in the main MBED version? Does the app need to make a PDM object, or is it part of the core? Looks like the library probably needs a little patch to instantiate it and make the PDM example pass.

The PDM object is declared in the library in PDM.cpp. In the instance declaration there is a check to see If the PDM pins were defined. Before compiling did you replaced the arduino pins file?

@earlephilhower
Copy link
Owner Author

Yes, that makes sense. I run CI only on the RPi Pico board, so the PDM example obviously will fail to build there. I've removed it and when CI goes through fine I'll merge and bump.

@earlephilhower earlephilhower merged commit b02b9c4 into master Jun 16, 2021
@earlephilhower earlephilhower deleted the pdm branch June 16, 2021 15:17
schkovich pushed a commit to schkovich/arduino-pico that referenced this pull request May 24, 2025
@lurch
Add SPI enums to Doxygen output, fixes earlephilhower#213 (earlephilh…
2592c59 
…ower#216)

Also fix a few typos
schkovich pushed a commit to schkovich/arduino-pico that referenced this pull request May 24, 2025
@lurch @kilograham
Add SPI enums to Doxygen output, fixes earlephilhower#213 (earlephilh…
1a2bb7c 
…ower#216)

Also fix a few typos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@earlephilhower @Claupio
0