Re: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-31 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: Justen, Jordan L Sent: Thursday, May 28, 2015 11:14 PM To: edk2-devel@lists.sourceforge.net Cc: Justen, Jordan L; Gao, Liming Subject: [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions For *.asm and *.s, there have been

Re: [edk2] [PATCH] MdeModulePkg DxeCore: Add debug message to assist TPL related issues.

2015-05-31 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: Zeng, Star Sent: Friday, May 29, 2015 5:27 PM To: edk2-devel@lists.sourceforge.net Cc: Yao, Jiewen; Gao, Liming Subject: [PATCH] MdeModulePkg DxeCore: Add debug message to assist TPL related issues. The debug message is to print the curr

Re: [edk2] [PATCH] Bump default driver supported efi version to 2.5

2015-05-31 Thread Tian, Feng
Could you split it to 2 patches? One is specific for MdeModulePkg and another is for OpromPkg. Then different package owners could review your changes individually. MdeModulePkg change is good: Reviewed-by: Feng Tian I will help check in the MdeModulePkg part. Thanks Feng -Original Messa

Re: [edk2] NVMe question

2015-05-31 Thread Tian, Feng
Hi, Baraneedharan Why I removed these two fields initialization is because NVMe spec 7.6.1 doesn't say it's mandatory. Do you see any real impact on this? Thanks Feng From: Anbazhagan, Baraneedharan [mailto:anbazha...@hp.com] Sent: Saturday, May 30, 2015 06:21 To: Tian, Feng; edk2-devel@lists.