Skip to content

Fix ticks #7845

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

Merged
merged 2 commits into from
Apr 7, 2023
Merged

Fix ticks #7845

merged 2 commits into from
Apr 7, 2023

Conversation

tannewt
Copy link
Member

@tannewt tannewt commented Apr 7, 2023

In #7497 port_background_task was renamed to port_background_tick but the actual call site wasn't changed. This meant that it was no longer called!

Rename more functions from task to tick to make it clearer which is which.

In micropython#7497 port_background_task was renamed to port_background_tick
but the actual call site wasn't changed. This meant that it was
no longer called!

Rename more functions from task to tick to make it clearer which is
which.
dhalbert
dhalbert previously approved these changes Apr 7, 2023
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

This makes sense to me. @DavePutz could you try this as a PulseOut fix?

Copy link
Collaborator

@DavePutz DavePutz left a comment

Choose a reason for hiding this comment

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

Verified that PulseIn and PulseOut both work correctly with this patch on ESP32-S2

@dhalbert dhalbert merged commit e4c6d76 into adafruit:main Apr 7, 2023
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