[v5,0/5] Create Intel PMC SSRAM Telemetry driver

This patch series removes the SSRAM support from Intel PMC Core driver
and creates a separate PCI driver for SSRAM device. The new Intel PMC
SSRAM driver provides the following functionalities:

1. Search and store the PMC information in a structure, including PWRMBASE
address and devid for each available PMC. Then Intel PMC Core driver
achieves the PMC information using the API provided by the new driver.
2. Search and register Intel Platform Monitoring Techology telemetry
regions so they would by available for read through sysfs and Intel PMT
API. Intel PMC Core driver can achieve Low Power Mode requirement
information from a telemetry region registered by the new driver.

The above functionalities was previously handled by Intel PMC Core
driver. Intel PMC Core driver returns -EPROBE_DEFER when trying to read
data from a telem region that is not available yet. This setup may
result in an infinite loop of .probe() calls as Intel PMC Core driver
creates child devices. Creating a separate PCI driver avoids the infinite
loop possibility.

v5->v4:
- Remove unneeded changes in pmc_core_ssram_get_lpm_reqs()
- Replace mutex_init() with devm_mutex_init()
 
V4->v3:
- Remove the first six patches that are accepted.
- Add a patch to change the return type of intel_vsec_registeri() from
  void to int.
- Change the return type of pmc_ssram_telemetry_get_pmc() so the probe
  function of Intel PMC SSRAM Telemetry driver will fail if
  intel_vsec_register() fails. The previous implementation hides the
  failure.
- Use __free(pci_dev_put) in core.c.
- Move pmc_ssram_telemetry_get_pmc_info() function comment to c file
  from h file.
- Add a patch to improve pmc_core_get_lpm_req() suggested by comment
  from last version.
 
v3->v2:
- Add memory barriers to the new driver to ensure write/read order of
  device_probed variable.
- Minor grammar changes: add needed white space and end of life new line.
- Add patch to move error handling to init function.
- Remove patch to enable SSRAM support of LNL platforms. This patch will be
  included in a separate series.

v2->v1:
- Rearrange and restructure patches completely based on feedback from v1
 
Xi Pardee (5):
  platform/x86:intel/vsec: Change return type of intel_vsec_register
  platform/x86:intel/pmc: Create Intel PMC SSRAM Telemetry driver
  platform/x86:intel/pmc: Use devm for mutex_init
  platform/x86:intel/pmc: Move error handling to init function
  platform/x86:intel/pmc: Improve pmc_core_get_lpm_req()

 drivers/platform/x86/intel/pmc/Kconfig        |   4 +
 drivers/platform/x86/intel/pmc/Makefile       |   8 +-
 drivers/platform/x86/intel/pmc/core.c         | 189 +++++++++++-------
 drivers/platform/x86/intel/pmc/core.h         |   7 -
 .../platform/x86/intel/pmc/ssram_telemetry.c  | 147 +++++++++-----
 .../platform/x86/intel/pmc/ssram_telemetry.h  |  24 +++
 drivers/platform/x86/intel/vsec.c             |   9 +-
 include/linux/intel_vsec.h                    |   5 +-
 8 files changed, 260 insertions(+), 133 deletions(-)
 create mode 100644 drivers/platform/x86/intel/pmc/ssram_telemetry.h

On Fri, 25 Apr 2025 12:52:28 -0700, Xi Pardee wrote:

> This patch series removes the SSRAM support from Intel PMC Core driver
> and creates a separate PCI driver for SSRAM device. The new Intel PMC
> SSRAM driver provides the following functionalities:
> 
> 1. Search and store the PMC information in a structure, including PWRMBASE
> address and devid for each available PMC. Then Intel PMC Core driver
> achieves the PMC information using the API provided by the new driver.
> 2. Search and register Intel Platform Monitoring Techology telemetry
> regions so they would by available for read through sysfs and Intel PMT
> API. Intel PMC Core driver can achieve Low Power Mode requirement
> information from a telemetry region registered by the new driver.
> 
> [...]


Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/5] platform/x86:intel/vsec: Change return type of intel_vsec_register
      commit: f6347ba78ad48eef2a79d2063d77a32d015c58af
[2/5] platform/x86:intel/pmc: Create Intel PMC SSRAM Telemetry driver
      commit: f3d766cced899fa626e3f6f6ace43e16061ea19e
[3/5] platform/x86:intel/pmc: Use devm for mutex_init
      commit: 95350cbf5c4b7453d15990ad776b7b101fd6a290
[4/5] platform/x86:intel/pmc: Move error handling to init function
      commit: 75c7b75459efeb5b600d99421f3d08f485d33896
[5/5] platform/x86:intel/pmc: Improve pmc_core_get_lpm_req()
      commit: a946156129a6adb702dff43111c1d8cdab4f9362

--
 i.