Re: [PATCH v2 0/2] Add new capabilities to PMF Smart PC

[Mario Limonciello <mario.limonciello@amd.com>]

On 12/17/2024 09:16, Shyam Sundar S K wrote:
> This series introduces the following enhancements:
> 
> - Support for device operating states in the AMD SFH driver through the
> SRA sensor.
> - Functionality in the PMF driver to export SRA data from the SFH driver
> to the PMF, allowing it to be used as input for the PMF-TA in evaluating
> policy conditions that involve SRA sensor data.
> 
> v2:
> -----
> - Address cosmetic remarks from Mario
> - Remove explicit assignment to enums in SFH.
> 
> Additional notes:
> 
> 1) This series is based on Mainline(v6.13-rc3)
> 2) This series builds upon the recent commit to PMF found in
> platform-drivers-x86/review-ilpo-next
> 
> 6000bc1f5a81 ("platform/x86/amd/pmf: Enable Custom BIOS Inputs for PMF-TA")
> 
> Aside from these two points, there should be no issues when merging them.
> 
> Basavaraj Natikar (1):
>    HID: amd_sfh: Add support to export device operating states
> 
> Shyam Sundar S K (1):
>    platform/x86/amd/pmf: Get SRA sensor data from AMD SFH driver
> 
>   drivers/hid/amd-sfh-hid/amd_sfh_common.h      |  1 +
>   drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c | 22 ++++++++
>   .../amd-sfh-hid/sfh1_1/amd_sfh_interface.c    | 38 ++++++++++++++
>   .../amd-sfh-hid/sfh1_1/amd_sfh_interface.h    | 24 ++++++++-
>   drivers/platform/x86/amd/pmf/pmf.h            | 18 ++++++-
>   drivers/platform/x86/amd/pmf/spc.c            | 51 +++++++++++++++++++
>   include/linux/amd-pmf-io.h                    | 15 ++++++
>   7 files changed, 166 insertions(+), 3 deletions(-)
> 

Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>