acpi/acpi_pm.c: refactor acpi_pm_state_for_* functions

Use just one function to get the chipset powerstate and add an argument
to specify the powerstate claimer {RTC,ELOG,WAKE} and adjust the
failure log accordingly.

TEST: compile tested and qemu emulation successfully run

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Change-Id: I8addc0b05f9e360afc52091c4bb731341d7213cf
Reviewed-on: https://review.coreboot.org/c/coreboot/+/67618
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>

src/acpi/acpi_pm.c

@@ -2,10 +2,17 @@
 
 #include <acpi/acpi.h>
 #include <acpi/acpi_pm.h>
+#include <assert.h>
 #include <cbmem.h>
 #include <console/console.h>
 #include <smbios.h>
 
+static const char *pm_fetch_failure_msg[PS_CLAIMER_MAX] = {
+	[PS_CLAIMER_ELOG]	= "no event recorded in ELOG.",
+	[PS_CLAIMER_RTC]	= "RTC init aborted.",
+	[PS_CLAIMER_WAKE]	= "wake source unknown.",
+};
+
 void __weak mainboard_suspend_resume(void)
 {
 }
@@ -38,31 +45,16 @@ struct chipset_power_state *acpi_get_pm_state(void)
 	return acpi_pm_state;
 }
 
-int acpi_pm_state_for_elog(const struct chipset_power_state **ps)
+int acpi_fetch_pm_state(const struct chipset_power_state **ps,
+			enum power_state_claimer ps_claimer)
 {
+	assert(ps_claimer < PS_CLAIMER_MAX);
 	*ps = acpi_get_pm_state();
-	if (!*ps) {
-		printk(BIOS_ERR, "No CBMEM_ID_POWER_STATE entry, no event recorded in ELOG.\n");
-		return -1;
-	}
-	return 0;
-}
 
-int acpi_pm_state_for_rtc(const struct chipset_power_state **ps)
-{
-	*ps = acpi_get_pm_state();
 	if (!*ps) {
-		printk(BIOS_ERR, "No CBMEM_ID_POWER_STATE entry, RTC init aborted.\n");
-		return -1;
-	}
-	return 0;
-}
-
-int acpi_pm_state_for_wake(const struct chipset_power_state **ps)
-{
-	*ps = acpi_get_pm_state();
-	if (!*ps) {
-		printk(BIOS_ERR, "No CBMEM_ID_POWER_STATE entry, wake source unknown.\n");
+		printk(BIOS_ERR, "No CBMEM_ID_POWER_STATE entry, %s\n",
+		       ps_claimer < PS_CLAIMER_MAX ?
+		       pm_fetch_failure_msg[ps_claimer] : "unknown claimer.");
 		return -1;
 	}
 	return 0;

src/include/acpi/acpi_pm.h

@@ -3,10 +3,16 @@
 #ifndef ACPI_PM_H
 #define ACPI_PM_H
 
+enum power_state_claimer {
+	PS_CLAIMER_ELOG,
+	PS_CLAIMER_RTC,
+	PS_CLAIMER_WAKE,
+	PS_CLAIMER_MAX /* Not a valid value, needs to be last element */
+};
+
 struct chipset_power_state;
 struct chipset_power_state *acpi_get_pm_state(void);
-int acpi_pm_state_for_elog(const struct chipset_power_state **ps);
-int acpi_pm_state_for_rtc(const struct chipset_power_state **ps);
-int acpi_pm_state_for_wake(const struct chipset_power_state **ps);
+int acpi_fetch_pm_state(const struct chipset_power_state **ps,
+		  enum power_state_claimer ps_claimer);
 
 #endif

src/soc/amd/common/block/acpi/acpi.c

@@ -79,7 +79,7 @@ void acpi_pm_gpe_add_events_print_events(void)
 	const struct chipset_power_state *ps;
 	const struct acpi_pm_gpe_state *state;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 
 	state = &ps->gpe_state;

src/soc/amd/common/block/acpi/pm_state.c

@@ -50,7 +50,7 @@ static void acpi_save_wake_source(void *unused)
 
 	if (acpi_reset_gnvs_for_wake(&gnvs) < 0)
 		return;
-	if (acpi_pm_state_for_wake(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_WAKE) < 0)
 		return;
 
 	pm_fill_gnvs(gnvs, &ps->gpe_state);

src/soc/amd/common/block/gpio/gpio.c

@@ -386,7 +386,7 @@ void gpio_add_events(void)
 	unsigned int i;
 	unsigned int end;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 	state = &ps->gpio_state;
 

src/soc/intel/alderlake/pmutil.c

@@ -175,7 +175,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return rtc_failed(ps->gen_pmcon_b);

src/soc/intel/apollolake/elog.c

@@ -89,7 +89,7 @@ void pch_log_state(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 
 	/* Power and Reset */

src/soc/intel/apollolake/pmutil.c

@@ -191,7 +191,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return rtc_failed(ps->gen_pmcon1);

src/soc/intel/baytrail/elog.c

@@ -77,7 +77,7 @@ void southcluster_log_state(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 
 	log_power_and_resets(ps);

src/soc/intel/baytrail/ramstage.c

@@ -147,7 +147,7 @@ static void acpi_save_wake_source(void *unused)
 
 	if (acpi_reset_gnvs_for_wake(&gnvs) < 0)
 		return;
-	if (acpi_pm_state_for_wake(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_WAKE) < 0)
 		return;
 
 	pm_fill_gnvs(gnvs, ps);

src/soc/intel/braswell/elog.c

@@ -77,7 +77,7 @@ void southcluster_log_state(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 
 	log_power_and_resets(ps);

src/soc/intel/broadwell/pch/elog.c

@@ -107,7 +107,7 @@ static void pch_log_state(void *unused)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 
 	/* Power and Reset */

src/soc/intel/broadwell/pch/ramstage.c

@@ -51,7 +51,7 @@ static void acpi_save_wake_source(void *unused)
 
 	if (acpi_reset_gnvs_for_wake(&gnvs) < 0)
 		return;
-	if (acpi_pm_state_for_wake(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_WAKE) < 0)
 		return;
 
 	pm_fill_gnvs(gnvs, ps);

src/soc/intel/cannonlake/pmutil.c

@@ -168,7 +168,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return rtc_failed(ps->gen_pmcon_b);

src/soc/intel/common/block/acpi/acpi_wake_source.c

@@ -54,7 +54,7 @@ static void acpi_save_wake_source(void *unused)
 
 	if (acpi_reset_gnvs_for_wake(&gnvs) < 0)
 		return;
-	if (acpi_pm_state_for_wake(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_WAKE) < 0)
 		return;
 
 	pm_fill_gnvs(gnvs, ps);

src/soc/intel/elkhartlake/pmutil.c

@@ -184,7 +184,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return check_rtc_failed(ps->gen_pmcon_b);

src/soc/intel/icelake/pmutil.c

@@ -184,7 +184,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return check_rtc_failed(ps->gen_pmcon_b);

src/soc/intel/jasperlake/pmutil.c

@@ -184,7 +184,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return check_rtc_failed(ps->gen_pmcon_b);

src/soc/intel/meteorlake/pmutil.c

@@ -166,7 +166,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return rtc_failed(ps->gen_pmcon_b);

src/soc/intel/skylake/elog.c

@@ -232,7 +232,7 @@ static void pch_log_state(void *unused)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_elog(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_ELOG) < 0)
 		return;
 
 	/* Power and Reset */

src/soc/intel/tigerlake/pmutil.c

@@ -190,7 +190,7 @@ int soc_get_rtc_failed(void)
 {
 	const struct chipset_power_state *ps;
 
-	if (acpi_pm_state_for_rtc(&ps) < 0)
+	if (acpi_fetch_pm_state(&ps, PS_CLAIMER_RTC) < 0)
 		return 1;
 
 	return check_rtc_failed(ps->gen_pmcon_b);