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

Support AMD CPUs #8

Open
Kostr opened this issue Mar 21, 2021 · 2 comments
Open

Support AMD CPUs #8

Kostr opened this issue Mar 21, 2021 · 2 comments

Comments

@Kostr
Copy link
Contributor

Kostr commented Mar 21, 2021

Currently x86-power-control app supports only the Intel CPUs.
To get it working with the AMD CPUs the Ethanol CRB needs a patch that is provided it its recipies (https://github.com/openbmc/openbmc/blob/master/meta-amd/meta-ethanolx/recipes-x86/chassis/x86-power-control/0001-Amd-power-control-modifications-for-EthanolX.patch).
Is it possible to integrate the AMD power control logic in the x86-power-control app? Peharps with the choice between CPU vendors as a compile option?

@vejmarie
Copy link
Contributor

Is that logic applicable to many AMD board ? We have openbmc working on ProLiant AMD with a common logic with Intel I think. I need to check that. Following the integration of a polarity option it might be doable to integrate the patch that your refer to, but I don't have any system to test.

@Kostr
Copy link
Contributor Author

Kostr commented Sep 21, 2021

@vejmarie I'm not sure about many boards, currently I have only EthanolX. But I saw a patch for DaytonaX board in gerrit that looked exactly the same https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/41677
Also AMD has created a fork of x86-power-control, maybe it has some clues about the common logic in AMD boards https://github.com/AMDESE/amd-power-control or maybe Supreeth Venkatesh @supven01 can give us some guides

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

No branches or pull requests

2 participants