[edk2] [PATCH 05/10] Platform/NXP : Add support for DUART library

Meenakshi Aggarwal posted 10 patches 7 years, 1 month ago
[edk2] [PATCH 05/10] Platform/NXP : Add support for DUART library
Posted by Meenakshi Aggarwal 7 years, 1 month ago
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
---
 Platform/NXP/Library/DUartPortLib/DUart.h          | 128 ++++++++
 Platform/NXP/Library/DUartPortLib/DUartPortLib.c   | 334 +++++++++++++++++++++
 Platform/NXP/Library/DUartPortLib/DUartPortLib.inf |  39 +++
 3 files changed, 501 insertions(+)
 create mode 100644 Platform/NXP/Library/DUartPortLib/DUart.h
 create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.c
 create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.inf

diff --git a/Platform/NXP/Library/DUartPortLib/DUart.h b/Platform/NXP/Library/DUartPortLib/DUart.h
new file mode 100644
index 0000000..907790b
--- /dev/null
+++ b/Platform/NXP/Library/DUartPortLib/DUart.h
@@ -0,0 +1,128 @@
+/** DUart.h
+*  Header defining the DUART constants (Base addresses, sizes, flags)
+*
+*  Based on Serial I/O Port library headers available in PL011Uart.h
+*
+*  Copyright (c) 2011-2012, ARM Limited. All rights reserved.
+*  Copyright (c) 2016, Freescale Semiconductor, Inc. All rights reserved.
+*  Copyright 2017 NXP
+*
+*  This program and the accompanying materials
+*  are licensed and made available under the terms and conditions of the BSD License
+*  which accompanies this distribution.  The full text of the license may be found at
+*  http://opensource.org/licenses/bsd-license.php
+*
+*  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+*  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+*
+**/
+
+#ifndef __DUART_H__
+#define __DUART_H__
+
+// FIFO Control Register
+#define DUART_FCR_FIFO_EN          0x01 /* Fifo enable */
+#define DUART_FCR_CLEAR_RCVR       0x02 /* Clear the RCVR FIFO */
+#define DUART_FCR_CLEAR_XMIT       0x04 /* Clear the XMIT FIFO */
+#define DUART_FCR_DMA_SELECT       0x08 /* For DMA applications */
+#define DUART_FCR_TRIGGER_MASK     0xC0 /* Mask for the FIFO trigger range */
+#define DUART_FCR_TRIGGER_1        0x00 /* Mask for trigger set at 1 */
+#define DUART_FCR_TRIGGER_4        0x40 /* Mask for trigger set at 4 */
+#define DUART_FCR_TRIGGER_8        0x80 /* Mask for trigger set at 8 */
+#define DUART_FCR_TRIGGER_14       0xC0 /* Mask for trigger set at 14 */
+#define DUART_FCR_RXSR             0x02 /* Receiver soft reset */
+#define DUART_FCR_TXSR             0x04 /* Transmitter soft reset */
+
+// Modem Control Register
+#define DUART_MCR_DTR              0x01 /* Reserved  */
+#define DUART_MCR_RTS              0x02 /* RTS   */
+#define DUART_MCR_OUT1             0x04 /* Reserved */
+#define DUART_MCR_OUT2             0x08 /* Reserved */
+#define DUART_MCR_LOOP             0x10 /* Enable loopback test mode */
+#define DUART_MCR_AFE              0x20 /* AFE (Auto Flow Control) */
+#define DUART_MCR_DMA_EN           0x04
+#define DUART_MCR_TX_DFR           0x08
+
+// Line Control Register
+/*
+* Note: if the word length is 5 bits (DUART_LCR_WLEN5), then setting
+* DUART_LCR_STOP will select 1.5 stop bits, not 2 stop bits.
+*/
+#define DUART_LCR_WLS_MSK          0x03 /* character length select mask */
+#define DUART_LCR_WLS_5            0x00 /* 5 bit character length */
+#define DUART_LCR_WLS_6            0x01 /* 6 bit character length */
+#define DUART_LCR_WLS_7            0x02 /* 7 bit character length */
+#define DUART_LCR_WLS_8            0x03 /* 8 bit character length */
+#define DUART_LCR_STB              0x04 /* # stop Bits, off=1, on=1.5 or 2) */
+#define DUART_LCR_PEN              0x08 /* Parity eneble */
+#define DUART_LCR_EPS              0x10 /* Even Parity Select */
+#define DUART_LCR_STKP             0x20 /* Stick Parity */
+#define DUART_LCR_SBRK             0x40 /* Set Break */
+#define DUART_LCR_BKSE             0x80 /* Bank select enable */
+#define DUART_LCR_DLAB             0x80 /* Divisor latch access bit */
+
+// Line Status Register
+#define DUART_LSR_DR               0x01 /* Data ready */
+#define DUART_LSR_OE               0x02 /* Overrun */
+#define DUART_LSR_PE               0x04 /* Parity error */
+#define DUART_LSR_FE               0x08 /* Framing error */
+#define DUART_LSR_BI               0x10 /* Break */
+#define DUART_LSR_THRE             0x20 /* Xmit holding register empty */
+#define DUART_LSR_TEMT             0x40 /* Xmitter empty */
+#define DUART_LSR_ERR              0x80 /* Error */
+
+// Modem Status Register
+#define DUART_MSR_DCTS             0x01 /* Delta CTS */
+#define DUART_MSR_DDSR             0x02 /* Reserved */
+#define DUART_MSR_TERI             0x04 /* Reserved */
+#define DUART_MSR_DDCD             0x08 /* Reserved */
+#define DUART_MSR_CTS              0x10 /* Clear to Send */
+#define DUART_MSR_DSR              0x20 /* Reserved */
+#define DUART_MSR_RI               0x40 /* Reserved */
+#define DUART_MSR_DCD              0x80 /* Reserved */
+
+// Interrupt Identification Register
+#define DUART_IIR_NO_INT           0x01 /* No interrupts pending */
+#define DUART_IIR_ID               0x06 /* Mask for the interrupt ID */
+#define DUART_IIR_MSI              0x00 /* Modem status interrupt */
+#define DUART_IIR_THRI             0x02 /* Transmitter holding register empty */
+#define DUART_IIR_RDI              0x04 /* Receiver data interrupt */
+#define DUART_IIR_RLSI             0x06 /* Receiver line status interrupt */
+
+//  Interrupt Enable Register
+#define DUART_IER_MSI              0x08 /* Enable Modem status interrupt */
+#define DUART_IER_RLSI             0x04 /* Enable receiver line status interrupt */
+#define DUART_IER_THRI             0x02 /* Enable Transmitter holding register int. */
+#define DUART_IER_RDI              0x01 /* Enable receiver data interrupt */
+
+// LCR defaults
+#define DUART_LCR_8N1              0x03
+#define DUART_LCRVAL               DUART_LCR_8N1          /* 8 data, 1 stop, no parity */
+#define DUART_MCRVAL               (DUART_MCR_DTR | \
+                                   DUART_MCR_RTS)         /* RTS/DTR */
+#define DUART_FCRVAL               (DUART_FCR_FIFO_EN | \
+                                   DUART_FCR_RXSR |    \
+                                   DUART_FCR_TXSR)        /* Clear & enable FIFOs */
+
+#define URBR         0x0
+#define UTHR         0x0
+#define UDLB         0x0
+#define UDMB         0x1
+#define UIER         0x1
+#define UIIR         0x2
+#define UFCR         0x2
+#define UAFR         0x2
+#define ULCR         0x3
+#define UMCR         0x4
+#define ULSR         0x5
+#define UMSR         0x6
+#define USCR         0x7
+#define UDSR         0x10
+
+extern
+UINT32
+CalculateBaudDivisor (
+  IN  UINT64 BaudRate
+  );
+
+#endif /* __DUART_H__ */
diff --git a/Platform/NXP/Library/DUartPortLib/DUartPortLib.c b/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
new file mode 100644
index 0000000..99766d1
--- /dev/null
+++ b/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
@@ -0,0 +1,334 @@
+/** DuartPortLib.c
+  DUART (NS16550) library functions
+
+  Based on Serial I/O Port library functions available in PL011SerialPortLib.c
+
+  Copyright (c) 2008 - 2010, Apple Inc. All rights reserved.<BR>
+  Copyright (c) 2012 - 2013, ARM Ltd. All rights reserved.<BR>
+  Copyright (c) 2016, Freescale Semiconductor, Inc. All rights reserved.
+  Copyright 2017 NXP
+
+  This program and the accompanying materials
+  are licensed and made available under the terms and conditions of the BSD License
+  which accompanies this distribution.  The full text of the license may be found at
+  http://opensource.org/licenses/bsd-license.php
+
+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+
+**/
+
+#include <Base.h>
+#include <Library/IoLib.h>
+#include <Library/PcdLib.h>
+#include <Library/SerialPortLib.h>
+
+#include "DUart.h"
+
+STATIC CONST UINT32 mInvalidControlBits = (EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE | \
+                                           EFI_SERIAL_DATA_TERMINAL_READY);
+
+/**
+  Assert or deassert the control signals on a serial port.
+  The following control signals are set according their bit settings :
+  . Request to Send
+  . Data Terminal Ready
+
+  @param[in]  Control   The following bits are taken into account :
+                        . EFI_SERIAL_REQUEST_TO_SEND : assert/deassert the
+                          "Request To Send" control signal if this bit is
+                          equal to one/zero.
+                        . EFI_SERIAL_DATA_TERMINAL_READY : assert/deassert
+                          the "Data Terminal Ready" control signal if this
+                          bit is equal to one/zero.
+                        . EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE : enable/disable
+                          the hardware loopback if this bit is equal to
+                          one/zero.
+                        . EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE : not supported.
+                        . EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE : enable/
+                          disable the hardware flow control based on CTS (Clear
+                          To Send) and RTS (Ready To Send) control signals.
+
+  @retval  EFI_SUCCESS      The new control bits were set on the device.
+  @retval  EFI_UNSUPPORTED  The device does not support this operation.
+
+**/
+EFI_STATUS
+EFIAPI
+SerialPortSetControl (
+  IN  UINT32  Control
+  )
+{
+  UINT32  McrBits;
+  UINTN   UartBase;
+
+  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
+
+  if (Control & (mInvalidControlBits)) {
+    return EFI_UNSUPPORTED;
+  }
+
+  McrBits = MmioRead8 (UartBase + UMCR);
+
+  if (Control & EFI_SERIAL_REQUEST_TO_SEND) {
+    McrBits |= DUART_MCR_RTS;
+  } else {
+    McrBits &= ~DUART_MCR_RTS;
+  }
+
+  if (Control & EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE) {
+    McrBits |= DUART_MCR_LOOP;
+  } else {
+    McrBits &= ~DUART_MCR_LOOP;
+  }
+
+  if (Control & EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE) {
+    McrBits |= DUART_MCR_AFE;
+  } else {
+    McrBits &= ~DUART_MCR_AFE;
+  }
+
+  MmioWrite32 (UartBase + UMCR, McrBits);
+
+  return EFI_SUCCESS;
+}
+
+/**
+  Retrieve the status of the control bits on a serial device.
+
+  @param[out]  Control   Status of the control bits on a serial device :
+
+                         . EFI_SERIAL_DATA_CLEAR_TO_SEND,
+                           EFI_SERIAL_DATA_SET_READY,
+                           EFI_SERIAL_RING_INDICATE,
+                           EFI_SERIAL_CARRIER_DETECT,
+                           EFI_SERIAL_REQUEST_TO_SEND,
+                           EFI_SERIAL_DATA_TERMINAL_READY
+                           are all related to the DTE (Data Terminal Equipment)
+                           and DCE (Data Communication Equipment) modes of
+                           operation of the serial device.
+                         . EFI_SERIAL_INPUT_BUFFER_EMPTY : equal to one if the
+                           receive buffer is empty, 0 otherwise.
+                         . EFI_SERIAL_OUTPUT_BUFFER_EMPTY : equal to one if the
+                           transmit buffer is empty, 0 otherwise.
+                         . EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE : equal to one if
+                           the hardware loopback is enabled (the ouput feeds the
+                           receive buffer), 0 otherwise.
+                         . EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE : equal to one if
+                           a loopback is accomplished by software, 0 otherwise.
+                         . EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE : equal to
+                           one if the hardware flow control based on CTS (Clear
+                           To Send) and RTS (Ready To Send) control signals is
+                           enabled, 0 otherwise.
+
+  @retval EFI_SUCCESS      The control bits were read from the serial device.
+
+**/
+
+EFI_STATUS
+EFIAPI
+SerialPortGetControl (
+  OUT  UINT32  *Control
+  )
+{
+  UINT32      MsrRegister;
+  UINT32      McrRegister;
+  UINT32      LsrRegister;
+  UINTN       UartBase;
+
+  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
+
+  MsrRegister = MmioRead8 (UartBase + UMSR);
+  McrRegister = MmioRead8 (UartBase + UMCR);
+  LsrRegister = MmioRead8 (UartBase + ULSR);
+
+  *Control = 0;
+
+  if ((MsrRegister & DUART_MSR_CTS) == DUART_MSR_CTS) {
+    *Control |= EFI_SERIAL_CLEAR_TO_SEND;
+  }
+
+  if ((McrRegister & DUART_MCR_RTS) == DUART_MCR_RTS) {
+    *Control |= EFI_SERIAL_REQUEST_TO_SEND;
+  }
+
+  if ((LsrRegister & DUART_LSR_TEMT) == DUART_LSR_TEMT) {
+    *Control |= EFI_SERIAL_OUTPUT_BUFFER_EMPTY;
+  }
+
+  if ((McrRegister & DUART_MCR_AFE) == DUART_MCR_AFE) {
+    *Control |= EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE;
+  }
+
+  if ((McrRegister & DUART_MCR_LOOP) == DUART_MCR_LOOP) {
+    *Control |= EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE;
+  }
+
+  return EFI_SUCCESS;
+}
+
+/**
+  Programmed hardware of Serial port.
+
+  @return    Always return EFI_SUCCESS.
+
+**/
+
+EFI_STATUS
+EFIAPI
+SerialPortInitialize (
+  VOID
+  )
+{
+  UINT64  BaudRate;
+  UINT32  BaudDivisor;
+  UINTN   UartBase;
+
+  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
+  BaudRate = (UINTN)PcdGet64 (PcdUartDefaultBaudRate);
+
+  BaudDivisor = CalculateBaudDivisor(BaudRate);
+
+  while (!(MmioRead8(UartBase + ULSR) & DUART_LSR_TEMT));
+
+  //
+  // Enable received data available interrupt, setup data format,
+  // setup baud divisor
+  //
+  MmioWrite8(UartBase + UIER, 0x1);
+  MmioWrite8(UartBase + ULCR, DUART_LCR_BKSE | DUART_LCRVAL);
+  MmioWrite8(UartBase + UDLB, 0);
+  MmioWrite8(UartBase + UDMB, 0);
+  MmioWrite8(UartBase + ULCR, DUART_LCRVAL);
+  MmioWrite8(UartBase + UMCR, DUART_MCRVAL);
+  MmioWrite8(UartBase + UFCR, DUART_FCRVAL);
+  MmioWrite8(UartBase + ULCR, DUART_LCR_BKSE | DUART_LCRVAL);
+  MmioWrite8(UartBase + UDLB, BaudDivisor & 0xff);
+  MmioWrite8(UartBase + UDMB, (BaudDivisor >> 8) & 0xff);
+  MmioWrite8(UartBase + ULCR, DUART_LCRVAL);
+
+  return EFI_SUCCESS;
+}
+
+/**
+  Write data to serial device.
+
+  @param  Buffer           Point of data buffer which need to be written.
+  @param  NumberOfBytes    Number of output bytes which are cached in Buffer.
+
+  @retval 0                Write data failed.
+  @retval !0               Actual number of bytes written to serial device.
+
+**/
+UINTN
+EFIAPI
+SerialPortWrite (
+  IN  UINT8     *Buffer,
+  IN  UINTN     NumberOfBytes
+  )
+{
+  UINT8   *Final;
+  UINTN   UartBase;
+
+  Final = &Buffer[NumberOfBytes];
+  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
+
+  while (Buffer < Final) {
+    while ((MmioRead8(UartBase + ULSR) & DUART_LSR_THRE) == 0);
+    MmioWrite8(UartBase + UTHR, *Buffer++);
+  }
+
+  return NumberOfBytes;
+}
+
+/**
+  Read data from serial device and save the data in buffer.
+
+  @param  Buffer           Point of data buffer which need to be written.
+  @param  NumberOfBytes    Number of output bytes which are cached in Buffer.
+
+  @retval 0                Read data failed.
+  @retval !0               Actual number of bytes read from serial device.
+
+**/
+UINTN
+EFIAPI
+SerialPortRead (
+  OUT UINT8     *Buffer,
+  IN  UINTN     NumberOfBytes
+  )
+{
+  UINTN   Count;
+  UINTN   UartBase;
+
+  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
+
+  for (Count = 0; Count < NumberOfBytes; Count++, Buffer++) {
+    /*
+     * Loop while waiting for a new char(s) to arrive in the
+     * RxFIFO
+     */
+    while ((MmioRead8(UartBase + ULSR) & DUART_LSR_DR) == 0);
+
+    *Buffer = MmioRead8(UartBase + URBR);
+  }
+
+  return NumberOfBytes;
+}
+
+/**
+  Check to see if any data is available to be read from the debug device.
+
+  @retval EFI_SUCCESS       At least one byte of data is available to be read
+  @retval EFI_NOT_READY     No data is available to be read
+  @retval EFI_DEVICE_ERROR  The serial device is not functioning properly
+
+**/
+BOOLEAN
+EFIAPI
+SerialPortPoll (
+  VOID
+  )
+{
+  UINTN   UartBase;
+
+  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
+
+  return ((MmioRead8 (UartBase + ULSR) & DUART_LSR_DR) != 0);
+}
+
+/**
+  Set new attributes to LS1043a.
+
+  @param  BaudRate                The baud rate of the serial device. If the baud rate is not supported,
+                                  the speed will be reduced down to the nearest supported one and the
+                                  variable's value will be updated accordingly.
+  @param  ReceiveFifoDepth        The number of characters the device will buffer on input. If the specified
+                                  value is not supported, the variable's value will be reduced down to the
+                                  nearest supported one.
+  @param  Timeout                 If applicable, the number of microseconds the device will wait
+                                  before timing out a Read or a Write operation.
+  @param  Parity                  If applicable, this is the EFI_PARITY_TYPE that is computed or checked
+                                  as each character is transmitted or received. If the device does not
+                                  support parity, the value is the default parity value.
+  @param  DataBits                The number of data bits in each character
+  @param  StopBits                If applicable, the EFI_STOP_BITS_TYPE number of stop bits per character.
+                                  If the device does not support stop bits, the value is the default stop
+                                  bit value.
+
+  @retval EFI_SUCCESS             All attributes were set correctly on the serial device.
+
+**/
+EFI_STATUS
+EFIAPI
+SerialPortSetAttributes (
+  IN  OUT  UINT64              *BaudRate,
+  IN  OUT  UINT32              *ReceiveFifoDepth,
+  IN  OUT  UINT32              *Timeout,
+  IN  OUT  EFI_PARITY_TYPE     *Parity,
+  IN  OUT  UINT8               *DataBits,
+  IN  OUT  EFI_STOP_BITS_TYPE  *StopBits
+  )
+{
+  return SerialPortInitialize ();
+}
diff --git a/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf b/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
new file mode 100644
index 0000000..ead8b40
--- /dev/null
+++ b/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
@@ -0,0 +1,39 @@
+#  DUartPortLib.inf
+#
+#  Component description file for DUartPortLib module
+#
+#  Copyright (c) 2013, Freescale Ltd. All rights reserved.
+#  Copyright 2017 NXP
+#
+#  This program and the accompanying materials
+#  are licensed and made available under the terms and conditions of the BSD License
+#  which accompanies this distribution.  The full text of the license may be found at
+#  http://opensource.org/licenses/bsd-license.php
+#
+#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+#
+#
+
+[Defines]
+  INF_VERSION                    = 0x00010005
+  BASE_NAME                      = DUartPortLib
+  FILE_GUID                      = 8ecefc8f-a2c4-4091-b80f-20f7aeb0567f
+  MODULE_TYPE                    = BASE
+  VERSION_STRING                 = 1.0
+  LIBRARY_CLASS                  = SerialPortLib
+
+[Sources.common]
+  DUartPortLib.c
+
+[LibraryClasses]
+  PcdLib
+  SocLib
+
+[Packages]
+  MdeModulePkg/MdeModulePkg.dec
+  MdePkg/MdePkg.dec
+
+[Pcd]
+  gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase
+  gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate
-- 
1.9.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 05/10] Platform/NXP : Add support for DUART library
Posted by Ard Biesheuvel 7 years, 1 month ago
On 7 November 2017 at 14:42, Meenakshi Aggarwal
<meenakshi.aggarwal@nxp.com> wrote:
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
> ---
>  Platform/NXP/Library/DUartPortLib/DUart.h          | 128 ++++++++
>  Platform/NXP/Library/DUartPortLib/DUartPortLib.c   | 334 +++++++++++++++++++++
>  Platform/NXP/Library/DUartPortLib/DUartPortLib.inf |  39 +++
>  3 files changed, 501 insertions(+)
>  create mode 100644 Platform/NXP/Library/DUartPortLib/DUart.h
>  create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.c
>  create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
>
> diff --git a/Platform/NXP/Library/DUartPortLib/DUart.h b/Platform/NXP/Library/DUartPortLib/DUart.h
> new file mode 100644
> index 0000000..907790b
> --- /dev/null
> +++ b/Platform/NXP/Library/DUartPortLib/DUart.h
> @@ -0,0 +1,128 @@
> +/** DUart.h
> +*  Header defining the DUART constants (Base addresses, sizes, flags)
> +*
> +*  Based on Serial I/O Port library headers available in PL011Uart.h
> +*
> +*  Copyright (c) 2011-2012, ARM Limited. All rights reserved.
> +*  Copyright (c) 2016, Freescale Semiconductor, Inc. All rights reserved.
> +*  Copyright 2017 NXP
> +*
> +*  This program and the accompanying materials
> +*  are licensed and made available under the terms and conditions of the BSD License
> +*  which accompanies this distribution.  The full text of the license may be found at
> +*  http://opensource.org/licenses/bsd-license.php
> +*
> +*  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +*  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +*
> +**/
> +
> +#ifndef __DUART_H__
> +#define __DUART_H__
> +
> +// FIFO Control Register
> +#define DUART_FCR_FIFO_EN          0x01 /* Fifo enable */
> +#define DUART_FCR_CLEAR_RCVR       0x02 /* Clear the RCVR FIFO */
> +#define DUART_FCR_CLEAR_XMIT       0x04 /* Clear the XMIT FIFO */
> +#define DUART_FCR_DMA_SELECT       0x08 /* For DMA applications */
> +#define DUART_FCR_TRIGGER_MASK     0xC0 /* Mask for the FIFO trigger range */
> +#define DUART_FCR_TRIGGER_1        0x00 /* Mask for trigger set at 1 */
> +#define DUART_FCR_TRIGGER_4        0x40 /* Mask for trigger set at 4 */
> +#define DUART_FCR_TRIGGER_8        0x80 /* Mask for trigger set at 8 */
> +#define DUART_FCR_TRIGGER_14       0xC0 /* Mask for trigger set at 14 */
> +#define DUART_FCR_RXSR             0x02 /* Receiver soft reset */
> +#define DUART_FCR_TXSR             0x04 /* Transmitter soft reset */
> +
> +// Modem Control Register
> +#define DUART_MCR_DTR              0x01 /* Reserved  */
> +#define DUART_MCR_RTS              0x02 /* RTS   */
> +#define DUART_MCR_OUT1             0x04 /* Reserved */
> +#define DUART_MCR_OUT2             0x08 /* Reserved */
> +#define DUART_MCR_LOOP             0x10 /* Enable loopback test mode */
> +#define DUART_MCR_AFE              0x20 /* AFE (Auto Flow Control) */
> +#define DUART_MCR_DMA_EN           0x04
> +#define DUART_MCR_TX_DFR           0x08
> +
> +// Line Control Register
> +/*
> +* Note: if the word length is 5 bits (DUART_LCR_WLEN5), then setting
> +* DUART_LCR_STOP will select 1.5 stop bits, not 2 stop bits.
> +*/
> +#define DUART_LCR_WLS_MSK          0x03 /* character length select mask */
> +#define DUART_LCR_WLS_5            0x00 /* 5 bit character length */
> +#define DUART_LCR_WLS_6            0x01 /* 6 bit character length */
> +#define DUART_LCR_WLS_7            0x02 /* 7 bit character length */
> +#define DUART_LCR_WLS_8            0x03 /* 8 bit character length */
> +#define DUART_LCR_STB              0x04 /* # stop Bits, off=1, on=1.5 or 2) */
> +#define DUART_LCR_PEN              0x08 /* Parity eneble */
> +#define DUART_LCR_EPS              0x10 /* Even Parity Select */
> +#define DUART_LCR_STKP             0x20 /* Stick Parity */
> +#define DUART_LCR_SBRK             0x40 /* Set Break */
> +#define DUART_LCR_BKSE             0x80 /* Bank select enable */
> +#define DUART_LCR_DLAB             0x80 /* Divisor latch access bit */
> +
> +// Line Status Register
> +#define DUART_LSR_DR               0x01 /* Data ready */
> +#define DUART_LSR_OE               0x02 /* Overrun */
> +#define DUART_LSR_PE               0x04 /* Parity error */
> +#define DUART_LSR_FE               0x08 /* Framing error */
> +#define DUART_LSR_BI               0x10 /* Break */
> +#define DUART_LSR_THRE             0x20 /* Xmit holding register empty */
> +#define DUART_LSR_TEMT             0x40 /* Xmitter empty */
> +#define DUART_LSR_ERR              0x80 /* Error */
> +
> +// Modem Status Register
> +#define DUART_MSR_DCTS             0x01 /* Delta CTS */
> +#define DUART_MSR_DDSR             0x02 /* Reserved */
> +#define DUART_MSR_TERI             0x04 /* Reserved */
> +#define DUART_MSR_DDCD             0x08 /* Reserved */
> +#define DUART_MSR_CTS              0x10 /* Clear to Send */
> +#define DUART_MSR_DSR              0x20 /* Reserved */
> +#define DUART_MSR_RI               0x40 /* Reserved */
> +#define DUART_MSR_DCD              0x80 /* Reserved */
> +
> +// Interrupt Identification Register
> +#define DUART_IIR_NO_INT           0x01 /* No interrupts pending */
> +#define DUART_IIR_ID               0x06 /* Mask for the interrupt ID */
> +#define DUART_IIR_MSI              0x00 /* Modem status interrupt */
> +#define DUART_IIR_THRI             0x02 /* Transmitter holding register empty */
> +#define DUART_IIR_RDI              0x04 /* Receiver data interrupt */
> +#define DUART_IIR_RLSI             0x06 /* Receiver line status interrupt */
> +
> +//  Interrupt Enable Register
> +#define DUART_IER_MSI              0x08 /* Enable Modem status interrupt */
> +#define DUART_IER_RLSI             0x04 /* Enable receiver line status interrupt */
> +#define DUART_IER_THRI             0x02 /* Enable Transmitter holding register int. */
> +#define DUART_IER_RDI              0x01 /* Enable receiver data interrupt */
> +
> +// LCR defaults
> +#define DUART_LCR_8N1              0x03
> +#define DUART_LCRVAL               DUART_LCR_8N1          /* 8 data, 1 stop, no parity */
> +#define DUART_MCRVAL               (DUART_MCR_DTR | \
> +                                   DUART_MCR_RTS)         /* RTS/DTR */
> +#define DUART_FCRVAL               (DUART_FCR_FIFO_EN | \
> +                                   DUART_FCR_RXSR |    \
> +                                   DUART_FCR_TXSR)        /* Clear & enable FIFOs */
> +
> +#define URBR         0x0
> +#define UTHR         0x0
> +#define UDLB         0x0
> +#define UDMB         0x1
> +#define UIER         0x1
> +#define UIIR         0x2
> +#define UFCR         0x2
> +#define UAFR         0x2
> +#define ULCR         0x3
> +#define UMCR         0x4
> +#define ULSR         0x5
> +#define UMSR         0x6
> +#define USCR         0x7
> +#define UDSR         0x10
> +
> +extern
> +UINT32
> +CalculateBaudDivisor (
> +  IN  UINT64 BaudRate
> +  );
> +
> +#endif /* __DUART_H__ */
> diff --git a/Platform/NXP/Library/DUartPortLib/DUartPortLib.c b/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
> new file mode 100644
> index 0000000..99766d1
> --- /dev/null
> +++ b/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
> @@ -0,0 +1,334 @@
> +/** DuartPortLib.c
> +  DUART (NS16550) library functions
> +
> +  Based on Serial I/O Port library functions available in PL011SerialPortLib.c
> +
> +  Copyright (c) 2008 - 2010, Apple Inc. All rights reserved.<BR>
> +  Copyright (c) 2012 - 2013, ARM Ltd. All rights reserved.<BR>
> +  Copyright (c) 2016, Freescale Semiconductor, Inc. All rights reserved.
> +  Copyright 2017 NXP
> +
> +  This program and the accompanying materials
> +  are licensed and made available under the terms and conditions of the BSD License
> +  which accompanies this distribution.  The full text of the license may be found at
> +  http://opensource.org/licenses/bsd-license.php
> +
> +  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +
> +**/
> +
> +#include <Base.h>
> +#include <Library/IoLib.h>
> +#include <Library/PcdLib.h>
> +#include <Library/SerialPortLib.h>
> +
> +#include "DUart.h"
> +
> +STATIC CONST UINT32 mInvalidControlBits = (EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE | \
> +                                           EFI_SERIAL_DATA_TERMINAL_READY);
> +
> +/**
> +  Assert or deassert the control signals on a serial port.
> +  The following control signals are set according their bit settings :
> +  . Request to Send
> +  . Data Terminal Ready
> +
> +  @param[in]  Control   The following bits are taken into account :
> +                        . EFI_SERIAL_REQUEST_TO_SEND : assert/deassert the
> +                          "Request To Send" control signal if this bit is
> +                          equal to one/zero.
> +                        . EFI_SERIAL_DATA_TERMINAL_READY : assert/deassert
> +                          the "Data Terminal Ready" control signal if this
> +                          bit is equal to one/zero.
> +                        . EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE : enable/disable
> +                          the hardware loopback if this bit is equal to
> +                          one/zero.
> +                        . EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE : not supported.
> +                        . EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE : enable/
> +                          disable the hardware flow control based on CTS (Clear
> +                          To Send) and RTS (Ready To Send) control signals.
> +
> +  @retval  EFI_SUCCESS      The new control bits were set on the device.
> +  @retval  EFI_UNSUPPORTED  The device does not support this operation.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +SerialPortSetControl (
> +  IN  UINT32  Control
> +  )
> +{
> +  UINT32  McrBits;
> +  UINTN   UartBase;
> +
> +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> +
> +  if (Control & (mInvalidControlBits)) {
> +    return EFI_UNSUPPORTED;
> +  }
> +
> +  McrBits = MmioRead8 (UartBase + UMCR);
> +
> +  if (Control & EFI_SERIAL_REQUEST_TO_SEND) {
> +    McrBits |= DUART_MCR_RTS;
> +  } else {
> +    McrBits &= ~DUART_MCR_RTS;
> +  }
> +
> +  if (Control & EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE) {
> +    McrBits |= DUART_MCR_LOOP;
> +  } else {
> +    McrBits &= ~DUART_MCR_LOOP;
> +  }
> +
> +  if (Control & EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE) {
> +    McrBits |= DUART_MCR_AFE;
> +  } else {
> +    McrBits &= ~DUART_MCR_AFE;
> +  }
> +
> +  MmioWrite32 (UartBase + UMCR, McrBits);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +/**
> +  Retrieve the status of the control bits on a serial device.
> +
> +  @param[out]  Control   Status of the control bits on a serial device :
> +
> +                         . EFI_SERIAL_DATA_CLEAR_TO_SEND,
> +                           EFI_SERIAL_DATA_SET_READY,
> +                           EFI_SERIAL_RING_INDICATE,
> +                           EFI_SERIAL_CARRIER_DETECT,
> +                           EFI_SERIAL_REQUEST_TO_SEND,
> +                           EFI_SERIAL_DATA_TERMINAL_READY
> +                           are all related to the DTE (Data Terminal Equipment)
> +                           and DCE (Data Communication Equipment) modes of
> +                           operation of the serial device.
> +                         . EFI_SERIAL_INPUT_BUFFER_EMPTY : equal to one if the
> +                           receive buffer is empty, 0 otherwise.
> +                         . EFI_SERIAL_OUTPUT_BUFFER_EMPTY : equal to one if the
> +                           transmit buffer is empty, 0 otherwise.
> +                         . EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE : equal to one if
> +                           the hardware loopback is enabled (the ouput feeds the
> +                           receive buffer), 0 otherwise.
> +                         . EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE : equal to one if
> +                           a loopback is accomplished by software, 0 otherwise.
> +                         . EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE : equal to
> +                           one if the hardware flow control based on CTS (Clear
> +                           To Send) and RTS (Ready To Send) control signals is
> +                           enabled, 0 otherwise.
> +
> +  @retval EFI_SUCCESS      The control bits were read from the serial device.
> +
> +**/
> +
> +EFI_STATUS
> +EFIAPI
> +SerialPortGetControl (
> +  OUT  UINT32  *Control
> +  )
> +{
> +  UINT32      MsrRegister;
> +  UINT32      McrRegister;
> +  UINT32      LsrRegister;
> +  UINTN       UartBase;
> +
> +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> +
> +  MsrRegister = MmioRead8 (UartBase + UMSR);
> +  McrRegister = MmioRead8 (UartBase + UMCR);
> +  LsrRegister = MmioRead8 (UartBase + ULSR);
> +
> +  *Control = 0;
> +
> +  if ((MsrRegister & DUART_MSR_CTS) == DUART_MSR_CTS) {
> +    *Control |= EFI_SERIAL_CLEAR_TO_SEND;
> +  }
> +
> +  if ((McrRegister & DUART_MCR_RTS) == DUART_MCR_RTS) {
> +    *Control |= EFI_SERIAL_REQUEST_TO_SEND;
> +  }
> +
> +  if ((LsrRegister & DUART_LSR_TEMT) == DUART_LSR_TEMT) {
> +    *Control |= EFI_SERIAL_OUTPUT_BUFFER_EMPTY;
> +  }
> +
> +  if ((McrRegister & DUART_MCR_AFE) == DUART_MCR_AFE) {
> +    *Control |= EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE;
> +  }
> +
> +  if ((McrRegister & DUART_MCR_LOOP) == DUART_MCR_LOOP) {
> +    *Control |= EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE;
> +  }
> +
> +  return EFI_SUCCESS;
> +}
> +
> +/**
> +  Programmed hardware of Serial port.
> +
> +  @return    Always return EFI_SUCCESS.
> +
> +**/
> +
> +EFI_STATUS
> +EFIAPI
> +SerialPortInitialize (
> +  VOID
> +  )
> +{
> +  UINT64  BaudRate;
> +  UINT32  BaudDivisor;
> +  UINTN   UartBase;
> +
> +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> +  BaudRate = (UINTN)PcdGet64 (PcdUartDefaultBaudRate);
> +
> +  BaudDivisor = CalculateBaudDivisor(BaudRate);
> +
> +  while (!(MmioRead8(UartBase + ULSR) & DUART_LSR_TEMT));
> +
> +  //
> +  // Enable received data available interrupt, setup data format,
> +  // setup baud divisor
> +  //
> +  MmioWrite8(UartBase + UIER, 0x1);
> +  MmioWrite8(UartBase + ULCR, DUART_LCR_BKSE | DUART_LCRVAL);
> +  MmioWrite8(UartBase + UDLB, 0);
> +  MmioWrite8(UartBase + UDMB, 0);
> +  MmioWrite8(UartBase + ULCR, DUART_LCRVAL);
> +  MmioWrite8(UartBase + UMCR, DUART_MCRVAL);
> +  MmioWrite8(UartBase + UFCR, DUART_FCRVAL);
> +  MmioWrite8(UartBase + ULCR, DUART_LCR_BKSE | DUART_LCRVAL);
> +  MmioWrite8(UartBase + UDLB, BaudDivisor & 0xff);
> +  MmioWrite8(UartBase + UDMB, (BaudDivisor >> 8) & 0xff);
> +  MmioWrite8(UartBase + ULCR, DUART_LCRVAL);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +/**
> +  Write data to serial device.
> +
> +  @param  Buffer           Point of data buffer which need to be written.
> +  @param  NumberOfBytes    Number of output bytes which are cached in Buffer.
> +
> +  @retval 0                Write data failed.
> +  @retval !0               Actual number of bytes written to serial device.
> +
> +**/
> +UINTN
> +EFIAPI
> +SerialPortWrite (
> +  IN  UINT8     *Buffer,
> +  IN  UINTN     NumberOfBytes
> +  )
> +{
> +  UINT8   *Final;
> +  UINTN   UartBase;
> +
> +  Final = &Buffer[NumberOfBytes];
> +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> +
> +  while (Buffer < Final) {
> +    while ((MmioRead8(UartBase + ULSR) & DUART_LSR_THRE) == 0);
> +    MmioWrite8(UartBase + UTHR, *Buffer++);

Space before ( [2x]

> +  }
> +
> +  return NumberOfBytes;
> +}
> +
> +/**
> +  Read data from serial device and save the data in buffer.
> +
> +  @param  Buffer           Point of data buffer which need to be written.
> +  @param  NumberOfBytes    Number of output bytes which are cached in Buffer.
> +
> +  @retval 0                Read data failed.
> +  @retval !0               Actual number of bytes read from serial device.
> +
> +**/
> +UINTN
> +EFIAPI
> +SerialPortRead (
> +  OUT UINT8     *Buffer,
> +  IN  UINTN     NumberOfBytes
> +  )
> +{
> +  UINTN   Count;
> +  UINTN   UartBase;
> +
> +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> +
> +  for (Count = 0; Count < NumberOfBytes; Count++, Buffer++) {
> +    /*
> +     * Loop while waiting for a new char(s) to arrive in the
> +     * RxFIFO
> +     */
> +    while ((MmioRead8(UartBase + ULSR) & DUART_LSR_DR) == 0);
> +
> +    *Buffer = MmioRead8(UartBase + URBR);

and here

> +  }
> +
> +  return NumberOfBytes;
> +}
> +
> +/**
> +  Check to see if any data is available to be read from the debug device.
> +
> +  @retval EFI_SUCCESS       At least one byte of data is available to be read
> +  @retval EFI_NOT_READY     No data is available to be read
> +  @retval EFI_DEVICE_ERROR  The serial device is not functioning properly
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +SerialPortPoll (
> +  VOID
> +  )
> +{
> +  UINTN   UartBase;
> +
> +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> +
> +  return ((MmioRead8 (UartBase + ULSR) & DUART_LSR_DR) != 0);
> +}
> +
> +/**
> +  Set new attributes to LS1043a.
> +
> +  @param  BaudRate                The baud rate of the serial device. If the baud rate is not supported,
> +                                  the speed will be reduced down to the nearest supported one and the
> +                                  variable's value will be updated accordingly.
> +  @param  ReceiveFifoDepth        The number of characters the device will buffer on input. If the specified
> +                                  value is not supported, the variable's value will be reduced down to the
> +                                  nearest supported one.
> +  @param  Timeout                 If applicable, the number of microseconds the device will wait
> +                                  before timing out a Read or a Write operation.
> +  @param  Parity                  If applicable, this is the EFI_PARITY_TYPE that is computed or checked
> +                                  as each character is transmitted or received. If the device does not
> +                                  support parity, the value is the default parity value.
> +  @param  DataBits                The number of data bits in each character
> +  @param  StopBits                If applicable, the EFI_STOP_BITS_TYPE number of stop bits per character.
> +                                  If the device does not support stop bits, the value is the default stop
> +                                  bit value.
> +
> +  @retval EFI_SUCCESS             All attributes were set correctly on the serial device.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +SerialPortSetAttributes (
> +  IN  OUT  UINT64              *BaudRate,
> +  IN  OUT  UINT32              *ReceiveFifoDepth,
> +  IN  OUT  UINT32              *Timeout,
> +  IN  OUT  EFI_PARITY_TYPE     *Parity,
> +  IN  OUT  UINT8               *DataBits,
> +  IN  OUT  EFI_STOP_BITS_TYPE  *StopBits
> +  )
> +{
> +  return SerialPortInitialize ();
> +}
> diff --git a/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf b/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
> new file mode 100644
> index 0000000..ead8b40
> --- /dev/null
> +++ b/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
> @@ -0,0 +1,39 @@
> +#  DUartPortLib.inf
> +#
> +#  Component description file for DUartPortLib module
> +#
> +#  Copyright (c) 2013, Freescale Ltd. All rights reserved.
> +#  Copyright 2017 NXP
> +#
> +#  This program and the accompanying materials
> +#  are licensed and made available under the terms and conditions of the BSD License
> +#  which accompanies this distribution.  The full text of the license may be found at
> +#  http://opensource.org/licenses/bsd-license.php
> +#
> +#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +#
> +#
> +
> +[Defines]
> +  INF_VERSION                    = 0x00010005

0x0001001A

> +  BASE_NAME                      = DUartPortLib
> +  FILE_GUID                      = 8ecefc8f-a2c4-4091-b80f-20f7aeb0567f

Please don't reuse the GUID of PL011SerialPortLib.inf

> +  MODULE_TYPE                    = BASE
> +  VERSION_STRING                 = 1.0
> +  LIBRARY_CLASS                  = SerialPortLib
> +
> +[Sources.common]
> +  DUartPortLib.c
> +
> +[LibraryClasses]
> +  PcdLib
> +  SocLib

Please reorder this patch after the patch that introduces SocLib

> +
> +[Packages]
> +  MdeModulePkg/MdeModulePkg.dec
> +  MdePkg/MdePkg.dec
> +
> +[Pcd]
> +  gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase
> +  gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate
> --
> 1.9.1
>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 05/10] Platform/NXP : Add support for DUART library
Posted by Meenakshi Aggarwal 7 years, 1 month ago

> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> Sent: Monday, November 13, 2017 5:59 PM
> To: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D
> <michael.d.kinney@intel.com>; edk2-devel@lists.01.org; Udit Kumar
> <udit.kumar@nxp.com>; Varun Sethi <V.Sethi@nxp.com>
> Subject: Re: [PATCH 05/10] Platform/NXP : Add support for DUART library
> 
> On 7 November 2017 at 14:42, Meenakshi Aggarwal
> <meenakshi.aggarwal@nxp.com> wrote:
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
> > ---
> >  Platform/NXP/Library/DUartPortLib/DUart.h          | 128 ++++++++
> >  Platform/NXP/Library/DUartPortLib/DUartPortLib.c   | 334
> +++++++++++++++++++++
> >  Platform/NXP/Library/DUartPortLib/DUartPortLib.inf |  39 +++
> >  3 files changed, 501 insertions(+)
> >  create mode 100644 Platform/NXP/Library/DUartPortLib/DUart.h
> >  create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.c
> >  create mode 100644 Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
> >
> > diff --git a/Platform/NXP/Library/DUartPortLib/DUart.h
> > b/Platform/NXP/Library/DUartPortLib/DUart.h
> > new file mode 100644
> > index 0000000..907790b
> > --- /dev/null
> > +++ b/Platform/NXP/Library/DUartPortLib/DUart.h
> > @@ -0,0 +1,128 @@
> > +/** DUart.h
> > +*  Header defining the DUART constants (Base addresses, sizes, flags)
> > +*
> > +*  Based on Serial I/O Port library headers available in PL011Uart.h
> > +*
> > +*  Copyright (c) 2011-2012, ARM Limited. All rights reserved.
> > +*  Copyright (c) 2016, Freescale Semiconductor, Inc. All rights reserved.
> > +*  Copyright 2017 NXP
> > +*
> > +*  This program and the accompanying materials
> > +*  are licensed and made available under the terms and conditions of
> > +the BSD License
> > +*  which accompanies this distribution.  The full text of the license
> > +may be found at
> > +*
> >
> +https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fop
> e
> > +nsource.org%2Flicenses%2Fbsd-
> license.php&data=02%7C01%7Cmeenakshi.agg
> >
> +arwal%40nxp.com%7C1ca9bf8b8a2e47ac2b4608d52a921bdc%7C686ea1d3bc
> 2b4c6f
> >
> +a92cd99c5c301635%7C0%7C0%7C636461729393076197&sdata=HSBbU2pCf%
> 2F1MAPr
> > +69WVzzxn6HwycvOQexxGhw%2Fwg4GU%3D&reserved=0
> > +*
> > +*  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> > +BASIS,
> > +*  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> > +*
> > +**/
> > +
> > +#ifndef __DUART_H__
> > +#define __DUART_H__
> > +
> > +// FIFO Control Register
> > +#define DUART_FCR_FIFO_EN          0x01 /* Fifo enable */
> > +#define DUART_FCR_CLEAR_RCVR       0x02 /* Clear the RCVR FIFO */
> > +#define DUART_FCR_CLEAR_XMIT       0x04 /* Clear the XMIT FIFO */
> > +#define DUART_FCR_DMA_SELECT       0x08 /* For DMA applications */
> > +#define DUART_FCR_TRIGGER_MASK     0xC0 /* Mask for the FIFO trigger
> range */
> > +#define DUART_FCR_TRIGGER_1        0x00 /* Mask for trigger set at 1 */
> > +#define DUART_FCR_TRIGGER_4        0x40 /* Mask for trigger set at 4 */
> > +#define DUART_FCR_TRIGGER_8        0x80 /* Mask for trigger set at 8 */
> > +#define DUART_FCR_TRIGGER_14       0xC0 /* Mask for trigger set at 14 */
> > +#define DUART_FCR_RXSR             0x02 /* Receiver soft reset */
> > +#define DUART_FCR_TXSR             0x04 /* Transmitter soft reset */
> > +
> > +// Modem Control Register
> > +#define DUART_MCR_DTR              0x01 /* Reserved  */
> > +#define DUART_MCR_RTS              0x02 /* RTS   */
> > +#define DUART_MCR_OUT1             0x04 /* Reserved */
> > +#define DUART_MCR_OUT2             0x08 /* Reserved */
> > +#define DUART_MCR_LOOP             0x10 /* Enable loopback test mode */
> > +#define DUART_MCR_AFE              0x20 /* AFE (Auto Flow Control) */
> > +#define DUART_MCR_DMA_EN           0x04
> > +#define DUART_MCR_TX_DFR           0x08
> > +
> > +// Line Control Register
> > +/*
> > +* Note: if the word length is 5 bits (DUART_LCR_WLEN5), then setting
> > +* DUART_LCR_STOP will select 1.5 stop bits, not 2 stop bits.
> > +*/
> > +#define DUART_LCR_WLS_MSK          0x03 /* character length select mask
> */
> > +#define DUART_LCR_WLS_5            0x00 /* 5 bit character length */
> > +#define DUART_LCR_WLS_6            0x01 /* 6 bit character length */
> > +#define DUART_LCR_WLS_7            0x02 /* 7 bit character length */
> > +#define DUART_LCR_WLS_8            0x03 /* 8 bit character length */
> > +#define DUART_LCR_STB              0x04 /* # stop Bits, off=1, on=1.5 or 2) */
> > +#define DUART_LCR_PEN              0x08 /* Parity eneble */
> > +#define DUART_LCR_EPS              0x10 /* Even Parity Select */
> > +#define DUART_LCR_STKP             0x20 /* Stick Parity */
> > +#define DUART_LCR_SBRK             0x40 /* Set Break */
> > +#define DUART_LCR_BKSE             0x80 /* Bank select enable */
> > +#define DUART_LCR_DLAB             0x80 /* Divisor latch access bit */
> > +
> > +// Line Status Register
> > +#define DUART_LSR_DR               0x01 /* Data ready */
> > +#define DUART_LSR_OE               0x02 /* Overrun */
> > +#define DUART_LSR_PE               0x04 /* Parity error */
> > +#define DUART_LSR_FE               0x08 /* Framing error */
> > +#define DUART_LSR_BI               0x10 /* Break */
> > +#define DUART_LSR_THRE             0x20 /* Xmit holding register empty */
> > +#define DUART_LSR_TEMT             0x40 /* Xmitter empty */
> > +#define DUART_LSR_ERR              0x80 /* Error */
> > +
> > +// Modem Status Register
> > +#define DUART_MSR_DCTS             0x01 /* Delta CTS */
> > +#define DUART_MSR_DDSR             0x02 /* Reserved */
> > +#define DUART_MSR_TERI             0x04 /* Reserved */
> > +#define DUART_MSR_DDCD             0x08 /* Reserved */
> > +#define DUART_MSR_CTS              0x10 /* Clear to Send */
> > +#define DUART_MSR_DSR              0x20 /* Reserved */
> > +#define DUART_MSR_RI               0x40 /* Reserved */
> > +#define DUART_MSR_DCD              0x80 /* Reserved */
> > +
> > +// Interrupt Identification Register
> > +#define DUART_IIR_NO_INT           0x01 /* No interrupts pending */
> > +#define DUART_IIR_ID               0x06 /* Mask for the interrupt ID */
> > +#define DUART_IIR_MSI              0x00 /* Modem status interrupt */
> > +#define DUART_IIR_THRI             0x02 /* Transmitter holding register
> empty */
> > +#define DUART_IIR_RDI              0x04 /* Receiver data interrupt */
> > +#define DUART_IIR_RLSI             0x06 /* Receiver line status interrupt */
> > +
> > +//  Interrupt Enable Register
> > +#define DUART_IER_MSI              0x08 /* Enable Modem status interrupt */
> > +#define DUART_IER_RLSI             0x04 /* Enable receiver line status
> interrupt */
> > +#define DUART_IER_THRI             0x02 /* Enable Transmitter holding
> register int. */
> > +#define DUART_IER_RDI              0x01 /* Enable receiver data interrupt */
> > +
> > +// LCR defaults
> > +#define DUART_LCR_8N1              0x03
> > +#define DUART_LCRVAL               DUART_LCR_8N1          /* 8 data, 1 stop, no
> parity */
> > +#define DUART_MCRVAL               (DUART_MCR_DTR | \
> > +                                   DUART_MCR_RTS)         /* RTS/DTR */
> > +#define DUART_FCRVAL               (DUART_FCR_FIFO_EN | \
> > +                                   DUART_FCR_RXSR |    \
> > +                                   DUART_FCR_TXSR)        /* Clear & enable FIFOs */
> > +
> > +#define URBR         0x0
> > +#define UTHR         0x0
> > +#define UDLB         0x0
> > +#define UDMB         0x1
> > +#define UIER         0x1
> > +#define UIIR         0x2
> > +#define UFCR         0x2
> > +#define UAFR         0x2
> > +#define ULCR         0x3
> > +#define UMCR         0x4
> > +#define ULSR         0x5
> > +#define UMSR         0x6
> > +#define USCR         0x7
> > +#define UDSR         0x10
> > +
> > +extern
> > +UINT32
> > +CalculateBaudDivisor (
> > +  IN  UINT64 BaudRate
> > +  );
> > +
> > +#endif /* __DUART_H__ */
> > diff --git a/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
> > b/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
> > new file mode 100644
> > index 0000000..99766d1
> > --- /dev/null
> > +++ b/Platform/NXP/Library/DUartPortLib/DUartPortLib.c
> > @@ -0,0 +1,334 @@
> > +/** DuartPortLib.c
> > +  DUART (NS16550) library functions
> > +
> > +  Based on Serial I/O Port library functions available in
> > + PL011SerialPortLib.c
> > +
> > +  Copyright (c) 2008 - 2010, Apple Inc. All rights reserved.<BR>
> > + Copyright (c) 2012 - 2013, ARM Ltd. All rights reserved.<BR>
> > + Copyright (c) 2016, Freescale Semiconductor, Inc. All rights reserved.
> > +  Copyright 2017 NXP
> > +
> > +  This program and the accompanying materials  are licensed and made
> > + available under the terms and conditions of the BSD License  which
> > + accompanies this distribution.  The full text of the license may be
> > + found at
> > +
> > +
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fop
> > + ensource.org%2Flicenses%2Fbsd-
> license.php&data=02%7C01%7Cmeenakshi.a
> > +
> ggarwal%40nxp.com%7C1ca9bf8b8a2e47ac2b4608d52a921bdc%7C686ea1d3b
> c2b4
> > +
> c6fa92cd99c5c301635%7C0%7C0%7C636461729393076197&sdata=HSBbU2pCf
> %2F1
> > + MAPr69WVzzxn6HwycvOQexxGhw%2Fwg4GU%3D&reserved=0
> > +
> > +  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> > + BASIS,  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND,
> EITHER EXPRESS OR IMPLIED.
> > +
> > +**/
> > +
> > +#include <Base.h>
> > +#include <Library/IoLib.h>
> > +#include <Library/PcdLib.h>
> > +#include <Library/SerialPortLib.h>
> > +
> > +#include "DUart.h"
> > +
> > +STATIC CONST UINT32 mInvalidControlBits =
> (EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE | \
> > +
> > +EFI_SERIAL_DATA_TERMINAL_READY);
> > +
> > +/**
> > +  Assert or deassert the control signals on a serial port.
> > +  The following control signals are set according their bit settings :
> > +  . Request to Send
> > +  . Data Terminal Ready
> > +
> > +  @param[in]  Control   The following bits are taken into account :
> > +                        . EFI_SERIAL_REQUEST_TO_SEND : assert/deassert the
> > +                          "Request To Send" control signal if this bit is
> > +                          equal to one/zero.
> > +                        . EFI_SERIAL_DATA_TERMINAL_READY : assert/deassert
> > +                          the "Data Terminal Ready" control signal if this
> > +                          bit is equal to one/zero.
> > +                        . EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE :
> enable/disable
> > +                          the hardware loopback if this bit is equal to
> > +                          one/zero.
> > +                        . EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE : not
> supported.
> > +                        . EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE : enable/
> > +                          disable the hardware flow control based on CTS (Clear
> > +                          To Send) and RTS (Ready To Send) control signals.
> > +
> > +  @retval  EFI_SUCCESS      The new control bits were set on the device.
> > +  @retval  EFI_UNSUPPORTED  The device does not support this
> operation.
> > +
> > +**/
> > +EFI_STATUS
> > +EFIAPI
> > +SerialPortSetControl (
> > +  IN  UINT32  Control
> > +  )
> > +{
> > +  UINT32  McrBits;
> > +  UINTN   UartBase;
> > +
> > +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> > +
> > +  if (Control & (mInvalidControlBits)) {
> > +    return EFI_UNSUPPORTED;
> > +  }
> > +
> > +  McrBits = MmioRead8 (UartBase + UMCR);
> > +
> > +  if (Control & EFI_SERIAL_REQUEST_TO_SEND) {
> > +    McrBits |= DUART_MCR_RTS;
> > +  } else {
> > +    McrBits &= ~DUART_MCR_RTS;
> > +  }
> > +
> > +  if (Control & EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE) {
> > +    McrBits |= DUART_MCR_LOOP;
> > +  } else {
> > +    McrBits &= ~DUART_MCR_LOOP;
> > +  }
> > +
> > +  if (Control & EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE) {
> > +    McrBits |= DUART_MCR_AFE;
> > +  } else {
> > +    McrBits &= ~DUART_MCR_AFE;
> > +  }
> > +
> > +  MmioWrite32 (UartBase + UMCR, McrBits);
> > +
> > +  return EFI_SUCCESS;
> > +}
> > +
> > +/**
> > +  Retrieve the status of the control bits on a serial device.
> > +
> > +  @param[out]  Control   Status of the control bits on a serial device :
> > +
> > +                         . EFI_SERIAL_DATA_CLEAR_TO_SEND,
> > +                           EFI_SERIAL_DATA_SET_READY,
> > +                           EFI_SERIAL_RING_INDICATE,
> > +                           EFI_SERIAL_CARRIER_DETECT,
> > +                           EFI_SERIAL_REQUEST_TO_SEND,
> > +                           EFI_SERIAL_DATA_TERMINAL_READY
> > +                           are all related to the DTE (Data Terminal Equipment)
> > +                           and DCE (Data Communication Equipment) modes of
> > +                           operation of the serial device.
> > +                         . EFI_SERIAL_INPUT_BUFFER_EMPTY : equal to one if the
> > +                           receive buffer is empty, 0 otherwise.
> > +                         . EFI_SERIAL_OUTPUT_BUFFER_EMPTY : equal to one if the
> > +                           transmit buffer is empty, 0 otherwise.
> > +                         . EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE : equal to one
> if
> > +                           the hardware loopback is enabled (the ouput feeds the
> > +                           receive buffer), 0 otherwise.
> > +                         . EFI_SERIAL_SOFTWARE_LOOPBACK_ENABLE : equal to one
> if
> > +                           a loopback is accomplished by software, 0 otherwise.
> > +                         . EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE : equal
> to
> > +                           one if the hardware flow control based on CTS (Clear
> > +                           To Send) and RTS (Ready To Send) control signals is
> > +                           enabled, 0 otherwise.
> > +
> > +  @retval EFI_SUCCESS      The control bits were read from the serial
> device.
> > +
> > +**/
> > +
> > +EFI_STATUS
> > +EFIAPI
> > +SerialPortGetControl (
> > +  OUT  UINT32  *Control
> > +  )
> > +{
> > +  UINT32      MsrRegister;
> > +  UINT32      McrRegister;
> > +  UINT32      LsrRegister;
> > +  UINTN       UartBase;
> > +
> > +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> > +
> > +  MsrRegister = MmioRead8 (UartBase + UMSR);  McrRegister =
> MmioRead8
> > + (UartBase + UMCR);  LsrRegister = MmioRead8 (UartBase + ULSR);
> > +
> > +  *Control = 0;
> > +
> > +  if ((MsrRegister & DUART_MSR_CTS) == DUART_MSR_CTS) {
> > +    *Control |= EFI_SERIAL_CLEAR_TO_SEND;  }
> > +
> > +  if ((McrRegister & DUART_MCR_RTS) == DUART_MCR_RTS) {
> > +    *Control |= EFI_SERIAL_REQUEST_TO_SEND;  }
> > +
> > +  if ((LsrRegister & DUART_LSR_TEMT) == DUART_LSR_TEMT) {
> > +    *Control |= EFI_SERIAL_OUTPUT_BUFFER_EMPTY;  }
> > +
> > +  if ((McrRegister & DUART_MCR_AFE) == DUART_MCR_AFE) {
> > +    *Control |= EFI_SERIAL_HARDWARE_FLOW_CONTROL_ENABLE;
> > +  }
> > +
> > +  if ((McrRegister & DUART_MCR_LOOP) == DUART_MCR_LOOP) {
> > +    *Control |= EFI_SERIAL_HARDWARE_LOOPBACK_ENABLE;
> > +  }
> > +
> > +  return EFI_SUCCESS;
> > +}
> > +
> > +/**
> > +  Programmed hardware of Serial port.
> > +
> > +  @return    Always return EFI_SUCCESS.
> > +
> > +**/
> > +
> > +EFI_STATUS
> > +EFIAPI
> > +SerialPortInitialize (
> > +  VOID
> > +  )
> > +{
> > +  UINT64  BaudRate;
> > +  UINT32  BaudDivisor;
> > +  UINTN   UartBase;
> > +
> > +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);  BaudRate =
> > + (UINTN)PcdGet64 (PcdUartDefaultBaudRate);
> > +
> > +  BaudDivisor = CalculateBaudDivisor(BaudRate);
> > +
> > +  while (!(MmioRead8(UartBase + ULSR) & DUART_LSR_TEMT));
> > +
> > +  //
> > +  // Enable received data available interrupt, setup data format,  //
> > + setup baud divisor  //  MmioWrite8(UartBase + UIER, 0x1);
> > + MmioWrite8(UartBase + ULCR, DUART_LCR_BKSE | DUART_LCRVAL);
> > + MmioWrite8(UartBase + UDLB, 0);  MmioWrite8(UartBase + UDMB, 0);
> > + MmioWrite8(UartBase + ULCR, DUART_LCRVAL);  MmioWrite8(UartBase
> +
> > + UMCR, DUART_MCRVAL);  MmioWrite8(UartBase + UFCR,
> DUART_FCRVAL);
> > + MmioWrite8(UartBase + ULCR, DUART_LCR_BKSE | DUART_LCRVAL);
> > + MmioWrite8(UartBase + UDLB, BaudDivisor & 0xff);
> > + MmioWrite8(UartBase + UDMB, (BaudDivisor >> 8) & 0xff);
> > + MmioWrite8(UartBase + ULCR, DUART_LCRVAL);
> > +
> > +  return EFI_SUCCESS;
> > +}
> > +
> > +/**
> > +  Write data to serial device.
> > +
> > +  @param  Buffer           Point of data buffer which need to be written.
> > +  @param  NumberOfBytes    Number of output bytes which are cached in
> Buffer.
> > +
> > +  @retval 0                Write data failed.
> > +  @retval !0               Actual number of bytes written to serial device.
> > +
> > +**/
> > +UINTN
> > +EFIAPI
> > +SerialPortWrite (
> > +  IN  UINT8     *Buffer,
> > +  IN  UINTN     NumberOfBytes
> > +  )
> > +{
> > +  UINT8   *Final;
> > +  UINTN   UartBase;
> > +
> > +  Final = &Buffer[NumberOfBytes];
> > +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> > +
> > +  while (Buffer < Final) {
> > +    while ((MmioRead8(UartBase + ULSR) & DUART_LSR_THRE) == 0);
> > +    MmioWrite8(UartBase + UTHR, *Buffer++);
> 
> Space before ( [2x]
> 
OK, will update all patches.
> > +  }
> > +
> > +  return NumberOfBytes;
> > +}
> > +
> > +/**
> > +  Read data from serial device and save the data in buffer.
> > +
> > +  @param  Buffer           Point of data buffer which need to be written.
> > +  @param  NumberOfBytes    Number of output bytes which are cached in
> Buffer.
> > +
> > +  @retval 0                Read data failed.
> > +  @retval !0               Actual number of bytes read from serial device.
> > +
> > +**/
> > +UINTN
> > +EFIAPI
> > +SerialPortRead (
> > +  OUT UINT8     *Buffer,
> > +  IN  UINTN     NumberOfBytes
> > +  )
> > +{
> > +  UINTN   Count;
> > +  UINTN   UartBase;
> > +
> > +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> > +
> > +  for (Count = 0; Count < NumberOfBytes; Count++, Buffer++) {
> > +    /*
> > +     * Loop while waiting for a new char(s) to arrive in the
> > +     * RxFIFO
> > +     */
> > +    while ((MmioRead8(UartBase + ULSR) & DUART_LSR_DR) == 0);
> > +
> > +    *Buffer = MmioRead8(UartBase + URBR);
> 
> and here
> 
> > +  }
> > +
> > +  return NumberOfBytes;
> > +}
> > +
> > +/**
> > +  Check to see if any data is available to be read from the debug device.
> > +
> > +  @retval EFI_SUCCESS       At least one byte of data is available to be read
> > +  @retval EFI_NOT_READY     No data is available to be read
> > +  @retval EFI_DEVICE_ERROR  The serial device is not functioning
> > + properly
> > +
> > +**/
> > +BOOLEAN
> > +EFIAPI
> > +SerialPortPoll (
> > +  VOID
> > +  )
> > +{
> > +  UINTN   UartBase;
> > +
> > +  UartBase = (UINTN)PcdGet64 (PcdSerialRegisterBase);
> > +
> > +  return ((MmioRead8 (UartBase + ULSR) & DUART_LSR_DR) != 0); }
> > +
> > +/**
> > +  Set new attributes to LS1043a.
> > +
> > +  @param  BaudRate                The baud rate of the serial device. If the baud
> rate is not supported,
> > +                                  the speed will be reduced down to the nearest
> supported one and the
> > +                                  variable's value will be updated accordingly.
> > +  @param  ReceiveFifoDepth        The number of characters the device will
> buffer on input. If the specified
> > +                                  value is not supported, the variable's value will be
> reduced down to the
> > +                                  nearest supported one.
> > +  @param  Timeout                 If applicable, the number of microseconds the
> device will wait
> > +                                  before timing out a Read or a Write operation.
> > +  @param  Parity                  If applicable, this is the EFI_PARITY_TYPE that is
> computed or checked
> > +                                  as each character is transmitted or received. If the device
> does not
> > +                                  support parity, the value is the default parity value.
> > +  @param  DataBits                The number of data bits in each character
> > +  @param  StopBits                If applicable, the EFI_STOP_BITS_TYPE number
> of stop bits per character.
> > +                                  If the device does not support stop bits, the value is the
> default stop
> > +                                  bit value.
> > +
> > +  @retval EFI_SUCCESS             All attributes were set correctly on the serial
> device.
> > +
> > +**/
> > +EFI_STATUS
> > +EFIAPI
> > +SerialPortSetAttributes (
> > +  IN  OUT  UINT64              *BaudRate,
> > +  IN  OUT  UINT32              *ReceiveFifoDepth,
> > +  IN  OUT  UINT32              *Timeout,
> > +  IN  OUT  EFI_PARITY_TYPE     *Parity,
> > +  IN  OUT  UINT8               *DataBits,
> > +  IN  OUT  EFI_STOP_BITS_TYPE  *StopBits
> > +  )
> > +{
> > +  return SerialPortInitialize ();
> > +}
> > diff --git a/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
> > b/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
> > new file mode 100644
> > index 0000000..ead8b40
> > --- /dev/null
> > +++ b/Platform/NXP/Library/DUartPortLib/DUartPortLib.inf
> > @@ -0,0 +1,39 @@
> > +#  DUartPortLib.inf
> > +#
> > +#  Component description file for DUartPortLib module # #  Copyright
> > +(c) 2013, Freescale Ltd. All rights reserved.
> > +#  Copyright 2017 NXP
> > +#
> > +#  This program and the accompanying materials #  are licensed and
> > +made available under the terms and conditions of the BSD License #
> > +which accompanies this distribution.  The full text of the license
> > +may be found at #
> >
> +https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fop
> e
> > +nsource.org%2Flicenses%2Fbsd-
> license.php&data=02%7C01%7Cmeenakshi.agg
> >
> +arwal%40nxp.com%7C1ca9bf8b8a2e47ac2b4608d52a921bdc%7C686ea1d3bc
> 2b4c6f
> >
> +a92cd99c5c301635%7C0%7C0%7C636461729393076197&sdata=HSBbU2pCf%
> 2F1MAPr
> > +69WVzzxn6HwycvOQexxGhw%2Fwg4GU%3D&reserved=0
> > +#
> > +#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> > +BASIS, #  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND,
> EITHER EXPRESS OR IMPLIED.
> > +#
> > +#
> > +
> > +[Defines]
> > +  INF_VERSION                    = 0x00010005
> 
> 0x0001001A
> 
Will correct this.

> > +  BASE_NAME                      = DUartPortLib
> > +  FILE_GUID                      = 8ecefc8f-a2c4-4091-b80f-20f7aeb0567f
> 
> Please don't reuse the GUID of PL011SerialPortLib.inf
> 
Will regenerate all GUIDS.

> > +  MODULE_TYPE                    = BASE
> > +  VERSION_STRING                 = 1.0
> > +  LIBRARY_CLASS                  = SerialPortLib
> > +
> > +[Sources.common]
> > +  DUartPortLib.c
> > +
> > +[LibraryClasses]
> > +  PcdLib
> > +  SocLib
> 
> Please reorder this patch after the patch that introduces SocLib
> 
Yes, my fault here

> > +
> > +[Packages]
> > +  MdeModulePkg/MdeModulePkg.dec
> > +  MdePkg/MdePkg.dec
> > +
> > +[Pcd]
> > +  gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase
> > +  gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate
> > --
> > 1.9.1
> >
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel