[coreboot-gerrit] New patch to review for coreboot: arm64: Set LOG_LEVEL=0 for BL31 if coreboot does not use serial

[Patrick Georgi (pgeorgi@google.com)]

Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11014

-gerrit

commit d23415d015e8ba3d5ab95514a83a6e53ad0e993d
Author: Furquan Shaikh <furquan at google.com>
Date:   Thu Jul 16 12:25:01 2015 -0700

    arm64: Set LOG_LEVEL=0 for BL31 if coreboot does not use serial
    
    Even if DEBUG=0, BL31 puts NOTICE(..) messages on serial console. Set
    LOG_LEVEL=0 if coreboot does not use serial.
    
    BUG=None
    BRANCH=None
    TEST=Compiles successfully and no console output from bl31 for
    production images.
    
    Change-Id: Ie77bcac3e2a0d314545b6811327c413536c77fb9
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: e8e3bcbf6249c80850a87dd66f34d3ff36158641
    Original-Change-Id: I1415a3816cd2fa9dd05bcbd36ac0abc3f2759960
    Original-Signed-off-by: Furquan Shaikh <furquan at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/286150
    Original-Tested-by: Furquan Shaikh <furquan at chromium.org>
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Commit-Queue: Furquan Shaikh <furquan at chromium.org>
    Original-Trybot-Ready: Furquan Shaikh <furquan at chromium.org>
---
 src/arch/arm64/Makefile.inc | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/arch/arm64/Makefile.inc b/src/arch/arm64/Makefile.inc
index 4cf012a..e7ee84c 100644
--- a/src/arch/arm64/Makefile.inc
+++ b/src/arch/arm64/Makefile.inc
@@ -192,6 +192,9 @@ endif
 # Build ARM TF in debug mode (with serial output) if coreboot uses serial
 ifeq ($(CONFIG_CONSOLE_SERIAL),y)
 BL31_MAKEARGS += DEBUG=1
+else
+# Turn off NOTICE messages from BL31 if coreboot does not use serial
+BL31_MAKEARGS += LOG_LEVEL=0
 endif # CONFIG_CONSOLE_SERIAL
 
 # Avoid build/release|build/debug distinction by overriding BUILD_PLAT directly