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

Add esp32 ble_scan example. #275

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add esp32 ble_scan example. #275

wants to merge 4 commits into from

Conversation

elrom
Copy link

@elrom elrom commented Jan 27, 2025

No description provided.

@elrom
Copy link
Author

elrom commented Jan 27, 2025

Interestingly, at least on my board (esp32s3), I'm not getting any results from the scan. @lulf any thoughts?

@lulf
Copy link
Member

lulf commented Jan 29, 2025

@elrom I've only tested it on nrf, so maybe an esp-specific thing yeah. I'll try to test this on my esp32c3 and see if I can find anything.

@lulf
Copy link
Member

lulf commented Feb 20, 2025

@elrom Seems like there is someone who got it to work after making L2CAP_MTU configurable in the example, is you make that modification I'd be happy to merge this PR (ref #288)

@Meigs2
Copy link
Contributor

Meigs2 commented Feb 20, 2025

Yes, all you have to do is pass in a const generic for the run method, pub async fn run<C, const L2CAP_MTU: usize>(controller: C) and use that const instead of the hardcoded one in the example. This will require modification to the other examples to pass in a value for L2CAP_MTU to get them to compile.

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