Re: [PATCH v2 0/5] PMF NPU metrics cleanup, command flag cleanup, and amdxdna integration

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

On 11/11/25 12:37 AM, Shyam Sundar S K wrote:
> This series streamlines NPU-related handling in AMD PMF, adds a
> lightweight in-kernel interface for fetching NPU metrics from PMF, and
> hooks AMD XDNA to that interface to expose real-time NPU power data to
> user space.
> 
> Changes include renaming legacy IPU fields to NPU in the metrics structure
> replacing ambiguous booleans and magic values with explicit
> SET_CMD/GET_CMD and METRICS_TABLE_ID constants, introducing an exported
> amd_pmf_get_npu_data() API (guarded by a mutex and platform checks) to
> pull NPU metrics from the SMU, and wiring those metrics into the amdxdna
> driver.
> 
> v2:
> ----
>   - Split the series into multiple patches per v1 feedback
>   - Added preliminary PMF driver cleanups as preparatory patches
>   - Introduced an initial amd_xdna patch that consumes PMF-provided data to
>     establish the plumbing; additional logic can be layered on in future
>     patches as more PMF metrics are utilized.
> 
> Shyam Sundar S K (4):
>    platform/x86/amd/pmf: Rename IPU metrics fields to NPU for consistency
>    platform/x86/amd/pmf: Use explicit SET_CMD/GET_CMD flags in
>      amd_pmf_send_cmd()
>    platform/x86/amd/pmf: replace magic table id with METRICS_TABLE_ID
>    platform/x86/amd/pmf: Introduce new interface to export NPU metrics
> 
> VinitKumar Shukla (1):
>    accel/amdxdna: Provide real-time NPU power estimate via AMD PMF
> 
>   drivers/accel/amdxdna/aie2_pci.h         |  2 +
>   drivers/accel/amdxdna/aie2_smu.c         | 12 ++++
>   drivers/platform/x86/amd/pmf/auto-mode.c | 14 ++---
>   drivers/platform/x86/amd/pmf/cnqf.c      | 14 ++---
>   drivers/platform/x86/amd/pmf/core.c      | 71 +++++++++++++++++++++++-
>   drivers/platform/x86/amd/pmf/pmf.h       | 18 ++++--
>   drivers/platform/x86/amd/pmf/spc.c       |  2 +-
>   drivers/platform/x86/amd/pmf/sps.c       | 38 ++++++-------
>   drivers/platform/x86/amd/pmf/tee-if.c    | 20 +++----
>   include/linux/amd-pmf-io.h               | 21 +++++++
>   10 files changed, 159 insertions(+), 53 deletions(-)
> 

Which tree do you think this should merge through?
drm-misc or platform-x86?

I tried and the series cleanly applies to both and compiles on both.  So 
I think it can go either way.