Hi, On 10/20/21 18:29, Mario Limonciello wrote: > Currently the "argument" for the "message" is listed as a boolean > value. This works well for the commands used currently, but an > additional upcoming command will pass more data in the message. > > Expand it to be a full 32 bit value. > > Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx> Thank you for your patch-series, I've applied the series to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > drivers/platform/x86/amd-pmc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/platform/x86/amd-pmc.c b/drivers/platform/x86/amd-pmc.c > index b7b6ad4629a7..99ac50616bc3 100644 > --- a/drivers/platform/x86/amd-pmc.c > +++ b/drivers/platform/x86/amd-pmc.c > @@ -126,7 +126,7 @@ struct amd_pmc_dev { > }; > > static struct amd_pmc_dev pmc; > -static int amd_pmc_send_cmd(struct amd_pmc_dev *dev, bool set, u32 *data, u8 msg, bool ret); > +static int amd_pmc_send_cmd(struct amd_pmc_dev *dev, u32 arg, u32 *data, u8 msg, bool ret); > > static inline u32 amd_pmc_reg_read(struct amd_pmc_dev *dev, int reg_offset) > { > @@ -337,7 +337,7 @@ static void amd_pmc_dump_registers(struct amd_pmc_dev *dev) > dev_dbg(dev->dev, "AMD_PMC_REGISTER_MESSAGE:%x\n", value); > } > > -static int amd_pmc_send_cmd(struct amd_pmc_dev *dev, bool set, u32 *data, u8 msg, bool ret) > +static int amd_pmc_send_cmd(struct amd_pmc_dev *dev, u32 arg, u32 *data, u8 msg, bool ret) > { > int rc; > u32 val; > @@ -356,7 +356,7 @@ static int amd_pmc_send_cmd(struct amd_pmc_dev *dev, bool set, u32 *data, u8 msg > amd_pmc_reg_write(dev, AMD_PMC_REGISTER_RESPONSE, 0); > > /* Write argument into response register */ > - amd_pmc_reg_write(dev, AMD_PMC_REGISTER_ARGUMENT, set); > + amd_pmc_reg_write(dev, AMD_PMC_REGISTER_ARGUMENT, arg); > > /* Write message ID to message ID register */ > amd_pmc_reg_write(dev, AMD_PMC_REGISTER_MESSAGE, msg); >
