diff options
| author | Fabio Aiuto <fabioaiuto83@gmail.com> | 2022-09-11 12:25:13 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-09-27 14:19:01 +0000 |
| commit | fdcf698a89ebfe2316fc6ef2f94df7c8203a7f4b (patch) | |
| tree | e88b014b82bcf76b546c7c1010d208ee337bf494 /src/soc/amd/common/block | |
| parent | e6d6d3620e20f6eddf92f71e30a7e0c47d403489 (diff) | |
| download | coreboot-fdcf698a89ebfe2316fc6ef2f94df7c8203a7f4b.tar.gz coreboot-fdcf698a89ebfe2316fc6ef2f94df7c8203a7f4b.tar.bz2 coreboot-fdcf698a89ebfe2316fc6ef2f94df7c8203a7f4b.zip | |
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>
Diffstat (limited to 'src/soc/amd/common/block')
| -rw-r--r-- | src/soc/amd/common/block/acpi/acpi.c | 2 | ||||
| -rw-r--r-- | src/soc/amd/common/block/acpi/pm_state.c | 2 | ||||
| -rw-r--r-- | src/soc/amd/common/block/gpio/gpio.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/src/soc/amd/common/block/acpi/acpi.c b/src/soc/amd/common/block/acpi/acpi.c index 9e4010d34044..75d4adf8114e 100644 --- a/src/soc/amd/common/block/acpi/acpi.c +++ b/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; diff --git a/src/soc/amd/common/block/acpi/pm_state.c b/src/soc/amd/common/block/acpi/pm_state.c index 787b7323532d..ccfef0a7522d 100644 --- a/src/soc/amd/common/block/acpi/pm_state.c +++ b/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); diff --git a/src/soc/amd/common/block/gpio/gpio.c b/src/soc/amd/common/block/gpio/gpio.c index 468d7aef703b..e8203a1f1579 100644 --- a/src/soc/amd/common/block/gpio/gpio.c +++ b/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; |