Jiaxin,
can you review this patch?
Regards,
Ray
>-Original Message-
>From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo
>Ersek
>Sent: Friday, February 24, 2017 5:58 AM
>To: edk2-devel-01
>Cc: Ni, Ruiyu ; Tomas Hoger
>Subject: [edk2
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Zeng, Star
>Sent: Friday, February 24, 2017 10:10 AM
>To: edk2-devel@lists.01.org
>Cc: Zeng, Star ; Kinney, Michael D
>; Gao, Liming
>; Carsey, Jaben ; Ni, Ruiyu
>
>Subject: [PATCH 3/3] ShellPkg
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Fu
> Siyuan
> Sent: Wednesday, February 22, 2017 4:15 PM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu
> Subject: [edk2] [Patch] Nt32P
st return Invalid Parameter for all cases?
Thanks/Ray
> -Original Message-
> From: Yao, Jiewen
> Sent: Wednesday, February 22, 2017 2:02 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Gao, Liming ; Fu, Siyuan
> Subject: RE: [PATCH 04/11] MdePkg/BaseLib: Add
>
Can we use EFI_STATUS variable to receive RETURN_STATUS type of value?
Thanks/Ray
From: Yao, Jiewen
Sent: Wednesday, February 22, 2017 1:28 PM
To: Ni, Ruiyu ; edk2-devel@lists.01.org
Subject: RE: [PATCH 10/11] SignedCapsulePkg/IniParsingLib: Use AsciiStrToGuid
in BaseLib
Hi
I do not suggest we
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Hao
> Wu
> Sent: Tuesday, February 21, 2017 7:36 PM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Ni, Ruiyu ; Yao,
> Jiewen
> Su
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Gao, Liming
> Sent: Wednesday, February 15, 2017 4:40 PM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu
> Subject: [Patch] MdeModulePkg UefiBootManagerLib: Correct usages of
> GUID an
Jaben,
"-ec" should be followed by a ID.
The patch fixes when "-ec" is followed by string other than ID, no error is
reported.
Regards,
Ray
>-Original Message-
>From: Carsey, Jaben
>Sent: Saturday, February 11, 2017 1:06 AM
>To: Ni, Ruiyu ; edk2-devel
Ersek [mailto:ler...@redhat.com]
> Sent: Wednesday, February 8, 2017 6:45 PM
> To: Ni, Ruiyu ; edk2-de...@ml01.01.org
> Cc: Fan, Jeff ; Gao, Liming
> Subject: Re: [edk2] [PATCH v4 6/6] MdePkg/Pci22.h: Deprecate out-of-Spec
> IncompatiblePciDevice macros
>
> On 02/
Leo,
Did you forget to add the BmDmaLib implementation in the patch?
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Leo Duran
> Sent: Saturday, January 14, 2017 6:14 AM
> To: edk2-devel@lists.01.org
> Cc: Tian, Feng ; brijesh.si.
I will change the commit message as Laszlo suggested.
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Wang, Sunny (HPS SW)
> Sent: Wednesday, February 8, 2017 5:38 PM
> To: Laszlo Ersek ; Ni, Ruiyu ;
> edk2-
Sunny,
It's impossible for the Handle which doesn't have SimpleFileSystem or BlockIo
installed.
Thanks/Ray
> -Original Message-
> From: Wang, Sunny (HPS SW) [mailto:sunnyw...@hpe.com]
> Sent: Wednesday, February 8, 2017 4:13 PM
> To: Ni, Ruiyu ; edk2-devel@lists.
Behalf Of Laszlo
Ersek
Sent: Wednesday, February 8, 2017 1:43 AM
To: Ni, Ruiyu ; edk2-de...@ml01.01.org
Cc: Fan, Jeff
Subject: Re: [edk2] [PATCH v3 6/6] MdePkg/Pci22.h: Deprecate out-of-Spec
IncompatiblePciDevice macros
On 02/07/17 04:33, Ruiyu Ni wrote:
> DEVICE_ID_NOCARE is defined as 0xF
Mike,
OK I will change to MAX_UINT64.
Thanks/Ray
> -Original Message-
> From: Kinney, Michael D
> Sent: Tuesday, February 7, 2017 1:55 AM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org; Kinney, Michael
> D
> Cc: Tian, Feng ; Fan, Jeff
> Subject: RE: [edk2] [PATCH
yes. That's a good idea to move the definitions to local.
Regards,
Ray
From: Gao, Liming
Sent: Friday, February 3, 2017 4:36 PM
To: Ni, Ruiyu ; Laszlo Ersek ;
edk2-devel@lists.01.org
Cc: Fan, Jeff
Subject: RE: [edk2] [PATCH 1/5] MdePkg/Pci22.h: Deprecate out-of-Spec
IncompatiblePciD
s/Pci/IncompatiblePciDeviceSupportDxe build will break.
What's your opinion about this driver?
Thanks/Ray
> -Original Message-
> From: Laszlo Ersek [mailto:ler...@redhat.com]
> Sent: Friday, February 3, 2017 4:13 PM
> To: Gao, Liming ; Ni, Ruiyu ;
> edk2-devel@lists.01.org
> Cc: Fan, Jeff
&g
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Thomas
>Palmer
>Sent: Wednesday, February 1, 2017 11:35 AM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [edk2]
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Thomas Huth [mailto:th...@redhat.com]
> Sent: Wednesday, January 25, 2017 6:02 PM
> To: edk2-de...@ml01.01.org
> Cc: Ni, Ruiyu
> Subject: [PATCH] OptionRomPkg: Remove superfluous return statement
>
>
Tim,
I agree with you.
If the text to display has to be in English. Embedding them in C file makes
code more readable.
Thanks/Ray
> -Original Message-
> From: Tim Lewis [mailto:tim.le...@insyde.com]
> Sent: Tuesday, January 24, 2017 4:52 AM
> To: Carsey, Jaben ; Ni, Ruiyu ;
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Zeng, Star
> Sent: Sunday, January 22, 2017 5:25 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Carsey, Jaben ; Zeng, Star
>
> Subject: RE: [PATCH 3/3] ShellPkg SmbiosView: Add decoding of SMBIOS
Star,
Why some strings are hardcoded in C file while some are defined in UNI file?
What's the rule?
Thanks/Ray
> -Original Message-
> From: Zeng, Star
> Sent: Sunday, January 22, 2017 4:18 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star ; Ni, Ruiyu ;
> Carsey, J
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Gao, Liming
> Sent: Friday, January 20, 2017 5:29 PM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu
> Subject: [Patch] Nt32Pkg FDF: Move StatusCode Handler run earlier in DXE
> phase
>
> Add Stat
Laszlo,
Thanks for your reply.
I will remove the comments in C function.
Regards,
Ray
From: Laszlo Ersek [mailto:ler...@redhat.com]
Sent: Thursday, January 12, 2017 6:42 PM
To: Ni, Ruiyu ; edk2-de...@ml01.01.org
Cc: Tian, Feng ; Zeng, Star
Subject: Re: [edk2] [PATCH 4/8] MdeModulePkg
Leo,
1. What does "Bm" stand for in "BmDmaLib"?
2. Include/Library/BmDmaLib.h says
" DMA abstraction library APIs. Based on PCI IO protocol DMA abstractions."
I think that means the BmDmaLib library class knows the PCI IO concept.
If so, why not directly use the EFI_PCI_ROT_BRIDGE_IO_PR
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Wu, Hao A
> Sent: Thursday, January 12, 2017 10:45 AM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Ni, Ruiyu
> Subject: [PATCH] MdeModulePkg/TerminalDxe: Refine
> SetTerminalDevicePath
>
> Pas
Mike,
I have no other concerns now.
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Kinney, Michael D
> Sent: Wednesday, January 11, 2017 5:10 AM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org; Kinney, Michael
> D
> Cc: Carsey, Jaben
> Subject: RE: [e
ists.01.org] On Behalf Of
> Hao Wu
> Sent: Wednesday, January 11, 2017 10:01 AM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Ni, Ruiyu ; Ard
> Biesheuvel
> Subject: [edk2] [PATCH] MdeModulePkg/NonDiscoverable: Compare
> SIZE_4GB with address type
>
> Refine the codes to c
DisplayDriverModelHandle() always return SUCCESS so we change the return type
to VOID.
Thanks/Ray
> -Original Message-
> From: Carsey, Jaben
> Sent: Monday, January 9, 2017 11:37 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Chen, Chen A ; Carsey, Jaben
>
> Su
Jaben,
Yes we will add the API to convert string to GUID. After the library API is
added, the local implementation will be removed.
Thanks/Ray
> -Original Message-
> From: Carsey, Jaben
> Sent: Monday, January 9, 2017 11:41 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org; Kin
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Carsey, Jaben
> Sent: Monday, January 9, 2017 11:42 PM
> To: Kinney, Michael D ; edk2-
> de...@lists.01.org
> Cc: Ni, Ruiyu ; Carsey, Jaben
> Subject: RE: [Patch] Nt32Pkg/WinNtSimpleFileSystemDxe: F
Send to correct mail address.
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Monday, January 9, 2017 10:19 AM
> To: edk2-devel@lists.01.org
> Cc: Dandan Bi ; Gao, Liming
> Subject: [edk2] [PATCH] MdeModulePkg/Bo
L."
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Michael Kinney
> Sent: Sunday, January 8, 2017 4:26 AM
> To: edk2-devel@lists.01.org
> Cc: Carsey, Jaben ; Ni, Ruiyu ;
> Kinney, Michael D
> Subjec
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Wu, Hao A
>Sent: Friday, January 6, 2017 4:53 PM
>To: edk2-devel@lists.01.org
>Cc: Wu, Hao A ; Ni, Ruiyu
>Subject: [PATCH] MdeModulePkg/UefiBootManagerLib: Remove redundant type cast
>
>The type of r
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Wu, Hao A
>Sent: Friday, January 6, 2017 4:51 PM
>To: edk2-devel@lists.01.org
>Cc: Wu, Hao A ; Ni, Ruiyu
>Subject: [PATCH] FatPkg: Explicitly declare FAT_CLUSTER_SPECIAL as unsigned
>type
>
>
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Zeng, Star
> Sent: Monday, December 26, 2016 3:24 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star ; Ni, Ruiyu ;
> Kinney, Michael D ; Tian, Feng
>
> Subject: [PATCH V2] MdeModulePkg ConSplitterD
t.
Thanks/Ray
From: Zeng, Star
Sent: Saturday, December 24, 2016 9:53 PM
To: Ni, Ruiyu ; edk2-devel@lists.01.org
Cc: Kinney, Michael D ; Tian, Feng
; Zeng, Star
Subject: Re: [edk2] [PATCH] MdeModulePkg ConSplitterDxe: Support toggle state
sync
Ray,
On 2016/12/23 19:53, Ni, Ruiyu wro
e ToggleStateSyncKeyNotify() has to use mConIn. Seems like a gap of
UEFI spec that doesn't have VOID * Context as the 2nd parameter of the hot key
notification function.
Regards,
Ray
>-Original Message-
>From: Zeng, Star
>Sent: Thursday, December 22, 2016 9:43 PM
>To: ed
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Zeng, Star
>Sent: Friday, December 23, 2016 4:13 PM
>To: edk2-devel@lists.01.org
>Cc: Zeng, Star ; Ni, Ruiyu ; Kinney,
>Michael D ;
>Tian, Feng ; Fan, Jeff ; Michael
>Zimmermann
>Subject: [PATCH
ists.01.org
>Cc: Ni, Ruiyu ; Tian, Feng ; Kinney,
>Michael D ;
>Michael Zimmermann ; Fan, Jeff ;
>Zeng, Star
>Subject: [edk2] [PATCH 0/5] Execute key notify function at TPL_CALLBACK
>
>Current implementation of keyboard driver executes key notify
>function in TimerHandler at
: Friday, December 23, 2016 3:05 PM
To: Ni, Ruiyu ; edk2-devel@lists.01.org
Cc: Kinney, Michael D ; Tian, Feng
; Zeng, Star
Subject: RE: [edk2] [PATCH 1/5] MdeModulePkg UsbKbDxe: Execute key notify func
at TPL_CALLBACK
Ray,
Good comment, it can make the code more clear, I agree it.
Thanks
age-
>From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Star
>Zeng
>Sent: Thursday, December 22, 2016 9:45 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu ; Kinney, Michael D
>; Tian, Feng ;
>Zeng, Star
>Subject: [edk2] [PATCH 1/5] MdeModulePkg U
0) == 0);
Regards,
Ray
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Carsey,
Jaben
Sent: Thursday, December 22, 2016 12:23 AM
To: Ni, Ruiyu ; edk2-devel@lists.01.org
Cc: Carsey, Jaben ; Chen, Chen A
Subject: Re: [edk2] [PATCH 2/2] ShellPkg/cd: Fix "cd" to
Jiewen,
I think Shell.efi should only support commands defined by Shell spec.
All other commands should be provided in standalone application way.
Jaben,
What's your idea?
Thanks/Ray
From: Yao, Jiewen
Sent: Tuesday, December 20, 2016 12:50 PM
To: Carsey, Jaben ; Ni, Ruiyu ;
Evan Lloy
d by the APP INF
file.
Since now we provide both library and application for acpiview, how about put
it in new location
ShellPkg/Application/AcpiView directory?
What's your option? If you agree, I can help to create the APP INF.
Thanks/Ray
> -----Original Message-
> From: Car
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Friday, December 16, 2016 10:26 AM
>To: edk2-devel@lists.01.org
>Cc: Ard Biesheuvel ; Ni, Ruiyu
>Subject: [patch 3/3] MdeModulePkg/NonDiscoverablePciDevice: Make variable
>definition f
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Friday, December 16, 2016 10:26 AM
>To: edk2-devel@lists.01.org
>Cc: Ard Biesheuvel ; Ni, Ruiyu
>Subject: [patch 2/3] MdeModulePkg/NonDiscoverablePciDeviceDxe: Add comments
>for
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Friday, December 16, 2016 10:26 AM
>To: edk2-devel@lists.01.org
>Cc: Ard Biesheuvel ; Ni, Ruiyu
>Subject: [patch 1/3] MdeModulePkg/NonDiscoverablePciDeviceDxe: Fix VS2010/2012
>bui
Reviewed-by: Ruiyu Ni
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ard Biesheuvel
> Sent: Friday, December 9, 2016 11:05 PM
> To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; Ni, Ruiyu
>
> Cc: Tian, Feng
Boaz,
Could you please verify this patch in your failed system?
I cannot reproduce the issue in my environment.
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Wednesday, December 14, 2016 6:01 PM
> To: edk2-devel
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Wu, Jiaxin
> Sent: Wednesday, December 14, 2016 4:05 PM
> To: edk2-devel@lists.01.org
> Cc: Long, Qin ; Ni, Ruiyu ; Ye, Ting
> ; Fu, Siyuan ; Zhang, Lubo
> ; Thomas Palmer ; Yao,
> Jiewen ; Wu, Jiaxin
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Wu, Jiaxin
> Sent: Wednesday, December 14, 2016 3:34 PM
> To: edk2-devel@lists.01.org
> Cc: Long, Qin ; Ni, Ruiyu ; Ye, Ting
> ; Fu, Siyuan ; Zhang, Lubo
> ; Thomas Palmer ; Wu,
> Jiaxin
> Sub
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Dandan Bi
> Sent: Monday, December 12, 2016 11:19 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu
> Subject: [edk2] [patch] FatPkg/E
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Wu, Hao A
> Sent: Monday, December 12, 2016 9:27 AM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Ard Biesheuvel
> ; Ni, Ruiyu
> Subject: [PATCH] MdeModulePkg/NonDiscoverablePciDev: Fix type
>
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Friday, December 9, 2016 10:21 AM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [PATCH v2 2/8] FatPkg/EnhancedFatDxe: Use global variable replace
>static local variable
>
done.
Regards,
Ray
From: Carsey, Jaben
Sent: Friday, December 9, 2016 6:07 AM
To: Tapan Shah ; edk2-devel@lists.01.org
Cc: Ni, Ruiyu ; Carsey, Jaben
Subject: RE: [PATCH] ShellPkg: Add missing header line for SFO flag in 'cls'
command
Reviewed-by: Jaben Carsey
mailto:jaben.car...
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [patch 1/8] FatPkg\EnhancedFatDxe: Avoid Non-Boolean type uses as
>Boolean
>
>Cc: R
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [patch 3/8] FatPkg\EnhancedFatDxe: Make function prototype align with
>definition
>
>
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [patch 4/8] FatPkg\EnhancedFatDxe: Make the variable name follow rule
>
>Cc: Ruiyu Ni
>C
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [patch 5/8] FatPkg\EnhancedFatDxe: Use typedef for complex type
>
>Cc: Ruiyu Ni
>C
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [patch 7/8] FatPkg\EnhancedFatDxe: Add comments for functions
>
>Cc: Ruiyu Ni
>C
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
>Subject: [patch 8/8] FatPkg: Fix format issues in dec/inf/dsc files
>
>Cc: Ruiyu Ni
>C
Dandan,
Could you please create a global array mMonthDays?
UINT8 mMonthDays[] = { 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 };
Regards,
Ray
>-Original Message-
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu
&
Regards,
Ray
>-Original Message-
>From: Leif Lindholm [mailto:leif.lindh...@linaro.org]
>Sent: Wednesday, November 30, 2016 10:07 PM
>To: Ni, Ruiyu
>Cc: Ard Biesheuvel ; edk2-devel@lists.01.org; Gao,
>Liming ;
>af...@apple.com; Kinney, Michael D ;
>m...@
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Leif Lindholm [mailto:leif.lindh...@linaro.org]
>Sent: Thursday, December 1, 2016 11:53 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu ; Justen, Jordan L
>; Andrew Fish ; Laszlo
>Ersek ; Kinney, Michae
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Fan, Jeff
> Sent: Tuesday, November 29, 2016 3:17 PM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu ; Yao, Jiewen ;
> Kinney, Michael D
> Subject: [PATCH] SourceLevelDebugPkg: Avoid to re-init IDT table
Comments in NonCoherentPciIoAllocateBuffer().
Thanks/Ray
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Friday, November 25, 2016 11:42 PM
> To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; Gao, Liming
> ; af...@apple
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Friday, November 25, 2016 11:42 PM
> To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; Gao, Liming
> ; af...@apple.com; Ni, Ruiyu ;
> Kin
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Friday, November 25, 2016 11:42 PM
> To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; Gao, Liming
> ; af...@apple.com; Ni, Ruiyu ;
> Kin
11:42 PM
> To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; Gao, Liming
> ; af...@apple.com; Ni, Ruiyu ;
> Kinney, Michael D ; m...@semihalf.com; Tian,
> Feng
> Cc: Ard Biesheuvel
> Subject: [PATCH v4 3/5] MdeModulePkg: implement generic PCI I/O driver
> for non-
1. Can you add "PCI" keyword into the protocol name?
e.g.: EDKII_NON_DISCOVERABLE_PCI_DEVICE_PROTOCOL_GUID
>>>
>>> No. This protocol does not describe pci devices, and it is a
>>> peculiarity of the
>>> edk2 driver stack that some non-pci devices can only
Regards,
Ray
>-Original Message-
>From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard
>Biesheuvel
>Sent: Friday, November 18, 2016 12:59 PM
>To: Ni, Ruiyu
>Cc: Kinney, Michael D ; edk2-devel@lists.01.org;
>Gao, Liming ;
>af...@apple.com;
Regards,
Ray
>-Original Message-
>From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
>Sent: Thursday, November 17, 2016 6:43 PM
>To: Ni, Ruiyu
>Cc: Kinney, Michael D ; edk2-devel@lists.01.org;
>Gao, Liming ;
>af...@apple.com; Leif Lindholm
>Subject:
Regards,
Ray
>-Original Message-
>From: Zeng, Star
>Sent: Thursday, November 17, 2016 6:55 PM
>To: Ni, Ruiyu ; edk2-devel@lists.01.org
>Cc: sean.bro...@microsoft.com; Zeng, Star
>Subject: RE: [edk2] [PATCH] PcAtChipsetPkg/PcRtc: Handle NULL table entry in
>RSDT/X
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ard Biesheuvel
> Sent: Thursday, November 17, 2016 2:07 PM
> To: Ni, Ruiyu
> Cc: Kinney, Michael D ; edk2-
> de...@lists.01.org; Gao, Liming ; af...@apple
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Wu, Hao A
> Sent: Thursday, November 17, 2016 2:32 PM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Ni, Ruiyu ; Gao,
> Liming
> Subject: [PATCH v2] MdePkg BaseLib: API PathRemoveLastItem not handle
Thanks/Ray
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Thursday, November 17, 2016 12:59 AM
> To: edk2-devel@lists.01.org; Gao, Liming ; Ni, Ruiyu
> ; Kinney, Michael D
> Cc: af...@apple.com; m...@semihalf.com; leif.lindh.
Ard,
I have two comments in below.
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Leif Lindholm
> Sent: Thursday, November 17, 2016 1:49 AM
> To: Ard Biesheuvel
> Cc: Ni, Ruiyu ; edk2-devel@lists.01.org;
>
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Bi, Dandan
> Sent: Thursday, November 17, 2016 10:26 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu
> Subject: [patch 2/4] Nt32Pkg: Add BaseCryptLib/OpensslLib/IntrinsicLib
>
> DriverSampleDxe w
Ok. I didn't realize your PCI access is different from the standard way. I
agree the PciLib suggestion is not good.
发自我的 iPhone
> 在 2016年11月16日,下午7:43,Ard Biesheuvel 写道:
>
>> On 15 November 2016 at 11:30, Ard Biesheuvel
>> wrote:
>>> On 15 November 2016 at
,
Ray
>-Original Message-
>From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Wang,
>Sunny (HPS SW)
>Sent: Wednesday, November 16, 2016 5:08 PM
>To: Ni, Ruiyu ; edk2-devel@lists.01.org
>Cc: Haskell, Darrell ; Lin, Jie
>Subject: Re: [edk2] [PATCH 3/3]
Ard,
Can you check whether PciLib can be used to replace the implementation in
NonDiscoverablePciDeviceIo.c?
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ard Biesheuvel
> Sent: Thursday, November 3, 2016 7:42 PM
> To: edk2-deve
Please have Edkii prefix for implementation specific protocols/GUIDs.
gNonDiscoverableDeviceProtocolGuid -> gEdkiiNonDiscoverableDeviceProtocolGuid
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ard Biesheuvel
> Sent: Thursday,
Ard,
For the below protocol structure, it assumes that the non-discoverable
PCI device contains only one BAR and the type is limited to USB/AHCI/SD/UFS/NVME
devices.
Could we have more types of such kind of device in future?
Could we have a device that contains multiple BAR?
Could we have a device
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Monday, November 14, 2016 1:26 PM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH] PcAtChipsetPkg/PcRtc: Handle NULL table entry in
> RSDT/XSDT
>
> The ACPI
Behalf Of
> Shah, Tapan
> Sent: Saturday, November 12, 2016 4:43 AM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org; Phillips, Chris J
> (Plano, TX)
> Cc: Carsey, Jaben ; Chen, Chen A
>
> Subject: Re: [edk2] [PATCH] ShellPkg/dmpstore: Support "-sfo"
>
> Ray,
>
Comments below.
Thanks/Ray
> -Original Message-
> From: Shah, Tapan [mailto:tapands...@hpe.com]
> Sent: Saturday, November 12, 2016 1:21 AM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Chen, Chen A ; Carsey, Jaben
>
> Subject: RE: [PATCH] ShellPkg/dmpstore: Su
Sorry for that! :(
I really forgot it.
发自我的 iPhone
> 在 2016年11月11日,下午7:16,Laszlo Ersek 写道:
>
> Ray,
>
>> On 11/08/16 14:04, Laszlo Ersek wrote:
>>> On 11/08/16 13:29, Ruiyu Ni wrote:
>>> Contributed-under: TianoCore Contribution Agreement 1.0
>>> Signed-off-by: Ruiyu Ni
>>> Cc: Jordan Justen
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Feng Tian
> Sent: Thursday, November 10, 2016 1:17 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming ; Jin, Eric
> Subject: [edk2] [patch] MdePkg: Fix spec m
Hao,
Thanks for making this helpful updates.
Reviewed-by: Ruiyu Ni
>-Original Message-
>From: Wu, Hao A
>Sent: Friday, November 11, 2016 4:27 PM
>To: edk2-devel@lists.01.org
>Cc: Wu, Hao A ; Ni, Ruiyu
>Subject: [PATCH 3/3] DuetPkg: Add POSTBUILD in DSC files
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Wu, Hao A
>Sent: Friday, November 11, 2016 4:27 PM
>To: edk2-devel@lists.01.org
>Cc: Wu, Hao A ; Ni, Ruiyu
>Subject: [PATCH 1/3] DuetPkg: Resolve white-space issues for post-build
>scripts &am
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Wu, Hao A
>Sent: Friday, November 11, 2016 4:27 PM
>To: edk2-devel@lists.01.org
>Cc: Wu, Hao A ; Ni, Ruiyu
>Subject: [PATCH 2/3] DuetPkg: Use 'echo off' in BATCH script files
>
>Instead of
Bell,
FreePool (Blt) purposely frees the pool of Image.Bitmap, which was returned
from previous loop. It's useful.
Thanks/Ray
> -Original Message-
> From: Song, BinX
> Sent: Thursday, November 10, 2016 1:42 PM
> To: Gao, Liming ; edk2-devel@lists.01.org
> Cc: Ni,
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Bi, Dandan
> Sent: Thursday, November 10, 2016 3:40 PM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu
> Subject: [patch] ShellPkg: Fix the coding style issue
>
> Cc: Ruiyu Ni
> Contributed-un
Thanks/Ray
> -Original Message-
> From: Laszlo Ersek [mailto:ler...@redhat.com]
> Sent: Saturday, November 5, 2016 12:48 AM
> To: Ni, Ruiyu ; Gao, Liming ;
> edk2-devel@lists.01.org
> Subject: Re: [edk2] [PATCH 0/4] Defer 3rd party images loading to after
> EndOfDx
No.
The open source platform patch will be sent out later.
Regards,
Ray
From: Gao, Liming
Sent: Friday, November 4, 2016 1:14 PM
To: Ni, Ruiyu ; edk2-devel@lists.01.org
Subject: RE: [edk2] [PATCH 0/4] Defer 3rd party images loading to after EndOfDxe
Ray:
Seemly, PlatformBdsLib library
Reviewed-by: Ruiyu Ni
Regards,
Ray
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo
Ersek
Sent: Thursday, October 27, 2016 4:17 PM
To: edk2-devel-01 ; Ni, Ruiyu
Subject: Re: [edk2] [PATCH 11/47] DuetPkg/DuetBdsLib: rebase to ARRAY_SIZE()
Ray,
On 10/26/16 21:04
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Laszlo Ersek [mailto:ler...@redhat.com]
>Sent: Thursday, October 27, 2016 3:05 AM
>To: edk2-devel-01
>Cc: Ni, Ruiyu
>Subject: [PATCH 14/47] FatPkg/EnhancedFatDxe: rebase to ARRAY_SIZE()
>
>Cc: Ruiy
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Laszlo Ersek [mailto:ler...@redhat.com]
>Sent: Thursday, October 27, 2016 3:05 AM
>To: edk2-devel-01
>Cc: Tian, Feng ; Ni, Ruiyu ; Zeng,
>Star
>Subject: [PATCH 27/47] MdeModulePkg/Logo: rebase to ARR
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Laszlo Ersek [mailto:ler...@redhat.com]
>Sent: Thursday, October 27, 2016 3:05 AM
>To: edk2-devel-01
>Cc: Tian, Feng ; Ni, Ruiyu ; Zeng,
>Star
>Subject: [PATCH 18/47] MdeModulePkg/PciBusDxe: rebase
Reviewed-by: Ruiyu Ni
Regards,
Ray
>-Original Message-
>From: Laszlo Ersek [mailto:ler...@redhat.com]
>Sent: Thursday, October 27, 2016 3:05 AM
>To: edk2-devel-01
>Cc: Tian, Feng ; Ni, Ruiyu ; Zeng,
>Star
>Subject: [PATCH 28/47] MdeModulePkg/BdsDxe: rebase to ARR
701 - 800 of 1176 matches
Mail list logo