Change in coreboot[master]: pc80/rtc/mc146818rtc: {read,save}_vbnv_cmos only if VBOOT is set

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37785 )

Change subject: pc80/rtc/mc146818rtc: {read,save}_vbnv_cmos only if VBOOT is set
......................................................................

pc80/rtc/mc146818rtc: {read,save}_vbnv_cmos only if VBOOT is set

Change-Id: Ib33e70efbe57b6ff57e132040bb3a6d08be7d50c
Signed-off-by: Elyes HAOUAS 
---
M src/drivers/pc80/rtc/mc146818rtc.c
1 file changed, 5 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/85/37785/1

diff --git a/src/drivers/pc80/rtc/mc146818rtc.c b/src/drivers/pc80/rtc/mc146818rtc.c
index d3efdec..29f498b 100644
--- a/src/drivers/pc80/rtc/mc146818rtc.c
+++ b/src/drivers/pc80/rtc/mc146818rtc.c
@@ -173,10 +173,12 @@
       clear vbnv data for other internal reasons. For that, always back up
       the vbnv contents and conditionally save them when __cmos_init()
       indicates cmos was cleared. */
-	read_vbnv_cmos(vbnv);
+	if (CONFIG(VBOOT)) {
+		read_vbnv_cmos(vbnv);

-	if (__cmos_init(invalid))
-		save_vbnv_cmos(vbnv);
+		if (__cmos_init(invalid))
+			save_vbnv_cmos(vbnv);
+	}
 }

void cmos_init(bool invalid)

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/37785 )

Change subject: pc80/rtc/mc146818rtc: {read,save}_vbnv_cmos only if VBOOT is set
......................................................................

Abandoned