From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web11.1150.1604537697969104127 for ; Wed, 04 Nov 2020 16:54:59 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Thu, 05 Nov 2020 08:54:54 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , Cc: "'Eric Dong'" , "'Chasel Chiu'" , "'Nate DeSimone'" References: <20201104080544.515-1-heng.luo@intel.com> In-Reply-To: <20201104080544.515-1-heng.luo@intel.com> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BhdGNoIFY0XSBNaW5QbGF0Zm9ybVBrZzogY29uc29sZSByZWRpcmVjdCBhZnRlciB0aGUgc2hlbGwgaXMgbG9hZGVk?= Date: Thu, 5 Nov 2020 08:54:55 +0800 Message-ID: <001701d6b30e$46e25380$d4a6fa80$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQEd+XYAKpYg6UWjIi1sxHSaCEvBNqsqKWtg Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Heng: Can you separate the patch for the different packages? If so, it will = be easily to be reviewed.=20 Thanks Liming > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: bounce+27952+66973+4905953+8761045@groups.io > =B4=FA=B1=ED Heng Luo > =B7=A2=CB=CD=CA=B1=BC=E4: 2020=C4=EA11=D4=C24=C8=D5 16:06 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io > =B3=AD=CB=CD: Eric Dong ; Chasel Chiu > ; Nate DeSimone = > =D6=F7=CC=E2: [edk2-devel] [Patch V4] MinPlatformPkg: console redirect = after the > shell is loaded >=20 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3014 >=20 > Use SerialPortTerminalLib to get console redirect after the shell is > loaded: > 1. Add SerialPortTerminalLib to MinPlatformPkg/Library, add PCDs to > configure serial port. > 2. Add SerialPortTerminalLib to BdsDxe driver, to add the serial = device to > ConIn and ConOut variables > 3. Include SerialDxe and TerminalDxe to CoreDxeInclude.dsc and > CoreUefiBootInclude.fdf. > 4. Use PcdSerialTerminalEnable to enable/disable this feature. > 5. Enable feature for UpXtreme, and disable it for other platform by default. >=20 > Cc: Eric Dong > Cc: Chasel Chiu > Cc: Nate DeSimone > Signed-off-by: Heng Luo > --- >=20 > Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkgPcd > .dsc | 1 + > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > | 1 + >=20 > Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.dsc > | 1 + > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc > | 15 +++++++++++++-- > Platform/Intel/MinPlatformPkg/Include/Fdf/CoreUefiBootInclude.fdf > | 8 +++++++- >=20 > Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortTer= mi > nalLib.c | 102 > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > ++++++++++++++++++++++++++++++++++++++++ >=20 > Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortTer= mi > nalLib.h | 48 > ++++++++++++++++++++++++++++++++++++++++++++++++ >=20 > Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortTer= mi > nalLib.inf | 40 ++++++++++++++++++++++++++++++++++++++++ > Platform/Intel/MinPlatformPkg/MinPlatformPkg.dec > | 25 +++++++++++++++++++++++++ > Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc > | 3 ++- > Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkgPcd.dsc > | 1 + >=20 > Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.dsc > | 1 + >=20 > Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardPkg > Pcd.dsc | 1 + > 13 files changed, 243 insertions(+), 4 deletions(-) >=20 > diff --git > a/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkgP > cd.dsc > b/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkgP > cd.dsc > index d611e99fc1..589b002d06 100644 > --- > a/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkgP > cd.dsc > +++ > b/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkgP > cd.dsc > @@ -131,6 +131,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|FALSE >=20 >=20 >=20 > !if gMinPlatformPkgTokenSpaceGuid.PcdBootStage >=3D 1 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit|TRUE >=20 > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > index 352416a489..44dacdf082 100644 > --- > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > @@ -130,6 +130,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|FALSE >=20 >=20 >=20 > !if gMinPlatformPkgTokenSpaceGuid.PcdBootStage >=3D 1 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit|TRUE >=20 > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.d > sc > b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.d > sc > index 9e8b148753..725596cbf7 100644 > --- > a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.d > sc > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.d > sc > @@ -168,6 +168,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|FALSE >=20 >=20 >=20 > !if gMinPlatformPkgTokenSpaceGuid.PcdBootStage >=3D 1 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit|TRUE >=20 > diff --git = a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc > b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc > index f0e578f8cc..2a4a574cdf 100644 > --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc > +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc > @@ -1,7 +1,7 @@ > ## @file >=20 > # Platform description. >=20 > # >=20 > -# Copyright (c) 2017 - 2019, Intel Corporation. All rights = reserved.
>=20 > +# Copyright (c) 2017 - 2020, Intel Corporation. All rights = reserved.
>=20 > # >=20 > # SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > # >=20 > @@ -46,7 +46,18 @@ >=20 >=20 >=20 > MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/MonotonicCounte > rRuntimeDxe.inf >=20 >=20 >=20 > - MdeModulePkg/Universal/BdsDxe/BdsDxe.inf >=20 > + MdeModulePkg/Universal/BdsDxe/BdsDxe.inf{ >=20 > + >=20 > +!if gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable =3D=3D TRUE >=20 > + > = NULL|SerialTerminalFeaturePkg/Library/SerialPortTerminalLib/SerialPortTer= m > inalLib.inf >=20 > +!endif >=20 > + } >=20 > + >=20 > +!if gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable =3D=3D TRUE >=20 > + MdeModulePkg/Universal/SerialDxe/SerialDxe.inf >=20 > + MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf >=20 > +!endif >=20 > + >=20 >=20 > MdeModulePkg/Universal/DriverHealthManagerDxe/DriverHealthManagerD > xe.inf >=20 > MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf { >=20 > >=20 > diff --git > a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreUefiBootInclude.fdf > b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreUefiBootInclude.fdf > index 7859c0b1a5..ef4576eedf 100644 > --- = a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreUefiBootInclude.fdf > +++ = b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreUefiBootInclude.fdf > @@ -1,7 +1,7 @@ > ## @file >=20 > # FDF file of Platform. >=20 > # >=20 > -# Copyright (c) 2017 - 2019, Intel Corporation. All rights = reserved.
>=20 > +# Copyright (c) 2017 - 2020, Intel Corporation. All rights = reserved.
>=20 > # >=20 > # SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > # >=20 > @@ -26,6 +26,12 @@ INF > MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf > INF > MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/MonotonicCounte > rRuntimeDxe.inf >=20 >=20 >=20 > INF MdeModulePkg/Universal/BdsDxe/BdsDxe.inf >=20 > + >=20 > +!if gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable =3D=3D TRUE >=20 > +INF MdeModulePkg/Universal/SerialDxe/SerialDxe.inf >=20 > +INF MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf >=20 > +!endif >=20 > + >=20 > INF > MdeModulePkg/Universal/DriverHealthManagerDxe/DriverHealthManagerD > xe.inf >=20 > INF MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf >=20 > INF > MdeModulePkg/Universal/CapsuleRuntimeDxe/CapsuleRuntimeDxe.inf >=20 > diff --git > a/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.c > b/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.c > new file mode 100644 > index 0000000000..94de475173 > --- /dev/null > +++ > b/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.c > @@ -0,0 +1,102 @@ > +/** @file >=20 > + Main file for NULL named library for Serial Port Terminal = Redirection > library. >=20 > + >=20 > + Copyright (c) 2020, Intel Corporation. All rights reserved.
>=20 > + SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > + >=20 > +**/ >=20 > + >=20 > +#include "SerialPortTerminalLib.h" >=20 > + >=20 > +GLOBAL_REMOVE_IF_UNREFERENCED SERIAL_DEVICE_PATH > mSerialDevicePath =3D { >=20 > + { >=20 > + { >=20 > + HARDWARE_DEVICE_PATH, >=20 > + HW_VENDOR_DP, >=20 > + { >=20 > + (UINT8) sizeof (VENDOR_DEVICE_PATH), >=20 > + (UINT8) ((sizeof (VENDOR_DEVICE_PATH)) >> 8) >=20 > + } >=20 > + }, >=20 > + EDKII_SERIAL_PORT_LIB_VENDOR_GUID >=20 > + }, >=20 > + { >=20 > + { >=20 > + MESSAGING_DEVICE_PATH, >=20 > + MSG_UART_DP, >=20 > + { >=20 > + (UINT8) sizeof (UART_DEVICE_PATH), >=20 > + (UINT8) ((sizeof (UART_DEVICE_PATH)) >> 8) >=20 > + } >=20 > + }, >=20 > + 0, // Reserved >=20 > + 0, // BaudRate >=20 > + 0, // DataBits >=20 > + 0, // Parity >=20 > + 0 // StopBits >=20 > + }, >=20 > + { >=20 > + { >=20 > + MESSAGING_DEVICE_PATH, >=20 > + MSG_VENDOR_DP, >=20 > + { >=20 > + (UINT8) (sizeof (VENDOR_DEVICE_PATH)), >=20 > + (UINT8) ((sizeof (VENDOR_DEVICE_PATH)) >> 8), >=20 > + } >=20 > + }, >=20 > + DEVICE_PATH_MESSAGING_PC_ANSI >=20 > + }, >=20 > + gEndEntire >=20 > +}; >=20 > + >=20 > +/** >=20 > + Updates the ConOut, ConIn, ErrOut variables with the serial = terminal > device path >=20 > + @param none >=20 > + @retval none >=20 > +**/ >=20 > +VOID >=20 > +AddSerialTerminal ( >=20 > + VOID >=20 > + ) >=20 > +{ >=20 > + DEBUG ((DEBUG_INFO, "[AddSerialPortTerminal]\n")); >=20 > + >=20 > + // >=20 > + // Append Serial Terminal into "ConIn" >=20 > + // >=20 > + EfiBootManagerUpdateConsoleVariable (ConOut, > (EFI_DEVICE_PATH_PROTOCOL *) &mSerialDevicePath, NULL); >=20 > + EfiBootManagerUpdateConsoleVariable (ConIn, > (EFI_DEVICE_PATH_PROTOCOL *) &mSerialDevicePath, NULL); >=20 > + EfiBootManagerUpdateConsoleVariable (ErrOut, > (EFI_DEVICE_PATH_PROTOCOL *) &mSerialDevicePath, NULL); >=20 > +} >=20 > + >=20 > + >=20 > +/** >=20 > + Constructor for the Serial Port Device controller library. >=20 > + >=20 > + @param ImageHandle the image handle of the process >=20 > + @param SystemTable the EFI System Table pointer >=20 > + >=20 > + @retval EFI_SUCCESS the shell command handlers were > installed sucessfully >=20 > + @retval EFI_UNSUPPORTED the shell level required was not found. >=20 > +**/ >=20 > +EFI_STATUS >=20 > +EFIAPI >=20 > +SerialPortTerminalLibConstructor ( >=20 > + IN EFI_HANDLE ImageHandle, >=20 > + IN EFI_SYSTEM_TABLE *SystemTable >=20 > + ) >=20 > +{ >=20 > + mSerialDevicePath.Uart.BaudRate =3D > PcdGet64(PcdSerialTerminalBaudRate); >=20 > + mSerialDevicePath.Uart.DataBits =3D = PcdGet8(PcdSerialTerminalDataBits); >=20 > + mSerialDevicePath.Uart.Parity =3D = PcdGet8(PcdSerialTerminalParity); >=20 > + mSerialDevicePath.Uart.StopBits =3D = PcdGet8(PcdSerialTerminalStopBits); >=20 > + DEBUG ((DEBUG_INFO, "[SerialPortTerminalLibConstructor] > [%d, %d, %d, %d]\n", >=20 > + mSerialDevicePath.Uart.BaudRate, >=20 > + mSerialDevicePath.Uart.DataBits, >=20 > + mSerialDevicePath.Uart.Parity, >=20 > + mSerialDevicePath.Uart.StopBits)); >=20 > + >=20 > + AddSerialTerminal(); >=20 > + >=20 > + return EFI_SUCCESS; >=20 > +} >=20 > diff --git > a/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.h > b/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.h > new file mode 100644 > index 0000000000..0a2eaae8b8 > --- /dev/null > +++ > b/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.h > @@ -0,0 +1,48 @@ > +/** @file >=20 > + Header file for NULL named library for for Serial Port Terminal Redirection > library. >=20 > + >=20 > + Copyright (c) 2020, Intel Corporation. All rights reserved.
>=20 > + SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > + >=20 > +**/ >=20 > + >=20 > +#ifndef _SERIAL_PORT_TERMINAL_LIB_H_ >=20 > +#define _SERIAL_PORT_TERMINAL_LIB_H_ >=20 > + >=20 > +#include >=20 > +#include >=20 > +#include >=20 > +#include >=20 > +#include >=20 > +#include >=20 > + >=20 > +// >=20 > +// Below is the platform console device path >=20 > +// >=20 > +typedef struct { >=20 > + VENDOR_DEVICE_PATH Guid; >=20 > + UART_DEVICE_PATH Uart; >=20 > + VENDOR_DEVICE_PATH TerminalType; >=20 > + EFI_DEVICE_PATH_PROTOCOL End; >=20 > +} SERIAL_DEVICE_PATH; >=20 > + >=20 > +#define gPciRootBridge \ >=20 > + { \ >=20 > + { \ >=20 > + ACPI_DEVICE_PATH, \ >=20 > + ACPI_DP, \ >=20 > + { \ >=20 > + (UINT8) (sizeof (ACPI_HID_DEVICE_PATH)), \ >=20 > + (UINT8) ((sizeof (ACPI_HID_DEVICE_PATH)) >> 8) \ >=20 > + }, \ >=20 > + }, \ >=20 > + EISA_PNP_ID (0x0A03), \ >=20 > + 0 \ >=20 > + } >=20 > + >=20 > +#define gEndEntire \ >=20 > + { \ >=20 > + END_DEVICE_PATH_TYPE, END_ENTIRE_DEVICE_PATH_SUBTYPE, > { END_DEVICE_PATH_LENGTH, 0 } \ >=20 > + } >=20 > + >=20 > +#endif >=20 > diff --git > a/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.inf > b/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.inf > new file mode 100644 > index 0000000000..b0a9326b1b > --- /dev/null > +++ > b/Platform/Intel/MinPlatformPkg/Library/SerialPortTerminalLib/SerialPortT= er > minalLib.inf > @@ -0,0 +1,40 @@ > +## @file >=20 > +# Component information file for Serial Port Terminal Redirection = Library >=20 > +# >=20 > +# INTEL CONFIDENTIAL >=20 > +# Copyright (c) 2020, Intel Corporation. All rights reserved.
>=20 > +# >=20 > +# SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > +# >=20 > +## >=20 > +[Defines] >=20 > + INF_VERSION =3D 0x00010006 >=20 > + BASE_NAME =3D SerialPortTerminalLib >=20 > + FILE_GUID =3D > E12BFA46-95F2-4ADC-9774-7E38DE78741E >=20 > + MODULE_TYPE =3D UEFI_DRIVER >=20 > + VERSION_STRING =3D 1.2 >=20 > + LIBRARY_CLASS =3D NULL|UEFI_DRIVER DXE_DRIVER > DXE_RUNTIME_DRIVER >=20 > + CONSTRUCTOR =3D > SerialPortTerminalLibConstructor >=20 > + >=20 > +[Packages] >=20 > + MdePkg/MdePkg.dec >=20 > + MdeModulePkg/MdeModulePkg.dec >=20 > + BoardModulePkg/BoardModulePkg.dec >=20 > + MinPlatformPkg/MinPlatformPkg.dec >=20 > + >=20 > +[Sources] >=20 > + SerialPortTerminalLib.c >=20 > + SerialPortTerminalLib.h >=20 > + >=20 > +[LibraryClasses] >=20 > + DevicePathLib >=20 > + DebugLib >=20 > + UefiDriverEntryPoint >=20 > + UefiBootManagerLib >=20 > + UefiLib >=20 > + >=20 > +[Pcd] >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalBaudRate >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalDataBits >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalParity >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalStopBits > \ No newline at end of file > diff --git a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dec > b/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dec > index 7ef189dac8..36050aa1a8 100644 > --- a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dec > +++ b/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dec > @@ -205,6 +205,30 @@ > [PcdsDynamic, PcdsDynamicEx] >=20 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdPcIoApicEnable|0x0|UINT32|0x900000 > 19 >=20 >=20 >=20 > + # The baud rate setting for the UART style device. A value of 0 >=20 > + # means that the device's default baud rate will be used. >=20 > + > gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalBaudRate|115200|UINT6 > 4|0x9000001A >=20 > + >=20 > + # The number of data bits for the UART style device. A value >=20 > + # of 0 means that the device's default number of data bits will be used. >=20 > + > gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalDataBits|0x8|UINT8|0x9 > 000001B >=20 > + >=20 > + # The parity setting for the UART style device. >=20 > + # Parity 0x00 - Default Parity. >=20 > + # Parity 0x01 - No Parity. >=20 > + # Parity 0x02 - Even Parity. >=20 > + # Parity 0x03 - Odd Parity. >=20 > + # Parity 0x04 - Mark Parity. >=20 > + # Parity 0x05 - Space Parity. >=20 > + > gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalParity|0x1|UINT8|0x900 > 0001C >=20 > + >=20 > + # The number of stop bits for the UART style device. >=20 > + # Stop Bits 0x00 - Default Stop Bits. >=20 > + # Stop Bits 0x01 - 1 Stop Bit. >=20 > + # Stop Bits 0x02 - 1.5 Stop Bits. >=20 > + # Stop Bits 0x03 - 2 Stop Bits. >=20 > + > gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalStopBits|0x1|UINT8|0x9 > 000001D >=20 > + >=20 > [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx] >=20 >=20 >=20 > ## >=20 > @@ -318,3 +342,4 @@ > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable > |FALSE|BOOLEAN|0xF00000A5 >=20 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdSmiHandlerProfileEnable|FALSE|BOOLE > AN|0xF00000A6 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable > |FALSE|BOOLEAN|0xF00000A7 >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable > |FALSE|BOOLEAN|0xF00000B0 >=20 > diff --git a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc > b/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc > index 112ddff7d9..d0b5593817 100644 > --- a/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc > +++ b/Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc > @@ -1,7 +1,7 @@ > ## @file >=20 > # Platform description. >=20 > # >=20 > -# Copyright (c) 2017 - 2019, Intel Corporation. All rights = reserved.
>=20 > +# Copyright (c) 2017 - 2020, Intel Corporation. All rights = reserved.
>=20 > # >=20 > # SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > # >=20 > @@ -44,6 +44,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdSmiHandlerProfileEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|FALSE >=20 >=20 >=20 >=20 > ############################################################## > ################## >=20 > # >=20 > diff --git > a/Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkgPcd.ds > c > b/Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkgPcd.ds > c > index 52cdf9ec0f..251f46f812 100644 > --- > a/Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkgPcd.ds > c > +++ > b/Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkgPcd.ds > c > @@ -50,6 +50,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterMemInit|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|FALSE >=20 >=20 >=20 > !if gMinPlatformPkgTokenSpaceGuid.PcdBootStage >=3D 1 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit|TRUE >=20 > diff --git > a/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.d > sc > b/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.d > sc > index d797ae9d94..e161bf06f4 100644 > --- > a/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.d > sc > +++ > b/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.d > sc > @@ -127,6 +127,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|TRUE >=20 >=20 >=20 > !if gMinPlatformPkgTokenSpaceGuid.PcdBootStage >=3D 1 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit|TRUE >=20 > diff --git > a/Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardP > kgPcd.dsc > b/Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardP > kgPcd.dsc > index 5d769e8153..83ada3c95c 100644 > --- > a/Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardP > kgPcd.dsc > +++ > b/Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardP > kgPcd.dsc > @@ -127,6 +127,7 @@ > gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable|FALSE >=20 > gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable|FALSE >=20 > + gMinPlatformPkgTokenSpaceGuid.PcdSerialTerminalEnable|FALSE >=20 >=20 >=20 > !if gMinPlatformPkgTokenSpaceGuid.PcdBootStage >=3D 1 >=20 > gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit|TRUE >=20 > -- > 2.24.0.windows.2 >=20 >=20 >=20 > -=3D-=3D-=3D-=3D-=3D-=3D > Groups.io Links: You receive all messages sent to this group. > View/Reply Online (#66973): = https://edk2.groups.io/g/devel/message/66973 > Mute This Topic: https://groups.io/mt/78024794/4905953 > Group Owner: devel+owner@edk2.groups.io > Unsubscribe: https://edk2.groups.io/g/devel/unsub > [gaoliming@byosoft.com.cn] > -=3D-=3D-=3D-=3D-=3D-=3D >=20