Reviewed-by: Kelly Steele <kelly.ste...@intel.com> > -----Original Message----- > From: Leahy, Leroy P > Sent: March 04, 2016 14:31 > To: Kinney, Michael D <michael.d.kin...@intel.com>; Steele, Kelly > <kelly.ste...@intel.com>; edk2-devel@lists.01.org; Bjorge, Erik C > <erik.c.bjo...@intel.com> > Cc: Leahy, Leroy P <leroy.p.le...@intel.com> > Subject: [PATCH v2 8/9] QuarkPkg: Move baud rate setting to top of .dsc file > > Move the baud rate setting to the top of the .dsc file. Use a single > setting for each board. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Lee Leahy <leroy.p.le...@intel.com> > --- > QuarkPlatformPkg/Quark.dsc | 25 ++++++++++++------------- > QuarkPlatformPkg/QuarkMin.dsc | 25 ++++++++++++------------- > 2 files changed, 24 insertions(+), 26 deletions(-) > > diff --git a/QuarkPlatformPkg/Quark.dsc b/QuarkPlatformPkg/Quark.dsc > index c87bb17..2b8038e 100644 > --- a/QuarkPlatformPkg/Quark.dsc > +++ b/QuarkPlatformPkg/Quark.dsc > @@ -46,6 +46,16 @@ > DEFINE GALILEO = GEN2 > > # > + # Specify the maximum baud rate for the board > + # > +!if $(GALILEO) == GEN1 > + DEFINE BAUD_RATE = 460800 > +!endif > +!if $(GALILEO) == GEN2 > + DEFINE BAUD_RATE = 921600 > +!endif > + > + # > # TPM 1.2 Hardware. Options are [NONE, LPC, ATMEL_I2C, INFINEON_I2C] > # > DEFINE TPM_12_HARDWARE = NONE > @@ -342,12 +352,7 @@ > !endif > gEfiMdePkgTokenSpaceGuid.PcdPostCodePropertyMask|0x18 > gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress|0xE0000000 > -!if $(GALILEO) == GEN1 > - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate|460800 > -!endif > -!if $(GALILEO) == GEN2 > - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate|921600 > -!endif > + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate|$(BAUD_RATE) > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultDataBits|8 > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity|1 > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits|1 > @@ -382,14 +387,8 @@ > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseMmio|TRUE > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0x9000B000 > -!if $(GALILEO) == GEN1 > - gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|460800 > + gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|$(BAUD_RATE) > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseHardwareFlowControl|FALSE > -!endif > -!if $(GALILEO) == GEN2 > - gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|921600 > - > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseHardwareFlowControl|FALSE > -!endif > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialLineControl|0x03 > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialFifoControl|0x07 > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialDetectCable|FALSE > diff --git a/QuarkPlatformPkg/QuarkMin.dsc b/QuarkPlatformPkg/QuarkMin.dsc > index f8a656e..7d89767 100644 > --- a/QuarkPlatformPkg/QuarkMin.dsc > +++ b/QuarkPlatformPkg/QuarkMin.dsc > @@ -40,6 +40,16 @@ > DEFINE PERFORMANCE_ENABLE = FALSE > DEFINE LOGGING = FALSE > > + # > + # Specify the maximum baud rate for the board > + # > +!if $(GALILEO) == GEN1 > + DEFINE BAUD_RATE = 460800 > +!endif > +!if $(GALILEO) == GEN2 > + DEFINE BAUD_RATE = 921600 > +!endif > + > !if $(TARGET) == "DEBUG" > DEFINE LOGGING = TRUE > !endif > @@ -311,12 +321,7 @@ > !endif > gEfiMdePkgTokenSpaceGuid.PcdPostCodePropertyMask|0x18 > gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress|0xE0000000 > -!if $(GALILEO) == GEN1 > - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate|460800 > -!endif > -!if $(GALILEO) == GEN2 > - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate|921600 > -!endif > + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate|$(BAUD_RATE) > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultDataBits|8 > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity|1 > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits|1 > @@ -351,14 +356,8 @@ > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseMmio|TRUE > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0x9000B000 > -!if $(GALILEO) == GEN1 > - gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|460800 > + gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|$(BAUD_RATE) > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseHardwareFlowControl|FALSE > -!endif > -!if $(GALILEO) == GEN2 > - gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|921600 > - > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseHardwareFlowControl|FALSE > -!endif > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialLineControl|0x03 > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialFifoControl|0x07 > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialDetectCable|FALSE > -- > 1.9.1
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel