[coreboot-gerrit] Change in coreboot[master]: src/vendorcode: add IS_ENABLED() around Kconfig symbol refer...

Martin Roth (Code Review) gerrit at coreboot.org
Sun Jun 25 21:07:56 CEST 2017


Martin Roth has uploaded this change for review. ( https://review.coreboot.org/20357


Change subject: src/vendorcode: add IS_ENABLED() around Kconfig symbol references
......................................................................

src/vendorcode: add IS_ENABLED() around Kconfig symbol references

Change-Id: I891cb4f799aaafcf4a0dd91b5533d2f8db7f3d61
Signed-off-by: Martin Roth <martinroth at google.com>
---
M src/vendorcode/amd/agesa/f10/Include/Ids.h
M src/vendorcode/google/chromeos/chromeos.h
M src/vendorcode/google/chromeos/ramoops.c
3 files changed, 5 insertions(+), 5 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/20357/1

diff --git a/src/vendorcode/amd/agesa/f10/Include/Ids.h b/src/vendorcode/amd/agesa/f10/Include/Ids.h
index 2607b3d..df3ddc0 100644
--- a/src/vendorcode/amd/agesa/f10/Include/Ids.h
+++ b/src/vendorcode/amd/agesa/f10/Include/Ids.h
@@ -400,7 +400,7 @@
     #define IDS_HDT_CONSOLE_INIT(x)
     #define IDS_HDT_CONSOLE_EXIT(x)
     #ifdef __GNUC__
-      #if CONFIG_REDIRECT_IDS_HDT_CONSOLE_TO_SERIAL
+      #if IS_ENABLED(CONFIG_REDIRECT_IDS_HDT_CONSOLE_TO_SERIAL)
         #define IDS_HDT_CONSOLE(s, args...) do_printk(BIOS_DEBUG, s, ##args)
         ///#define IDS_HDT_CONSOLE(f, s, args...) printk(((MEM_FLOW) - (f) + (BIOS_DEBUG)), s, ##args)
       #else
diff --git a/src/vendorcode/google/chromeos/chromeos.h b/src/vendorcode/google/chromeos/chromeos.h
index e535751..a739ab7 100644
--- a/src/vendorcode/google/chromeos/chromeos.h
+++ b/src/vendorcode/google/chromeos/chromeos.h
@@ -24,7 +24,7 @@
 #include <vboot/misc.h>
 #include <vboot/vboot_common.h>
 
-#if CONFIG_CHROMEOS
+#if IS_ENABLED(CONFIG_CHROMEOS)
 /* functions implemented in watchdog.c */
 void mark_watchdog_tombstone(void);
 void reboot_from_watchdog(void);
@@ -38,9 +38,9 @@
 #include "gnvs.h"
 struct device;
 
-#if CONFIG_CHROMEOS_RAMOOPS
+#if IS_ENABLED(CONFIG_CHROMEOS_RAMOOPS)
 void chromeos_ram_oops_init(chromeos_acpi_t *chromeos);
-#if CONFIG_CHROMEOS_RAMOOPS_DYNAMIC
+#if IS_ENABLED(CONFIG_CHROMEOS_RAMOOPS_DYNAMIC)
 static inline void chromeos_reserve_ram_oops(struct device *dev, int idx) {}
 #else /* CONFIG_CHROMEOS_RAMOOPS_DYNAMIC */
 void chromeos_reserve_ram_oops(struct device *dev, int idx);
diff --git a/src/vendorcode/google/chromeos/ramoops.c b/src/vendorcode/google/chromeos/ramoops.c
index 313025d..c72af00 100644
--- a/src/vendorcode/google/chromeos/ramoops.c
+++ b/src/vendorcode/google/chromeos/ramoops.c
@@ -50,7 +50,7 @@
 	set_ramoops(chromeos, ram_oops, size);
 }
 
-#if CONFIG_CHROMEOS_RAMOOPS_DYNAMIC
+#if IS_ENABLED(CONFIG_CHROMEOS_RAMOOPS_DYNAMIC)
 static inline void set_global_chromeos_pointer(chromeos_acpi_t *chromeos) {}
 #else /* !CONFIG_CHROMEOS_RAMOOPS_DYNAMIC */
 

-- 
To view, visit https://review.coreboot.org/20357
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I891cb4f799aaafcf4a0dd91b5533d2f8db7f3d61
Gerrit-Change-Number: 20357
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170625/145b0abc/attachment-0001.html>


More information about the coreboot-gerrit mailing list