Patrick Georgi (pgeorgi@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@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@chromium.org Original-Commit-Id: e8e3bcbf6249c80850a87dd66f34d3ff36158641 Original-Change-Id: I1415a3816cd2fa9dd05bcbd36ac0abc3f2759960 Original-Signed-off-by: Furquan Shaikh furquan@google.com Original-Reviewed-on: https://chromium-review.googlesource.com/286150 Original-Tested-by: Furquan Shaikh furquan@chromium.org Original-Reviewed-by: Julius Werner jwerner@chromium.org Original-Commit-Queue: Furquan Shaikh furquan@chromium.org Original-Trybot-Ready: Furquan Shaikh furquan@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