[edk2] [patch 3/8] MdeModulePkg: Update comments in SimpleTextInEx according to UEFI 2.7

2017-06-18 Thread Dandan Bi
Cc: Star Zeng Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Bus/Usb/UsbKbDxe/EfiKey.c | 12 MdeModulePkg/Bus/Usb/UsbKbDxe/EfiKey.h | 12 MdeModulePkg/Universal/Co

[edk2] [patch 1/8] MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Star Zeng Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/SimpleTextInEx.h | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/MdePkg/Include/Protocol/SimpleTextInEx.h

[edk2] [patch 4/8] MdePkg/HiiImageDecoder.h: Remove the unnecessary comments

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/HiiImageDecoder.h | 9 - 1 file changed, 9 deletions(-) diff --git a/MdePkg/Include/Protocol/HiiImageDecoder.h b/MdePkg/Include/Protocol/HiiImageDec

[edk2] [patch 6/8] MdePkg: update comments in ResetSystem() according to UEFI2.7

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Library/UefiRuntimeLib.h | 7 +++ MdePkg/Include/Uefi/UefiSpec.h | 3 +++ 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/MdePkg/Include/Li

[edk2] [patch 0/8] Update function comments according to UEFI2.7 spec

2017-06-18 Thread Dandan Bi
This patch serial is to update fucntion comments accoding to new UEFI spec. Dandan Bi (8): MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec IntelFrameworkModulePkg: Update comments in SimpleTextInEx MdeModulePkg: Update comments in SimpleTextInEx according to UEFI 2.7 MdeP

[edk2] [patch 5/8] MdePkg: Update comments in EFI_MEMORY_DESCRIPTOR according to UEFI2.7

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Uefi/UefiSpec.h | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSp

[edk2] [patch 2/8] IntelFrameworkModulePkg: Update comments in SimpleTextInEx

2017-06-18 Thread Dandan Bi
Update comments in SimpleTextInEx according to UEFI 2.7. Cc: Star Zeng Cc: Ruiyu Ni Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Bus/Isa/Ps2KeyboardDxe/Ps2KbdTextIn.c| 10 +++--- .../Bus/Isa/Ps2KeyboardDxe/Ps2Keyboar

[edk2] [patch 7/8] MdePkg: Clean ACPI 2.0 characters in UEFI spec

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/PciIo.h | 25 - MdePkg/Include/Protocol/PciRootBridgeIo.h | 18 +- 2 files changed, 21 insertions(+), 22 de

[edk2] [patch 8/8] MdeModulePkg: Clean ACPI 2.0 characters in UEFI spec

2017-06-18 Thread Dandan Bi
Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c | 6 +++--- MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h | 6 +++--- MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridge.h | 6 +

Re: [edk2] [patch 2/8] IntelFrameworkModulePkg: Update comments in SimpleTextInEx

2017-06-18 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Bi, Dandan > Sent: Monday, June 19, 2017 9:21 AM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Ni, Ruiyu ; Fan, > Jeff > Subject: [patch 2/8] IntelFrameworkModulePkg: Update comments in > SimpleTextInEx > > Update comment

Re: [edk2] [patch 4/8] MdePkg/HiiImageDecoder.h: Remove the unnecessary comments

2017-06-18 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Bi, Dandan > Sent: Monday, June 19, 2017 9:21 AM > To: edk2-devel@lists.01.org > Cc: Gao, Liming ; Ni, Ruiyu > Subject: [patch 4/8] MdePkg/HiiImageDecoder.h: Remove the unnecessary > comments > > Cc: Liming Gao > Cc: Ruiyu

Re: [edk2] [patch 3/8] MdeModulePkg: Update comments in SimpleTextInEx according to UEFI 2.7

2017-06-18 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Bi, Dandan > Sent: Monday, June 19, 2017 9:21 AM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Ni, Ruiyu > Subject: [patch 3/8] MdeModulePkg: Update comments in SimpleTextInEx > according to UEFI 2.7 > > Cc: Star Zeng >

Re: [edk2] [patch 1/8] MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec

2017-06-18 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Bi, Dandan > Sent: Monday, June 19, 2017 9:21 AM > To: edk2-devel@lists.01.org > Cc: Gao, Liming ; Zeng, Star ; > Ni, Ruiyu > Subject: [patch 1/8] MdePkg/SimpleTextInEx: Update comments according > to UEFI 2.7 spec > > Cc: L

Re: [edk2] [patch 6/8] MdePkg: update comments in ResetSystem() according to UEFI2.7

2017-06-18 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Bi, Dandan > Sent: Monday, June 19, 2017 9:22 AM > To: edk2-devel@lists.01.org > Cc: Gao, Liming ; Ni, Ruiyu > Subject: [patch 6/8] MdePkg: update comments in ResetSystem() according > to UEFI2.7 > > Cc: Liming Gao > Cc: Ru

Re: [edk2] [patch 1/8] MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec

2017-06-18 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Bi, Dandan Sent: Monday, June 19, 2017 9:21 AM To: edk2-devel@lists.01.org Cc: Gao, Liming ; Zeng, Star ; Ni, Ruiyu Subject: [patch 1/8] MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec Cc: Liming Gao Cc: Star Zeng Cc

Re: [edk2] [patch 2/8] IntelFrameworkModulePkg: Update comments in SimpleTextInEx

2017-06-18 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Bi, Dandan Sent: Monday, June 19, 2017 9:21 AM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Ni, Ruiyu ; Fan, Jeff Subject: [patch 2/8] IntelFrameworkModulePkg: Update comments in SimpleTextInEx Update comments in SimpleTextInEx according

Re: [edk2] [patch 3/8] MdeModulePkg: Update comments in SimpleTextInEx according to UEFI 2.7

2017-06-18 Thread Zeng, Star
Dandan, The change also needs to be in Ps2KeyboardDxe in MdeModulePkg, right? Please double confirm that. Thanks, Star -Original Message- From: Bi, Dandan Sent: Monday, June 19, 2017 9:21 AM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Ni, Ruiyu Subject: [patch 3/8] MdeModulePkg: Update

Re: [edk2] [patch 5/8] MdePkg: Update comments in EFI_MEMORY_DESCRIPTOR according to UEFI2.7

2017-06-18 Thread Zeng, Star
Please update the " Physical address of the first byte of the memory region. " to " Physical address of the first byte in the memory region. " instead of removing it. Thanks, Star -Original Message- From: Bi, Dandan Sent: Monday, June 19, 2017 9:21 AM To: edk2-devel@lists.01.org Cc: Gao,

Re: [edk2] [patch 7/8] MdePkg: Clean ACPI 2.0 characters in UEFI spec

2017-06-18 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Bi, Dandan Sent: Monday, June 19, 2017 9:22 AM To: edk2-devel@lists.01.org Cc: Gao, Liming ; Zeng, Star Subject: [patch 7/8] MdePkg: Clean ACPI 2.0 characters in UEFI spec Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contri

Re: [edk2] [patch 3/8] MdeModulePkg: Update comments in SimpleTextInEx according to UEFI 2.7

2017-06-18 Thread Bi, Dandan
Hi Star, Yes. That's right. I missed the Ps2KeyboardDxe in MdeModulePkg. I will send new patch to cover this driver. Thank you for your comments. Regards, Dandan -Original Message- From: Zeng, Star Sent: Monday, June 19, 2017 10:41 AM To: Bi, Dandan ; edk2-devel@lists.01.org Cc: Ni, R

Re: [edk2] [patch 8/8] MdeModulePkg: Clean ACPI 2.0 characters in UEFI spec

2017-06-18 Thread Zeng, Star
Dandan, Seemingly, there are some missing changes in RootBridgeIoConfiguration(), please double confirm that. Thanks, Star -Original Message- From: Bi, Dandan Sent: Monday, June 19, 2017 9:22 AM To: edk2-devel@lists.01.org Cc: Zeng, Star Subject: [patch 8/8] MdeModulePkg: Clean ACPI 2.

[edk2] [PATCH] MdePkg: Add EFI UFS Device Config Protocol definitions

2017-06-18 Thread Hao Wu
Cc: Star Zeng Cc: Michael D Kinney Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Protocol/UfsDeviceConfig.h | 143 ++ MdePkg/MdePkg.dec | 3 + 2 files changed, 146 insertions

[edk2] [PATCH v2 4/8] MdePkg/HiiImageDecoder.h: Remove the unnecessary comments

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/HiiImageDecoder.h | 9 - 1 file changed, 9 deletions(-) diff --git a/MdePkg/Include/Protocol/HiiImageDecoder.h b/MdePkg/Include/Protocol/HiiImageDec

[edk2] [PATCH v2 3/8] MdeModulePkg: Update comments in SimpleTextInEx according to UEFI 2.7

2017-06-18 Thread Dandan Bi
v2: Add some missing changes Ps2KeyboardDxe. Cc: Star Zeng Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2KbdTextIn.c | 9 ++--- MdeModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2Keyboard.h |

[edk2] [PATCH v2 5/8] MdePkg: Update comments in EFI_MEMORY_DESCRIPTOR according to UEFI2.7

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Uefi/UefiSpec.h | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSp

[edk2] [PATCH v2 0/8] Update function comments according to UEFI2.7 spec

2017-06-18 Thread Dandan Bi
v2: Add some missing changes in previous pacth 3 and patch 8. This patch serial is to update fucntion comments accoding to new UEFI spec. Dandan Bi (8): MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec IntelFrameworkModulePkg: Update comments in SimpleTextInEx MdeModulePkg

[edk2] [PATCH v2 6/8] MdePkg: update comments in ResetSystem() according to UEFI2.7

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Library/UefiRuntimeLib.h | 7 +++ MdePkg/Include/Uefi/UefiSpec.h | 3 +++ 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/MdePkg/Include/Li

[edk2] [PATCH v2 1/8] MdePkg/SimpleTextInEx: Update comments according to UEFI 2.7 spec

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Star Zeng Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/SimpleTextInEx.h | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/MdePkg/Include/Protocol/SimpleTextInEx.h

[edk2] [PATCH v2 7/8] MdePkg: Clean ACPI 2.0 characters in UEFI spec

2017-06-18 Thread Dandan Bi
Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdePkg/Include/Protocol/PciIo.h | 25 - MdePkg/Include/Protocol/PciRootBridgeIo.h | 18 +- 2 files changed, 21 insertions(+), 22 de

[edk2] [PATCH v2 2/8] IntelFrameworkModulePkg: Update comments in SimpleTextInEx

2017-06-18 Thread Dandan Bi
Update comments in SimpleTextInEx according to UEFI 2.7. Cc: Star Zeng Cc: Ruiyu Ni Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- .../Bus/Isa/Ps2KeyboardDxe/Ps2KbdTextIn.c| 10 +++--- .../Bus/Isa/Ps2KeyboardDxe/Ps2Keyboar

[edk2] [PATCH v2 8/8] MdeModulePkg: Clean ACPI 2.0 characters in UEFI spec

2017-06-18 Thread Dandan Bi
v2: Add some missing changes. Cc: Liming Gao Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c | 6 +++--- MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h | 6 +++--- MdeModulePkg

Re: [edk2] [PATCH v2 0/8] Update function comments according to UEFI2.7 spec

2017-06-18 Thread Zeng, Star
Reviewed-by: Star Zeng to this patch series. -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Dandan Bi Sent: Monday, June 19, 2017 12:46 PM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH v2 0/8] Update function comments according to UEFI2.7 s

Re: [edk2] [PATCH v2 4/8] MdePkg/HiiImageDecoder.h: Remove the unnecessary comments

2017-06-18 Thread Zeng, Star
Just found below comments in MdePkg.dec, should it be removed also? ## ## In UEFI 2.6 spec,this guid value is duplicate with ## EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_GUID. Now update this guid value to ## avoid the duplicate guid issue. So its value is not consistent with ## UEFI spec definiti

Re: [edk2] [PATCH v2 4/8] MdePkg/HiiImageDecoder.h: Remove the unnecessary comments

2017-06-18 Thread Bi, Dandan
Yes. I will remove it before checking in the codes. Thanks. Regards, Dandan -Original Message- From: Zeng, Star Sent: Monday, June 19, 2017 12:55 PM To: Bi, Dandan ; edk2-devel@lists.01.org Cc: Ni, Ruiyu ; Gao, Liming ; Zeng, Star Subject: RE: [edk2] [PATCH v2 4/8] MdePkg/HiiImageDecode

[edk2] [PATCH] MdePkg/BluetoothConfig: Update comments for GetData/GetRemoteData

2017-06-18 Thread Hao Wu
Updates the GetData and GetRemoteData services description comments according to latest UEFI spec. Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Protocol/BluetoothConfig.h | 12 ++-- 1 file changed, 6 insertions(+), 6 delet

Re: [edk2] [PATCH] MdePkg/BluetoothConfig: Update comments for GetData/GetRemoteData

2017-06-18 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Wu, Hao A > Sent: Monday, June 19, 2017 1:29 PM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A ; Ni, Ruiyu > Subject: [PATCH] MdePkg/BluetoothConfig: Update comments for > GetData/GetRemoteData > > Updates the GetData and Get

Re: [edk2] [PATCH] MdePkg: Add EFI UFS Device Config Protocol definitions

2017-06-18 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Wu, Hao A Sent: Monday, June 19, 2017 12:39 PM To: edk2-devel@lists.01.org Cc: Wu, Hao A ; Zeng, Star ; Kinney, Michael D ; Gao, Liming Subject: [PATCH] MdePkg: Add EFI UFS Device Config Protocol definitions Cc: Star Zeng Cc: Michael D