NACK/Cmnt: [SRU][P:linux][PATCH 5/5] platform/x86/intel/pmc: Add Arrow Lake U/H support to intel_pmc_core driver
Manuel Diewald
manuel.diewald at canonical.com
Wed Jun 11 10:11:32 UTC 2025
On Wed, Jun 04, 2025 at 06:27:39PM +0200, Thibault Ferrante wrote:
> From: Xi Pardee <xi.pardee at linux.intel.com>
>
> BugLink: https://bugs.launchpad.net/bugs/2112469
>
> Add Arrow Lake U and Arrow Lake H support in intel_pmc_core driver.
>
> Signed-off-by: Rajvi Jingar <rajvi.jingar at linux.intel.com>
> Signed-off-by: Xi Pardee <xi.pardee at linux.intel.com>
> Link: https://lore.kernel.org/r/20250207225615.401235-7-xi.pardee@linux.intel.com
> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen at linux.intel.com>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen at linux.intel.com>
> (cherry picked from commit bd820906ea9dc3acfcac9de4f1be89b78609e2ac)
> Signed-off-by: Thibault Ferrante <thibault.ferrante at canonical.com>
> ---
> drivers/platform/x86/intel/pmc/arl.c | 37 +++++++++++++++++++++++++++
> drivers/platform/x86/intel/pmc/core.c | 2 ++
> drivers/platform/x86/intel/pmc/core.h | 1 +
> 3 files changed, 40 insertions(+)
>
> diff --git a/drivers/platform/x86/intel/pmc/arl.c b/drivers/platform/x86/intel/pmc/arl.c
> index f57da4e01f6f..54b0e8e56de9 100644
> --- a/drivers/platform/x86/intel/pmc/arl.c
> +++ b/drivers/platform/x86/intel/pmc/arl.c
> @@ -16,6 +16,7 @@
> #define IOEP_LPM_REQ_GUID 0x5077612
> #define SOCS_LPM_REQ_GUID 0x8478657
> #define PCHS_LPM_REQ_GUID 0x9684572
> +#define SOCM_LPM_REQ_GUID 0x2625030
>
> static const u8 ARL_LPM_REG_INDEX[] = {0, 4, 5, 6, 8, 9, 10, 11, 12, 13, 14, 15, 16, 20};
>
> @@ -650,6 +651,7 @@ const struct pmc_reg_map arl_pchs_reg_map = {
> .etr3_offset = ETR3_OFFSET,
> };
>
> +#define PMC_DEVID_SOCM 0x777f
> #define PMC_DEVID_SOCS 0xae7f
> #define PMC_DEVID_IOEP 0x7ecf
> #define PMC_DEVID_PCHS 0x7f27
> @@ -669,11 +671,17 @@ static struct pmc_info arl_pmc_info_list[] = {
> .devid = PMC_DEVID_PCHS,
> .map = &arl_pchs_reg_map,
> },
> + {
> + .guid = SOCM_LPM_REQ_GUID,
> + .devid = PMC_DEVID_SOCM,
> + .map = &mtl_socm_reg_map,
> + },
> {}
> };
>
> #define ARL_NPU_PCI_DEV 0xad1d
> #define ARL_GNA_PCI_DEV 0xae4c
> +#define ARL_H_GNA_PCI_DEV 0x774c
> /*
> * Set power state of select devices that do not have drivers to D3
> * so that they do not block Package C entry.
> @@ -684,6 +692,12 @@ static void arl_d3_fixup(void)
> pmc_core_set_device_d3(ARL_GNA_PCI_DEV);
> }
>
> +static void arl_h_d3_fixup(void)
> +{
> + pmc_core_set_device_d3(ARL_NPU_PCI_DEV);
> + pmc_core_set_device_d3(ARL_H_GNA_PCI_DEV);
> +}
> +
> static int arl_resume(struct pmc_dev *pmcdev)
> {
> arl_d3_fixup();
> @@ -691,12 +705,25 @@ static int arl_resume(struct pmc_dev *pmcdev)
> return cnl_resume(pmcdev);
> }
>
> +static int arl_h_resume(struct pmc_dev *pmcdev)
> +{
> + arl_h_d3_fixup();
> +
> + return cnl_resume(pmcdev);
> +}
> +
> static int arl_core_init(struct pmc_dev *pmcdev, struct pmc_dev_info *pmc_dev_info)
> {
> arl_d3_fixup();
> return generic_core_init(pmcdev, pmc_dev_info);
> }
>
> +static int arl_h_core_init(struct pmc_dev *pmcdev, struct pmc_dev_info *pmc_dev_info)
> +{
> + arl_h_d3_fixup();
> + return generic_core_init(pmcdev, pmc_dev_info);
> +}
> +
> struct pmc_dev_info arl_pmc_dev = {
> .pci_func = 0,
> .dmu_guid = ARL_PMT_DMU_GUID,
> @@ -706,3 +733,13 @@ struct pmc_dev_info arl_pmc_dev = {
> .resume = arl_resume,
> .init = arl_core_init,
> };
> +
> +struct pmc_dev_info arl_h_pmc_dev = {
> + .pci_func = 2,
> + .dmu_guid = ARL_PMT_DMU_GUID,
> + .regmap_list = arl_pmc_info_list,
> + .map = &mtl_socm_reg_map,
> + .suspend = cnl_suspend,
> + .resume = arl_h_resume,
> + .init = arl_h_core_init,
> +};
> diff --git a/drivers/platform/x86/intel/pmc/core.c b/drivers/platform/x86/intel/pmc/core.c
> index 0503a3d000fb..1b85800ec10f 100644
> --- a/drivers/platform/x86/intel/pmc/core.c
> +++ b/drivers/platform/x86/intel/pmc/core.c
> @@ -1410,6 +1410,8 @@ static const struct x86_cpu_id intel_pmc_core_ids[] = {
> X86_MATCH_VFM(INTEL_RAPTORLAKE_S, &adl_pmc_dev),
> X86_MATCH_VFM(INTEL_METEORLAKE_L, &mtl_pmc_dev),
> X86_MATCH_VFM(INTEL_ARROWLAKE, &arl_pmc_dev),
> + X86_MATCH_VFM(INTEL_ARROWLAKE_H, &arl_h_pmc_dev),
> + X86_MATCH_VFM(INTEL_ARROWLAKE_U, &arl_h_pmc_dev),
> X86_MATCH_VFM(INTEL_LUNARLAKE_M, &lnl_pmc_dev),
> {}
> };
> diff --git a/drivers/platform/x86/intel/pmc/core.h b/drivers/platform/x86/intel/pmc/core.h
> index b004b784bfeb..987fde3ffc1c 100644
> --- a/drivers/platform/x86/intel/pmc/core.h
> +++ b/drivers/platform/x86/intel/pmc/core.h
> @@ -624,6 +624,7 @@ extern struct pmc_dev_info tgl_pmc_dev;
> extern struct pmc_dev_info adl_pmc_dev;
> extern struct pmc_dev_info mtl_pmc_dev;
> extern struct pmc_dev_info arl_pmc_dev;
> +extern struct pmc_dev_info arl_h_pmc_dev;
> extern struct pmc_dev_info lnl_pmc_dev;
>
> void cnl_suspend(struct pmc_dev *pmcdev);
> --
> 2.48.1
>
>
> --
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
I think there's a fix commit available in 6.15 that we should
consider including:
f2eae58c4428 platform/x86/intel/pmc: Fix Arrow Lake U/H NPU PCI ID
--
Manuel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20250611/5c49aa8a/attachment.sig>
More information about the kernel-team
mailing list