Skip to content

Conversation

@laurensvalk
Copy link
Member

Attempted fix for pybricks/support#2176, but cannot reproduce locally.

@coveralls
Copy link

coveralls commented Oct 6, 2025

Coverage Status

coverage: 59.789% (-0.008%) from 59.797%
when pulling b45b272 on power-button-debounce
into 4af839f on master.

@laurensvalk laurensvalk force-pushed the power-button-debounce branch from 7d56fe8 to 503b524 Compare October 7, 2025 10:04
@laurensvalk laurensvalk force-pushed the power-button-debounce branch from 503b524 to b45b272 Compare October 7, 2025 11:08
@laurensvalk laurensvalk merged commit b45b272 into master Oct 7, 2025
32 checks passed
@dlech
Copy link
Member

dlech commented Oct 7, 2025

I don't understand what this actually does and there is no explanation in the commit message.

@laurensvalk
Copy link
Member Author

Hmm, this wasn't supposed to be merged yet.

@dlech
Copy link
Member

dlech commented Oct 7, 2025

Oops. 😄

@laurensvalk
Copy link
Member Author

But what it does is:

  • Use pbdrv_button_deinit() to request the gpio debounce process to gracefully stop.
  • It waits for release, it waits another 200 ms before considering deinit complete, so as to be sure it is really released before we proceed with the rest of poweroff.

@dlech
Copy link
Member

dlech commented Oct 7, 2025

Ah, makes sense now.

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.

4 participants