Thanks for the understanding. I will post the patch to revert the change.
Regards, Ray >-----Original Message----- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo >Ersek >Sent: Wednesday, March 23, 2016 4:53 PM >To: Ni, Ruiyu <ruiyu...@intel.com> >Cc: Tian, Feng <feng.t...@intel.com>; Ard Biesheuvel ><ard.biesheu...@linaro.org>; edk2-devel@lists.01.org ><edk2-de...@ml01.01.org>; Heyi Guo <heyi....@linaro.org>; Ryan Harkin ><ryan.har...@linaro.org>; Zeng, Star ><star.z...@intel.com> >Subject: Re: [edk2] [PATCH v2 2/3] MdeModulePkg/SerialDxe: Set FIFO depth with >PCD > >On 03/23/16 09:33, Ni, Ruiyu wrote: >> Laszlo, >> Since the patch below lets SerialDxe driver use the new introduced >> PCD gEfiMdePkgTokenSpaceGuid.PcdUartDefaultReceiveFifoDepth. >> Would you mind to revert the check in @ 31ae446b >> --> MdeModulePkg: TerminalDxe: select the UART's default receive FIFO depth? > >If I understand correctly, this series increases the terminal polling >rate, so that cursor movement escape sequences (and other burst-like >sequences) can be processed even with a receive fifo depth of 1. Hence >31ae446b should become unnecessary. > >If Ryan and Ard (CC'd), who tested 31ae446b originally, can also test >your revert of 31ae446b -- on top of this series from Heyi --, and the >cursor keys work with 31ae446b reverted, then I don't mind, sure. > >Thanks >Laszlo > >> >> Regards, >> Ray >> >> >>> -----Original Message----- >>> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Heyi >>> Guo >>> Sent: Thursday, March 17, 2016 10:37 PM >>> To: edk2-devel@lists.01.org >>> Cc: Heyi Guo <heyi....@linaro.org>; Tian, Feng <feng.t...@intel.com>; Zeng, >>> Star <star.z...@intel.com> >>> Subject: [edk2] [PATCH v2 2/3] MdeModulePkg/SerialDxe: Set FIFO depth with >>> PCD >>> >>> Set UART receive FIFO depth with PCD instead of fixed number "1". >>> The default value of PCD is also 1, so it makes no difference for >>> platforms which do not explicitly set this PCD. >>> >>> Contributed-under: TianoCore Contribution Agreement 1.0 >>> Signed-off-by: Heyi Guo <heyi....@linaro.org> >>> Cc: Feng Tian <feng.t...@intel.com> >>> Cc: Star Zeng <star.z...@intel.com> >>> --- >>> MdeModulePkg/Universal/SerialDxe/SerialDxe.inf | 9 +++++---- >>> MdeModulePkg/Universal/SerialDxe/SerialIo.c | 3 ++- >>> 2 files changed, 7 insertions(+), 5 deletions(-) >>> >>> diff --git a/MdeModulePkg/Universal/SerialDxe/SerialDxe.inf >>> b/MdeModulePkg/Universal/SerialDxe/SerialDxe.inf >>> index 164060b..a1453bd 100644 >>> --- a/MdeModulePkg/Universal/SerialDxe/SerialDxe.inf >>> +++ b/MdeModulePkg/Universal/SerialDxe/SerialDxe.inf >>> @@ -41,10 +41,11 @@ >>> gEfiDevicePathProtocolGuid ## PRODUCES >>> >>> [Pcd] >>> - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate ## CONSUMES >>> - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultDataBits ## CONSUMES >>> - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity ## CONSUMES >>> - gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits ## CONSUMES >>> + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultBaudRate ## CONSUMES >>> + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultDataBits ## CONSUMES >>> + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultParity ## CONSUMES >>> + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits ## CONSUMES >>> + gEfiMdePkgTokenSpaceGuid.PcdUartDefaultReceiveFifoDepth ## CONSUMES >>> >>> [Depex] >>> TRUE >>> diff --git a/MdeModulePkg/Universal/SerialDxe/SerialIo.c >>> b/MdeModulePkg/Universal/SerialDxe/SerialIo.c >>> index f5b3064..d2383e5 100644 >>> --- a/MdeModulePkg/Universal/SerialDxe/SerialIo.c >>> +++ b/MdeModulePkg/Universal/SerialDxe/SerialIo.c >>> @@ -236,7 +236,7 @@ SerialReset ( >>> // >>> // Set the Serial I/O mode >>> // >>> - This->Mode->ReceiveFifoDepth = 1; >>> + This->Mode->ReceiveFifoDepth = PcdGet16 >>> (PcdUartDefaultReceiveFifoDepth); >>> This->Mode->Timeout = 1000 * 1000; >>> This->Mode->BaudRate = PcdGet64 (PcdUartDefaultBaudRate); >>> This->Mode->DataBits = (UINT32) PcdGet8 (PcdUartDefaultDataBits); >>> @@ -508,6 +508,7 @@ SerialDxeInitialize ( >>> mSerialIoMode.DataBits = (UINT32) PcdGet8 (PcdUartDefaultDataBits); >>> mSerialIoMode.Parity = (UINT32) PcdGet8 (PcdUartDefaultParity); >>> mSerialIoMode.StopBits = (UINT32) PcdGet8 (PcdUartDefaultStopBits); >>> + mSerialIoMode.ReceiveFifoDepth = PcdGet16 >>> (PcdUartDefaultReceiveFifoDepth); >>> mSerialDevicePath.Uart.BaudRate = PcdGet64 (PcdUartDefaultBaudRate); >>> mSerialDevicePath.Uart.DataBits = PcdGet8 (PcdUartDefaultDataBits); >>> mSerialDevicePath.Uart.Parity = PcdGet8 (PcdUartDefaultParity); >>> -- >>> 2.7.0 >>> >>> _______________________________________________ >>> edk2-devel mailing list >>> edk2-devel@lists.01.org >>> https://lists.01.org/mailman/listinfo/edk2-devel >> _______________________________________________ >> edk2-devel mailing list >> edk2-devel@lists.01.org >> https://lists.01.org/mailman/listinfo/edk2-devel >> > >_______________________________________________ >edk2-devel mailing list >edk2-devel@lists.01.org >https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
