[edk2-devel] [PATCH v2 1/1] UefiPayloadPkg: Remove x86 legacy UART defaults

2020-07-13 Thread Marcello Sylvester Bauer
From: Patrick Rudolph The BaseSerialPortLib16550 does fallback to a fixed address UART defined by PcdSerialRegisterBase and does not initialize if it is zero. Do not assume a serial port at 0x3F8, otherwise it could cause errors during initialisation of a non-existent serial port on non legacy pl

[edk2-devel] [PATCH v2 1/1] UefiPayloadPkg: Remove x86 legacy UART defaults

2020-07-13 Thread Marcello Sylvester Bauer
From: Patrick Rudolph The BaseSerialPortLib16550 does fallback to a fixed address UART defined by PcdSerialRegisterBase and does not initialize if it is zero. Do not assume a serial port at 0x3F8, otherwise it could cause errors during initialisation of a non-existent serial port on non legacy pl

Re: [edk2-devel] [PATCH v2 1/1] UefiPayloadPkg: Remove x86 legacy UART defaults

2020-07-13 Thread Ma, Maurice
As I mentioned in the previous comments, could you please CC all UefiPayloadPkg maintainers listed in the Maintainers.txt under the root of the EDK2 directory for code changes in UefiPayloadPkg ? Other than that, this patch looks good to me. Reviewed-by: Maurice Ma Thanks Maurice > -Ori

Re: [edk2-devel] [PATCH v2 1/1] UefiPayloadPkg: Remove x86 legacy UART defaults

2020-07-13 Thread Guo Dong
Zeng, Star > > Subject: [edk2-devel] [PATCH v2 1/1] UefiPayloadPkg: Remove x86 legacy UART > defaults > > From: Patrick Rudolph > > The BaseSerialPortLib16550 does fallback to a fixed address UART defined > by PcdSerialRegisterBase and does not initialize if it