Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31556
Change subject: Documentation: Add myself as potential mentor for QEMU targets
......................................................................
Documentation: Add myself as potential mentor for QEMU targets
Change-Id: I11df0283f14ae03243247fe9377754b216df0442
Signed-off-by: Patrick Georgi <pgeorgi(a)google.com>
---
M Documentation/contributing/project_ideas.md
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/31556/1
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md
index 3bd8216..9a2d414 100644
--- a/Documentation/contributing/project_ideas.md
+++ b/Documentation/contributing/project_ideas.md
@@ -78,6 +78,7 @@
hardware is available.
### Mentors
+* Patrick Georgi <patrick(a)georgi.software>
## Add Kernel Address Sanitizer functionality to coreboot
The Kernel Address Sanitizer (KASAN) is a runtime dynamic memory error detector.
--
To view, visit https://review.coreboot.org/c/coreboot/+/31556
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I11df0283f14ae03243247fe9377754b216df0442
Gerrit-Change-Number: 31556
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-MessageType: newchange
Hello Aaron Durbin, Patrick Georgi,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/31773
to review the following change.
Change subject: Add new CONFIG(XXX) macro to replace IS_ENABLED(CONFIG_XXX)
......................................................................
Add new CONFIG(XXX) macro to replace IS_ENABLED(CONFIG_XXX)
The IS_ENABLED() macro is pretty long and unwieldy for something so
widely used, and often forces line breaks just for checking two Kconfigs
in a row. Let's replace it with something that takes up less space to
make our code more readable. From now on,
if (IS_ENABLED(CONFIG_XXX))
#if IS_ENABLED(CONFIG_XXX)
shall become
if (CONFIG(XXX))
#if CONFIG(XXX)
Change-Id: I2468427b569b974303084574125a9e1d9f6db596
Signed-off-by: Julius Werner <jwerner(a)chromium.org>
---
M payloads/libpayload/include/kconfig.h
M src/include/kconfig.h
2 files changed, 6 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/31773/1
diff --git a/payloads/libpayload/include/kconfig.h b/payloads/libpayload/include/kconfig.h
index adb3403..3a44529 100644
--- a/payloads/libpayload/include/kconfig.h
+++ b/payloads/libpayload/include/kconfig.h
@@ -17,5 +17,7 @@
#define __config_enabled(arg1_or_junk) ___config_enabled(arg1_or_junk 1, 0, 0)
#define ___config_enabled(__ignored, val, ...) val
-#define IS_ENABLED(option) config_enabled(option)
+#define IS_ENABLED(option) config_enabled(option) /* deprecated */
+#define CONFIG(option) config_enabled(CONFIG_#option)
+
#endif
diff --git a/src/include/kconfig.h b/src/include/kconfig.h
index 73106e9..0478548 100644
--- a/src/include/kconfig.h
+++ b/src/include/kconfig.h
@@ -17,5 +17,7 @@
#define __config_enabled(arg1_or_junk) ___config_enabled(arg1_or_junk 1, 0, 0)
#define ___config_enabled(__ignored, val, ...) val
-#define IS_ENABLED(option) config_enabled(option)
+#define IS_ENABLED(option) config_enabled(option) /* deprecated */
+#define CONFIG(option) config_enabled(CONFIG_##option)
+
#endif
--
To view, visit https://review.coreboot.org/c/coreboot/+/31773
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2468427b569b974303084574125a9e1d9f6db596
Gerrit-Change-Number: 31773
Gerrit-PatchSet: 1
Gerrit-Owner: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-MessageType: newchange