[PATCH v6 14/15] platform/x86/intel/pmc/ssram: Make PMT registration optional
From: David E. Box
Date: Sun May 31 2026 - 15:48:48 EST
The SSRAM telemetry driver extracts essential PMC device ID and power
management base address information that intel_pmc_core depends on for core
functionality. If PMT registration failure prevents this critical data from
being available, intel_pmc_core operation would break entirely. Therefore,
PMT registration failures must not block access to this data.
Change the behavior to log a warning when PMT registration fails but
continue with successful driver initialization, ensuring the primary
telemetry data remains accessible to dependent drivers.
Signed-off-by: David E. Box <david.e.box@xxxxxxxxxxxxxxx>
---
V6 - No changes
V5 - No changes
V4 - No changes
V3 changes:
- Dropped the standalone cleanup-pattern patch from this refreshed
series retaining the simpler ssram pointer flow requested in review.
- Folded PMT-registration-optional handling onto that simpler flow
with no intended functional change.
V2 changes:
- Update commit message for clarity
- Also apply the PCI telemetry path
drivers/platform/x86/intel/pmc/ssram_telemetry.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/platform/x86/intel/pmc/ssram_telemetry.c b/drivers/platform/x86/intel/pmc/ssram_telemetry.c
index 7ebfbb177499..b7cc4b540221 100644
--- a/drivers/platform/x86/intel/pmc/ssram_telemetry.c
+++ b/drivers/platform/x86/intel/pmc/ssram_telemetry.c
@@ -195,7 +195,7 @@ pmc_ssram_telemetry_get_pmc_pci(struct pci_dev *pcidev,
/* Find and register and PMC telemetry entries */
ret = pmc_ssram_telemetry_add_pmt(pcidev, ssram_base, ssram);
if (ret)
- return ret;
+ dev_warn(&pcidev->dev, "could not register PMT\n");
probe_cache->valid_mask |= BIT(pmc_idx);
@@ -288,7 +288,7 @@ static int pmc_ssram_telemetry_acpi_init(struct pci_dev *pcidev,
ret = intel_vsec_register(&pcidev->dev, &info);
if (ret)
- return ret;
+ dev_warn(&pcidev->dev, "could not register PMT\n");
return pmc_ssram_telemetry_get_pmc_acpi(pcidev, probe_cache, index);
}
--
2.43.0