drivers/elog: Add elog_boot_notify()

Change-Id: I898188d31fcfd153eb95d0a7324fa9fd85316e3c
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35373
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>

src/drivers/intel/fsp1_1/romstage.c

@@ -48,8 +48,7 @@ static void raminit_common(struct romstage_params *params)
 
 	s3wake = params->power_state->prev_sleep_state == ACPI_S3;
 
-	if (CONFIG(ELOG_BOOT_COUNT) && !s3wake)
+	elog_boot_notify(s3wake);
-		boot_count_increment();
 
 	/* Perform remaining SOC initialization */
 	soc_pre_ram_init(params);

src/drivers/intel/fsp2_0/memory_init.c

@@ -392,8 +392,7 @@ void fsp_memory_init(bool s3wake)
 	struct memranges memmap;
 	struct range_entry freeranges[2];
 
-	if (CONFIG(ELOG_BOOT_COUNT) && !s3wake)
+	elog_boot_notify(s3wake);
-		boot_count_increment();
 
 	if (cbfs_boot_locate(&file_desc, name, NULL)) {
 		printk(BIOS_CRIT, "Could not locate %s in CBFS\n", name);

src/include/elog.h

@@ -274,6 +274,12 @@ static inline u32 boot_count_read(void)
 #endif
 u32 boot_count_increment(void);
 
+static inline void elog_boot_notify(int s3_resume)
+{
+	if (CONFIG(ELOG_BOOT_COUNT) && !s3_resume)
+		boot_count_increment();
+}
+
 /*
  * Callback from GSMI handler to allow platform to log any wake source
  * information.

src/northbridge/intel/nehalem/early_init.c

@@ -139,8 +139,7 @@ void nehalem_early_initialization(int chipset_type)
 	s3_resume = (inw(DEFAULT_PMBASE + PM1_STS) & WAK_STS) &&
 	    (((inl(DEFAULT_PMBASE + PM1_CNT) >> 10) & 7) == SLP_TYP_S3);
 
-	if (CONFIG(ELOG_BOOT_COUNT) && !s3_resume)
+	elog_boot_notify(s3_resume);
-		boot_count_increment();
 
 	/* Device Enable */
 	pci_write_config32(PCI_DEV(0, 0, 0), D0F0_DEVEN,

src/northbridge/intel/sandybridge/romstage.c

@@ -74,8 +74,7 @@ void mainboard_romstage_entry(void)
 
 	s3resume = southbridge_detect_s3_resume();
 
-	if (CONFIG(ELOG_BOOT_COUNT) && !s3resume)
+	elog_boot_notify(s3resume);
-		boot_count_increment();
 
 	post_code(0x38);
 

src/soc/amd/picasso/romstage.c

@@ -49,12 +49,10 @@ asmlinkage void car_stage_entry(void)
 	console_init();
 
 	mainboard_romstage_entry_s3(s3_resume);
+	elog_boot_notify(s3_resume);
 
 	if (!s3_resume) {
 		post_code(0x40);
-
-		if (CONFIG(ELOG_BOOT_COUNT))
-			boot_count_increment();
 	} else {
 		printk(BIOS_INFO, "S3 detected\n");
 		post_code(0x60);

src/soc/amd/stoneyridge/romstage.c

@@ -97,6 +97,7 @@ asmlinkage void car_stage_entry(void)
 		load_smu_fw1();
 
 	mainboard_romstage_entry_s3(s3_resume);
+	elog_boot_notify(s3_resume);
 
 	bsp_agesa_call();
 
@@ -132,8 +133,6 @@ asmlinkage void car_stage_entry(void)
 		msr_t sys_cfg = rdmsr(SYSCFG_MSR);
 		sys_cfg.lo &= ~SYSCFG_MSR_TOM2WB;
 		wrmsr(SYSCFG_MSR, sys_cfg);
-		if (CONFIG(ELOG_BOOT_COUNT))
-			boot_count_increment();
 	} else {
 		printk(BIOS_INFO, "S3 detected\n");
 		post_code(0x60);

src/soc/intel/baytrail/romstage/romstage.c

@@ -185,10 +185,7 @@ void mainboard_romstage_entry(void)
 
 	printk(BIOS_DEBUG, "prev_sleep_state = S%d\n", prev_sleep_state);
 
-#if CONFIG(ELOG_BOOT_COUNT)
+	elog_boot_notify(prev_sleep_state == ACPI_S3);
-	if (prev_sleep_state != ACPI_S3)
-		boot_count_increment();
-#endif
 
 	/* Initialize RAM */
 	raminit(&mp, prev_sleep_state);

src/soc/intel/broadwell/romstage/romstage.c

@@ -62,6 +62,8 @@ void mainboard_romstage_entry(void)
 	/* Get power state */
 	rp.power_state = fill_power_state();
 
+	elog_boot_notify(rp.power_state->prev_sleep_state == ACPI_S3);
+
 	/* Print useful platform information */
 	report_platform_info();
 
@@ -80,10 +82,6 @@ void mainboard_romstage_entry(void)
 
 	rp.pei_data.boot_mode = rp.power_state->prev_sleep_state;
 
-	if (CONFIG(ELOG_BOOT_COUNT)
-			&& rp.power_state->prev_sleep_state != ACPI_S3)
-		boot_count_increment();
-
 	/* Print ME state before MRC */
 	intel_me_status();
 

src/southbridge/intel/lynxpoint/early_pch.c

@@ -124,10 +124,7 @@ int early_pch_init(const void *gpio_map,
 
 	wake_from_s3 = southbridge_detect_s3_resume();
 
-#if CONFIG(ELOG_BOOT_COUNT)
+	elog_boot_notify(wake_from_s3);
-	if (!wake_from_s3)
-		boot_count_increment();
-#endif
 
 	/* Report if we are waking from s3. */
 	return wake_from_s3;