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

include: drivers: peci: Add missing subsystem macro #51415

Merged
merged 2 commits into from
Nov 8, 2022

Conversation

albertofloyd
Copy link
Collaborator

Add subsystem macro for peci driver API.
Add MEC172x overlay to samples/drivers/peci

Fixes #45222

@albertofloyd albertofloyd requested a review from nashif as a code owner October 19, 2022 00:59
@zephyrbot zephyrbot added the area: PECI Platform Environment Control Interface label Oct 19, 2022
nashif
nashif previously approved these changes Oct 19, 2022
@albertofloyd
Copy link
Collaborator Author

Build failure is unrelated to this PR.
Same build failure is is seen in mainline without this change.

It seem it has been there for a while but when unnoticed because CONFIG_PECI=y didn't result in CONFIG_PECI_NPCX=y
Raised an issue #51464

Add subsystem macro for peci driver API.

Signed-off-by: Jose Alberto Meza <[email protected]>
Define DT alias for PECI

Signed-off-by: Jose Alberto Meza <[email protected]>
@albertofloyd
Copy link
Collaborator Author

@nashif could re-review this? Build is now passing.

@fabiobaltieri fabiobaltieri merged commit 2a8f718 into zephyrproject-rtos:main Nov 8, 2022
@albertofloyd albertofloyd deleted the peci_userspace branch February 7, 2023 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: PECI Platform Environment Control Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

drivers: peci: user space handlers not building correctly
6 participants