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

tests: fix GPIO API usage issues #83059

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

Conversation

gmarull
Copy link
Member

@gmarull gmarull commented Dec 16, 2024

  • Add missing CONFIG_GPIO=y
  • Add/remove header if used/not used

A tiny step towards the removal of CONFIG_GPIO=y from all boards.

peter-mitsis
peter-mitsis previously approved these changes Dec 16, 2024
fabiobaltieri
fabiobaltieri previously approved these changes Dec 16, 2024
@fabiobaltieri
Copy link
Member

ci is sad, some header issue it looks like

All tests using the GPIO API must enable the CONFIG_GPIO Kconfig
option.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
Only include the header if using any of the GPIO API calls/
struct/defines.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
@gmarull gmarull dismissed stale reviews from fabiobaltieri and peter-mitsis via bf490ac December 17, 2024 08:38
@gmarull gmarull force-pushed the gpio-missing-kconfig-tests branch from 311a1ef to bf490ac Compare December 17, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants