[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/braswell/pmutil.c: Use GEN_PMCON1 for RTC status

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32724 )

Change subject: soc/intel/braswell/pmutil.c:  Use GEN_PMCON1 for RTC status
......................................................................

soc/intel/braswell/pmutil.c:  Use GEN_PMCON1 for RTC status

cbmem_find is not available in every stage.
Remove usage of cbmem_find() and use GEN_PMCON1 always.

BUG=NA
TEST=Booting Embedded Linux on Facebook FBG-1701

Change-Id: Id97d57864b3e241e8f046d9b1caebdce199a46b1
Signed-off-by: Frans Hendriks 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32724
Tested-by: build bot (Jenkins) 
Reviewed-by: Nico Huber 
---
M src/soc/intel/braswell/pmutil.c
1 file changed, 6 insertions(+), 7 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved

diff --git a/src/soc/intel/braswell/pmutil.c b/src/soc/intel/braswell/pmutil.c
index 271eefe..4bc621b 100644
--- a/src/soc/intel/braswell/pmutil.c
+++ b/src/soc/intel/braswell/pmutil.c
@@ -16,9 +16,9 @@

#include <arch/acpi.h>
 #include <arch/io.h>
+#include <assert.h>
 #include <device/mmio.h>
 #include <device/pci_ops.h>
-#include <cbmem.h>
 #include <console/console.h>
 #include <soc/iomap.h>
 #include <soc/lpc.h>
@@ -364,15 +364,14 @@
 {
    uint32_t gen_pmcon1;
    int rtc_fail;
-	struct chipset_power_state *ps = cbmem_find(CBMEM_ID_POWER_STATE);

-	if (ps != NULL)
-		gen_pmcon1 = ps->gen_pmcon1;
-	else
-		gen_pmcon1 = read32((u32 *)(PMC_BASE_ADDRESS + GEN_PMCON1));
+	/* not usable in ramstage as GEN_PMCON1 gets reset */
+	if (ENV_RAMSTAGE)
+		dead_code();
+
+	gen_pmcon1 = read32((u32 *)(PMC_BASE_ADDRESS + GEN_PMCON1));

rtc_fail = !!(gen_pmcon1 & RPS);
-
    if (rtc_fail)
    	printk(BIOS_DEBUG, "RTC failure.\n");