[coreboot-gerrit] Change in coreboot[master]: Enable Serial Debug for UEFI payload Debug build

[Wonkyu Kim (Code Review)]

Wonkyu Kim has uploaded this change for review. ( https://review.coreboot.org/29004


Change subject: Enable Serial Debug for UEFI payload Debug build
......................................................................

Enable Serial Debug for UEFI payload Debug build

Change-Id: I29953250ca278d5b1977be3763359df5110f67c7
Signed-off-by: Wonkyu Kim <wonkyu.kim at intel.com>
---
R payloads/external/tianocore/patches/05_CorebootPayloadPkg-don-t-use-serial-output-for-Relea.patch
1 file changed, 21 insertions(+), 14 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/29004/1

diff --git a/payloads/external/tianocore/patches/05_CorebootPayloadPkg_noserial.patch b/payloads/external/tianocore/patches/05_CorebootPayloadPkg-don-t-use-serial-output-for-Relea.patch
similarity index 60%
rename from payloads/external/tianocore/patches/05_CorebootPayloadPkg_noserial.patch
rename to payloads/external/tianocore/patches/05_CorebootPayloadPkg-don-t-use-serial-output-for-Relea.patch
index 0e84b7d..09ff6b0 100644
--- a/payloads/external/tianocore/patches/05_CorebootPayloadPkg_noserial.patch
+++ b/payloads/external/tianocore/patches/05_CorebootPayloadPkg-don-t-use-serial-output-for-Relea.patch
@@ -1,39 +1,46 @@
-From 77f75370d6fbfefb04456f6e1c32c85d4bac8bf5 Mon Sep 17 00:00:00 2001
-From: Matt Devo <matt.devillier at gmail.com>
-Date: Thu, 23 Feb 2017 14:11:14 -0600
-Subject: [PATCH] CorebootPayloadPkg: don't use serial output
+From 147174ea833e14ef68a8c5f1366cfbd1468dd1c8 Mon Sep 17 00:00:00 2001
+From: Wonkyu Kim <wonkyu.kim at intel.com>
+Date: Wed, 10 Oct 2018 10:59:51 -0700
+Subject: [PATCH] CorebootPayloadPkg: don't use serial output for Release build
 
+Signed-off-by: Wonkyu Kim <wonkyu.kim at intel.com>
 ---
- CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc    | 2 +-
- CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc | 2 +-
- 2 files changed, 2 insertions(+), 2 deletions(-)
+ CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc    | 4 ++++
+ CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc | 4 ++++
+ 2 files changed, 8 insertions(+)
 
 diff --git a/CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc b/CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc
-index 1d80fc4f5c..ce360c9e9e 100644
+index 6ddd64faf7a5..2e7055815add 100644
 --- a/CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc
 +++ b/CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc
-@@ -261,7 +261,7 @@
+@@ -261,7 +261,11 @@
  #
  ################################################################################
  [PcdsFeatureFlag]
--  gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRUE
++!if $(TARGET) == DEBUG
+   gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRUE
++!else
 +  gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|FALSE
++!endif
    gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseMemory|FALSE
    gEfiMdeModulePkgTokenSpaceGuid.PcdDxeIplSwitchToLongMode|FALSE
    gEfiMdeModulePkgTokenSpaceGuid.PcdConOutGopSupport|TRUE
 diff --git a/CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc b/CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc
-index 52b26eb3d0..a27b0873a1 100644
+index 19c203e6cbb6..b00b7b0e1134 100644
 --- a/CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc
 +++ b/CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc
-@@ -261,7 +261,7 @@
+@@ -261,7 +261,11 @@
  #
  ################################################################################
  [PcdsFeatureFlag]
--  gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRUE
++!if $(TARGET) == DEBUG
+   gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|TRUE
++!else
 +  gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseSerial|FALSE
++!endif
    gEfiMdeModulePkgTokenSpaceGuid.PcdStatusCodeUseMemory|FALSE
    gEfiMdeModulePkgTokenSpaceGuid.PcdDxeIplSwitchToLongMode|TRUE
    gEfiMdeModulePkgTokenSpaceGuid.PcdConOutGopSupport|TRUE
 -- 
-2.13.2.725.g09c95d1e9-goog
+2.17.1
 

-- 
To view, visit https://review.coreboot.org/29004
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I29953250ca278d5b1977be3763359df5110f67c7
Gerrit-Change-Number: 29004
Gerrit-PatchSet: 1
Gerrit-Owner: Wonkyu Kim <wonkyu.kim at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181010/bf819671/attachment-0001.html>