[edk2] [PATCH] MdeModulePkg/NvmExpressDxe: fix error status override

2017-12-02 Thread Heyi Guo
Commit f6b139b added return status handling to PciIo->Mem.Write. However, the second status handling will override EFI_DEVICE_ERROR returned in this branch: // // Check the NVMe cmd execution result // if (Status != EFI_TIMEOUT) { if ((Cq->Sct == 0) && (Cq->Sc == 0)) { Status =

Re: [edk2] [PATCH] ShellPkg/ShellPkg.dec: Change comments for PcdShellLibAutoInitialize

2017-12-02 Thread Ni, Ruiyu
That's fine. Faster than me in the review response. > -Original Message- > From: Carsey, Jaben > Sent: Saturday, December 2, 2017 1:00 AM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Subject: RE: [PATCH] ShellPkg/ShellPkg.dec: Change comments for >

Re: [edk2] [PATCH v2 3/8] OvmfPkg: conditionally disable reserved memory varstore emulation at build

2017-12-02 Thread Ard Biesheuvel
On 1 December 2017 at 11:41, Laszlo Ersek wrote: > On 12/01/17 11:53, Ard Biesheuvel wrote: >> On 1 December 2017 at 10:52, Laszlo Ersek wrote: >>> On 12/01/17 09:51, Ard Biesheuvel wrote: On 30 November 2017 at 16:30, Laszlo Ersek

Re: [edk2] [PATCH 2/2] QuarkPlatformPkg: Use DpDynamicCommand to replace PerformancePkg/dp

2017-12-02 Thread Ni, Ruiyu
Kelly, I pushed the commit, following your below recommendation. > -Original Message- > From: Steele, Kelly > Sent: Saturday, December 2, 2017 1:18 AM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Cc: Kinney, Michael D > Subject: RE: [PATCH