[edk2-devel] [PATCH 1/2] BoardModulePkg: Add SerialPortTerminalLib

Dong, Eric eric.dong at intel.com
Thu Oct 22 01:23:15 UTC 2020


Hi Heng,

Seems like it is a common feature not related to board scope, I think Features folder is a better place to hold it.

Thanks,
Eric

-----Original Message-----
From: Luo, Heng <heng.luo at intel.com> 
Sent: Tuesday, October 20, 2020 10:22 AM
To: devel at edk2.groups.io
Cc: Dong, Eric <eric.dong at intel.com>; Chiu, Chasel <chasel.chiu at intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone at intel.com>
Subject: [PATCH 1/2] BoardModulePkg: Add SerialPortTerminalLib

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3014

Add the serial port terminal library to get console redirect
after the shell is loaded.

Cc: Eric Dong <eric.dong at intel.com>
Cc: Chasel Chiu <chasel.chiu at intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone at intel.com>

Signed-off-by: Heng Luo <heng.luo at intel.com>
---
 Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.c   | 93 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.h   | 49 +++++++++++++++++++++++++++++++++++++++++++++++++
 Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.inf | 35 +++++++++++++++++++++++++++++++++++
 3 files changed, 177 insertions(+)

diff --git a/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.c b/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.c
new file mode 100644
index 0000000000..36c0e7110f
--- /dev/null
+++ b/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.c
@@ -0,0 +1,93 @@
+/** @file

+  Main file for NULL named library for Serial Port Deviceboard controller librarr.

+

+  INTEL CONFIDENTIAL

+  Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>

+  SPDX-License-Identifier: BSD-2-Clause-Patent

+

+**/

+

+#include "SerialPortTerminalLib.h"

+

+GLOBAL_REMOVE_IF_UNREFERENCED SERIAL_DEVICE_PATH mSerialDevicePath = {

+  {

+    {

+      HARDWARE_DEVICE_PATH,

+      HW_VENDOR_DP,

+      {

+        (UINT8) sizeof (VENDOR_DEVICE_PATH),

+        (UINT8) ((sizeof (VENDOR_DEVICE_PATH)) >> 8)

+      }

+    },

+    EDKII_SERIAL_PORT_LIB_VENDOR_GUID

+  },

+  {

+    {

+      MESSAGING_DEVICE_PATH,

+      MSG_UART_DP,

+      {

+        (UINT8) sizeof (UART_DEVICE_PATH),

+        (UINT8) ((sizeof (UART_DEVICE_PATH)) >> 8)

+      }

+    },

+    0,                  // Reserved

+    115200,             // BaudRate

+    8,                  // DataBits

+    1,                  // Parity

+    1                   // StopBits

+  },

+  {

+    {

+      MESSAGING_DEVICE_PATH,

+      MSG_VENDOR_DP,

+      {

+        (UINT8) (sizeof (VENDOR_DEVICE_PATH)),

+        (UINT8) ((sizeof (VENDOR_DEVICE_PATH)) >> 8),

+      }

+    },

+    DEVICE_PATH_MESSAGING_PC_ANSI

+  },

+  gEndEntire

+};

+

+/**

+  Updates the ConOut, ConIn, ErrOut variables with the serial terminal device path

+  @param                        none

+  @retval                       none

+**/

+VOID

+AddSerialTerminal (

+  VOID

+  )

+{

+  DEBUG ((DEBUG_INFO, "[AddSerialPortTerminal]\n"));

+

+  //

+  // Append Serial Terminal into "ConIn"

+  //

+  EfiBootManagerUpdateConsoleVariable (ConOut, (EFI_DEVICE_PATH_PROTOCOL *) &mSerialDevicePath, NULL);

+  EfiBootManagerUpdateConsoleVariable (ConIn, (EFI_DEVICE_PATH_PROTOCOL *) &mSerialDevicePath, NULL);

+  EfiBootManagerUpdateConsoleVariable (ErrOut, (EFI_DEVICE_PATH_PROTOCOL *) &mSerialDevicePath, NULL);

+}

+

+

+/**

+  Constructor for the Serial Port Device controller library.

+

+  @param ImageHandle    the image handle of the process

+  @param SystemTable    the EFI System Table pointer

+

+  @retval EFI_SUCCESS        the shell command handlers were installed sucessfully

+  @retval EFI_UNSUPPORTED    the shell level required was not found.

+**/

+EFI_STATUS

+EFIAPI

+SerialPortTerminalLibConstructor (

+  IN EFI_HANDLE        ImageHandle,

+  IN EFI_SYSTEM_TABLE  *SystemTable

+  )

+{

+  AddSerialTerminal();

+

+  return EFI_SUCCESS;

+}

diff --git a/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.h b/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.h
new file mode 100644
index 0000000000..afb3eb2cb3
--- /dev/null
+++ b/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.h
@@ -0,0 +1,49 @@
+/** @file

+  Header file for NULL named library for Ps2 keyboard controller library.

+

+  INTEL CONFIDENTIAL

+  Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>

+  SPDX-License-Identifier: BSD-2-Clause-Patent

+

+**/

+

+#ifndef _PS2_KBC_LIB_H

+#define _PS2_KBC_LIB_H

+

+#include <Uefi.h>

+#include <Guid/SerialPortLibVendor.h>

+#include <Library/UefiLib.h>

+#include <Library/DevicePathLib.h>

+#include <Library/DebugLib.h>

+#include <Library/UefiBootManagerLib.h>

+

+//

+// Below is the platform console device path

+//

+typedef struct {

+  VENDOR_DEVICE_PATH        Guid;

+  UART_DEVICE_PATH          Uart;

+  VENDOR_DEVICE_PATH        TerminalType;

+  EFI_DEVICE_PATH_PROTOCOL  End;

+} SERIAL_DEVICE_PATH;

+

+#define gPciRootBridge \

+  { \

+    { \

+      ACPI_DEVICE_PATH, \

+      ACPI_DP, \

+      { \

+        (UINT8) (sizeof (ACPI_HID_DEVICE_PATH)), \

+        (UINT8) ((sizeof (ACPI_HID_DEVICE_PATH)) >> 8) \

+      }, \

+    }, \

+    EISA_PNP_ID (0x0A03), \

+    0 \

+  }

+

+#define gEndEntire \

+  { \

+    END_DEVICE_PATH_TYPE, END_ENTIRE_DEVICE_PATH_SUBTYPE, { END_DEVICE_PATH_LENGTH, 0 } \

+  }

+

+#endif

diff --git a/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.inf b/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.inf
new file mode 100644
index 0000000000..4f63d1626f
--- /dev/null
+++ b/Platform/Intel/BoardModulePkg/Library/SerialPortTerminalLib/SerialPortTerminalLib.inf
@@ -0,0 +1,35 @@
+## @file

+# Component information file for Serial Port Terminal Redirection Library

+#

+# INTEL CONFIDENTIAL

+# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>

+#

+# SPDX-License-Identifier: BSD-2-Clause-Patent

+#

+##

+[Defines]

+  INF_VERSION                    = 0x00010006

+  BASE_NAME                      = SerialPortTerminalLib

+  FILE_GUID                      = E12BFA46-95F2-4ADC-9774-7E38DE78741E

+  MODULE_TYPE                    = UEFI_DRIVER

+  VERSION_STRING                 = 1.2

+  LIBRARY_CLASS                  = NULL|UEFI_DRIVER

+  CONSTRUCTOR                    = SerialPortTerminalLibConstructor

+

+[Packages]

+  MdePkg/MdePkg.dec

+  MdeModulePkg/MdeModulePkg.dec

+  BoardModulePkg/BoardModulePkg.dec

+

+[Sources]

+  SerialPortTerminalLib.c

+  SerialPortTerminalLib.h

+

+[LibraryClasses]

+  DevicePathLib

+  DebugLib

+  UefiDriverEntryPoint

+  UefiBootManagerLib

+  UefiLib

+

+[Pcd]

-- 
2.24.0.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#66517): https://edk2.groups.io/g/devel/message/66517
Mute This Topic: https://groups.io/mt/77674093/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-






More information about the edk2-devel-archive mailing list