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

Simplify example of stdio_set_chars_available_callback #495

Merged

Conversation

peterharperuk
Copy link
Contributor

The example in here uses an async callback to avoid a race when using stdio_usb. But there's a fix available for this issue so simplify the example.

To test pass -DPICO_STDIO_USB=1 on the cmake command line.

Fixes #461

The example in here uses an async callback to avoid a race when using
stdio_usb. But there's a fix available for this issue so simplify the
example.

To test pass -DPICO_STDIO_USB=1 on the cmake command line.

Fixes raspberrypi#461
@peterharperuk
Copy link
Contributor Author

Requires this fix in the sdk raspberrypi/pico-sdk#1649

@peterharperuk peterharperuk added this to the 1.5.2 milestone May 1, 2024
@peterharperuk
Copy link
Contributor Author

@kilograham Simple fix to remove some of my sillyness. Will merge if that's ok.

Copy link
Contributor

@kilograham kilograham left a comment

Choose a reason for hiding this comment

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

prepared to believe you ;-) though curious what / why of the disable_sta_mode?

@peterharperuk
Copy link
Contributor Author

This is "demonstrating" clean shutdown of the wifi stack. The loop keeps going while the link is up so pressing "d" calls disable_sta_mode which disables wifi and causes the program to exit.

@peterharperuk peterharperuk merged commit cbb5d61 into raspberrypi:develop Jul 12, 2024
1 check passed
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