Skip to content

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
schkovich pushed a commit to schkovich/arduino-pico that referenced this pull request May 24, 2025
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.

3 participants