Re: [PATCH 0/3] platform/x86/amd/hsmp: ACPI, Non-ACPI code split and other misc changes

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

On 4/23/2024 04:14, Suma Hegde wrote:
> Patch1:
> The first patch is to address the concerns mentioned by Limonciello,
> Mario i.e to address the issue of
> "Probing registers that don't match the same mailboxes randomly on
> client or embedded parts might lead to unexpected behaviors".
> 
> Because of some limitations, instead of the methods suggested by Mario,
> family check is added to differentiate server socs and client socs.
> This check is only for backward compatibility, Going forward all
> systems will have ACPI based probing only.
> 

Thanks!  Hans, please drop the PM profile patch from your review queue 
in favor of this series instead.

> 
> Patch2:
> The second patch splits ACPI code into  separate file there by removing the
> dependency of ACPI in Kconfig.
> This is based on the suggestion from Ilpo Jarvinen.
> 
> 
> Patch3:
> The third patch makes HSMP as "default m" so that anyone who uses standard
> distros will be able to use HSMP by dynamically loading it.
> 
> 
> Suma Hegde (3):
>    platform/x86/amd/hsmp: Check HSMP support on AMD family of processors
>    platform/x86/amd/hsmp: Split the ACPI and non-ACPI code
>    platform/x86/amd/hsmp: Make HSMP as default m
> 
>   MAINTAINERS                                |   2 +-
>   drivers/platform/x86/amd/Kconfig           |  14 +-
>   drivers/platform/x86/amd/Makefile          |   3 +-
>   drivers/platform/x86/amd/hsmp/Kconfig      |  18 +
>   drivers/platform/x86/amd/hsmp/Makefile     |   9 +
>   drivers/platform/x86/amd/hsmp/hsmp-acpi.c  | 219 ++++++++++++
>   drivers/platform/x86/amd/{ => hsmp}/hsmp.c | 378 +++++----------------
>   drivers/platform/x86/amd/hsmp/hsmp.h       |  82 +++++
>   8 files changed, 425 insertions(+), 300 deletions(-)
>   create mode 100644 drivers/platform/x86/amd/hsmp/Kconfig
>   create mode 100644 drivers/platform/x86/amd/hsmp/Makefile
>   create mode 100644 drivers/platform/x86/amd/hsmp/hsmp-acpi.c
>   rename drivers/platform/x86/amd/{ => hsmp}/hsmp.c (71%)
>   create mode 100644 drivers/platform/x86/amd/hsmp/hsmp.h
>