search

openbmc / x86-power-control

Apache License 2.0
12 stars 9 forks source link

Support AMD CPUs #8

Open Kostr opened 3 years ago

Kostr commented 3 years ago

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 commented 3 years ago

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 commented 2 years ago

@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