diff --git a/src/soc/intel/common/block/pcie/rtd3/rtd3.c b/src/soc/intel/common/block/pcie/rtd3/rtd3.c index d61e853256..7d2efdb236 100644 --- a/src/soc/intel/common/block/pcie/rtd3/rtd3.c +++ b/src/soc/intel/common/block/pcie/rtd3/rtd3.c @@ -131,6 +131,8 @@ pcie_rtd3_acpi_method_on(unsigned int pcie_rp, acpigen_write_method_serialized("_ON", 0); + acpigen_write_debug_string("PCIe RTD3 _ON"); + /* The _STA returns current power status of device, so we can skip _ON * if _STA returns 1 * Example: @@ -147,7 +149,6 @@ pcie_rtd3_acpi_method_on(unsigned int pcie_rp, acpigen_write_return_op(ONE_OP); acpigen_write_if_end(); - /* When this feature is enabled, ONSK indicates if the previous _OFF was * skipped. If so, since the device was not in Off state, and the current * _ON can be skipped as well. @@ -201,6 +202,8 @@ pcie_rtd3_acpi_method_off(int pcie_rp, { acpigen_write_method_serialized("_OFF", 0); + acpigen_write_debug_string("PCIe RTD3 _OFF"); + /* When this feature is enabled, ONSK is checked to see if the device * wants _OFF to be skipped for once. ONSK is normally incremented in the * device method, such as reset _RST, which is invoked during driver reload. @@ -394,8 +397,8 @@ static void pcie_rtd3_acpi_fill_ssdt(const struct device *dev) } } - printk(BIOS_INFO, "%s: Enable RTD3 for %s (%s)\n", scope, dev_path(parent), - config->desc ?: dev->chip_ops->name); + printk(BIOS_INFO, "%s: Enable RTD3 for %s (%s) on RP #%d\n", scope, dev_path(parent), + config->desc ?: dev->chip_ops->name, pcie_rp + 1); /* Create a mutex for exclusive access to the PMC registers. */ if (rp_type == PCIE_RP_PCH && !mutex_created) {