[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: add GPE event logs

[Aaron Durbin (Code Review)]

Aaron Durbin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/44489 )

Change subject: soc/amd/common: add GPE event logs
......................................................................

soc/amd/common: add GPE event logs

GPE events were not be recorded in the eventlog. Add those
to the eventlog when the status register indicates those events.

BUG=b:159947207

Signed-off-by: Aaron Durbin 
Change-Id: Ifb3167fd24f2171b2baf1a65eb81a318eb3e7a86
Reviewed-on: https://review.coreboot.org/c/coreboot/+/44489
Tested-by: build bot (Jenkins) 
Reviewed-by: Furquan Shaikh 
---
M src/soc/amd/common/block/acpi/acpi.c
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Furquan Shaikh: Looks good to me, approved

diff --git a/src/soc/amd/common/block/acpi/acpi.c b/src/soc/amd/common/block/acpi/acpi.c
index 17c0ed1..2d75ec0 100644
--- a/src/soc/amd/common/block/acpi/acpi.c
+++ b/src/soc/amd/common/block/acpi/acpi.c
@@ -83,6 +83,17 @@
    	elog_add_event_wake(ELOG_WAKE_SOURCE_PCIE, 0);
 }

+static void log_gpe_events(const struct acpi_pm_gpe_state *state)
+{
+	int i;
+	uint32_t valid_gpe = state->gpe0_sts & state->gpe0_en;
+
+	for (i = 0; i <= 31; i++) {
+		if (valid_gpe & (1U << i))
+			elog_add_event_wake(ELOG_WAKE_SOURCE_GPIO, i);
+	}
+}
+
 void acpi_fill_pm_gpe_state(struct acpi_pm_gpe_state *state)
 {
    state->pm1_sts = acpi_read16(MMIO_ACPI_PM1_STS);
@@ -97,6 +108,7 @@
 {
    log_pm1_status(state->pm1_sts);
    print_pm1_status(state->pm1_sts);
+	log_gpe_events(state);
 }

void acpi_clear_pm_gpe_status(void)