Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Build ASL files before C files

2019-11-03 Thread Bob Feng
Hi Pierre, How do you compile 'SsdtSerialPortTemplate.asl' with the '-tc' to the .HEX file? , based on the build_rule.txt, I think the .asl output file is a .aml file. [Acpi-Source-Language-File] ?.asl, ?.Asl, ?.ASL $(OUTPUT_DIR)(+)${s_dir}(+)${s_base}.aml Thanks,

[edk2-devel] [PATCH 1/4] ShellPkg/SmbiosView: SMBIOS 3.3.0 add PCI gen4 values for type 9

2019-11-03 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2305 System Slots (Type 9): – SMBIOSCR00184: add PCI Express Gen 4 values Cc: Ray Ni Cc: Sai Chaganty Signed-off-by: Zhichao Gao --- .../SmbiosView/QueryTable.c | 24 +++ 1 file changed, 24 insertions(+)

[edk2-devel] [PATCH 3/4] ShellPkg/SmbiosView: SMBIOS 3.3.0 Add value HBM and Die for type 17

2019-11-03 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2305 Memory Device (Type 17): – SMBIOSCR00178: add new memory device type value (HBM) and new form factor value (Die) Cc: Ray Ni Cc: Sai Chaganty Signed-off-by: Zhichao Gao --- .../SmbiosView/QueryTable.c | 12

[edk2-devel] [PATCH 0/4] Update SMBIOS 3.3.0

2019-11-03 Thread Gao, Zhichao
The following changes were made to version 3.2.0 of the document to produce this version: • System Slots (Type 9): – Add PCI Express Gen 4 values – Clarify bus number usage for PCI Express • Memory Device (Type 17): – Add new memory device type value (HBM) and new form factor value (Die) – Update

[edk2-devel] [PATCH 2/4] ShellPkg/SmbiosView: SMBIOS 3.3.0 add support for CXL Flexbus

2019-11-03 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2305 Various: – SMBIOSCR00183: add support for CXL Flexbus Cc: Ray Ni Cc: Sai Chaganty Signed-off-by: Zhichao Gao --- .../UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c| 8 1 file changed, 8 insertions(+) diff --git

[edk2-devel] [PATCH 4/4] ShellPkg/SmbiosView: SMBIOS 3.3.0 Update "Intel persistent memory"

2019-11-03 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2305 Memory Device (Type 17): – SMBIOSCR00179: update the string for Intel persistent memory Cc: Ray Ni Cc: Sai Chaganty Signed-off-by: Zhichao Gao --- .../Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c | 2 +- 1 file changed,

Re: [edk2-devel] [edk2] Smbuslib and SmbusPpi

2019-11-03 Thread Tiger Liu(BJ-RD)
Hi, Liming: Got it. Thanks 发件人: Gao, Liming 发送时间: 2019年11月4日 13:09 收件人: devel@edk2.groups.io; Tiger Liu(BJ-RD) 主题: RE: [edk2-devel] [edk2] Smbuslib and SmbusPpi Tiger: Yes. Quark SmBusLib is the real library instance. Mde SmBusLib bases on PPI. Platform can decide to implement the real

Re: [edk2-devel] [edk2] Smbuslib and SmbusPpi

2019-11-03 Thread Liming Gao
Tiger: Yes. Quark SmBusLib is the real library instance. Mde SmBusLib bases on PPI. Platform can decide to implement the real SmbusLib library instance or provide SmbusPPI implementation. Thanks Liming From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Tiger Liu(BJ-RD)

Re: [edk2-devel] [edk2] Smbuslib and SmbusPpi

2019-11-03 Thread Tiger Liu(BJ-RD)
Hi, Liming: I found two smbuslib implemented in below path: 1. edk2-platforms\Silicon\Intel\QuarkSocPkg\QuarkNorthCluster\Library\SmbusLib I named this smbuslib as quark-smbuslib 2. MdePkg\Library\PeiSmbusLibSmbus2Ppi\ I name this smbuslib as Mde-smbuslib My questions are: 1.

Re: [edk2-devel] [PATCH v1 0/2] Update the SRAT Acpiview parser to ACPI 6.3

2019-11-03 Thread Gao, Zhichao
Exactly. The #2 should be updated. Thanks, Zhichao > -Original Message- > From: Gao, Liming > Sent: Monday, November 4, 2019 11:27 AM > To: Gao, Zhichao ; Sudipto Paul > ; Sami Mujawar ; > Krzysztof Koch ; devel@edk2.groups.io > Cc: Carsey, Jaben ; Ni, Ray ; > Kinney, Michael D ; Matteo

Re: [edk2-devel] How /sys/firmware/fdt getting created

2019-11-03 Thread Prabhakar Kushwaha
Dear Dann Frazier, On Fri, Nov 1, 2019 at 9:55 PM dann frazier wrote: > > On Fri, Nov 01, 2019 at 10:17:30AM +0530, Prabhakar Kushwaha wrote: > > On Fri, Nov 1, 2019 at 2:04 AM dann frazier > > wrote: > > > > > > On Thu, Oct 31, 2019 at 12:55:10PM -0600, dann frazier wrote: > > > > On Thu, Oct

Re: [edk2-devel] [PATCH] SecurityPkg/OpalPassword: Remove dependency on EFI_BLOCK_IO_PROTOCOL

2019-11-03 Thread Yao, Jiewen
Hello May I know what test has been done for this patch? > -Original Message- > From: Chu, Maggie > Sent: Monday, November 4, 2019 12:04 PM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Zhang, Chao B > ; Yao, Jiewen > Subject: [PATCH] SecurityPkg/OpalPassword: Remove dependency on >

[edk2-devel] [PATCH] SecurityPkg/OpalPassword: Remove dependency on EFI_BLOCK_IO_PROTOCOL

2019-11-03 Thread Maggie Chu
https://bugzilla.tianocore.org/show_bug.cgi?id=2327 RAID drivers abstract their physical drives that make up the array into a single unit, and do not supply individual EFI_BLOCK_IO_PROTOCOL instances for each physical drive in the array. This breaks support for the Security Storage Command

Re: [edk2-devel] [edk2] NOOPT and PcdOptimizeCompilerEnable

2019-11-03 Thread Liming Gao
Andrew: I did some investigation. Seemly, there is no way to disable VS command line warning. Thanks Liming From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Andrew Fish via Groups.Io Sent: Thursday, October 31, 2019 10:07 AM To: devel@edk2.groups.io; Gao, Liming Cc:

Re: [edk2-devel] [edk2] Smbuslib and SmbusPpi

2019-11-03 Thread Liming Gao
SmbusPpi is defined in PI spec. SmbusLib is defined in Edk2 MdePkg. Edk2 PEIM can base on SmBusLib. SmBusLib implementation wraps SmBusPpi. Thanks Liming From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Tiger Liu(BJ-RD) Sent: Thursday, October 31, 2019 6:51 PM To:

Re: [edk2-devel] [PATCH 2/2] BaseTools: Add support for parseing map files generated by CLANG9 in GenFv

2019-11-03 Thread Liming Gao
Reviewed-by: Liming Gao >-Original Message- >From: Liu, Zhiguang >Sent: Friday, November 01, 2019 2:27 PM >To: devel@edk2.groups.io >Cc: Feng, Bob C ; Gao, Liming >Subject: [PATCH 2/2] BaseTools: Add support for parseing map files generated >by CLANG9 in GenFv > >REF:

Re: [edk2-devel] [PATCH 1/2] BaseTools: Add map file parsing support for CLANG9

2019-11-03 Thread Liming Gao
Reviewed-by: Liming Gao >-Original Message- >From: Liu, Zhiguang >Sent: Friday, November 01, 2019 2:27 PM >To: devel@edk2.groups.io >Cc: Feng, Bob C ; Gao, Liming >Subject: [PATCH 1/2] BaseTools: Add map file parsing support for CLANG9 > >Cc: Bob Feng >Cc: Liming Gao > >Signed-off-by:

Re: [edk2-devel] [PATCH v1 0/2] Update the SRAT Acpiview parser to ACPI 6.3

2019-11-03 Thread Liming Gao
I see the change in MdePkg has been pushed. So, the remaining discussion is about the change in ShellPkg, right? Thanks Liming >-Original Message- >From: Gao, Zhichao >Sent: Monday, November 04, 2019 10:13 AM >To: Sudipto Paul ; Sami Mujawar >; Krzysztof Koch ; >devel@edk2.groups.io >Cc:

Re: [edk2-devel] [PATCH v1 0/2] Update the SRAT Acpiview parser to ACPI 6.3

2019-11-03 Thread Gao, Zhichao
Thanks for the T-B. But the patch has conflict now, can you help to address it? As I know the previous change aims to remove the declaration of the internal used function. So I suggest to remove the declaration of the new added function and move the definition to the location of the

Re: [edk2-devel] [PATCH] MdePkg/UefiDebugLibConOut: Pass the correct buffer size

2019-11-03 Thread Liming Gao
Push @ 5ae6c993ab75c694831547b7436543a41d60458a >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Liming Gao >Sent: Monday, October 21, 2019 11:12 AM >To: marvin.haeu...@outlook.com; devel@edk2.groups.io >Cc: vit9...@protonmail.com; Kinney,

Re: [edk2-devel] [PATCH] MdePkg/UefiDebugLibStdErr: Pass the correct buffer size

2019-11-03 Thread Liming Gao
Push @ 787c4baace1c0c49fa1628990402be8997294d17 >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Liming Gao >Sent: Monday, October 21, 2019 11:12 AM >To: marvin.haeu...@outlook.com; devel@edk2.groups.io >Cc: vit9...@protonmail.com; Kinney,

Re: [edk2-devel] [PATCH] MdePkg/UefiFileHandleLib: Fix potential NULL dereference

2019-11-03 Thread Liming Gao
Push @ 1009b59b6525c16724fe2684c344a6359af28b55 >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Liming Gao >Sent: Monday, October 21, 2019 11:12 AM >To: marvin.haeu...@outlook.com; devel@edk2.groups.io >Cc: vit9...@protonmail.com; Kinney,

Re: [edk2-devel] [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

2019-11-03 Thread Liming Gao
Push @ c18708d2f002a6ea43ed65d6c76dc4020687005c >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Liming Gao >Sent: Wednesday, October 30, 2019 1:46 PM >To: Huang, Qing ; devel@edk2.groups.io >Subject: Re: [edk2-devel] [PATCH] MdePkg-UefiSpec.h:

Re: [edk2-devel] [PATCH v1 1/1] MdePkg: Add AML OpCode definition for ExternalOp

2019-11-03 Thread Liming Gao
Push @ b10ef8a4e75899cfad421dc60d0bbf93c53d37f4 >-Original Message- >From: Sami Mujawar [mailto:sami.muja...@arm.com] >Sent: Wednesday, October 30, 2019 11:15 PM >To: devel@edk2.groups.io; Pierre Gondois >Cc: Kinney, Michael D ; Gao, Liming >; Sami Mujawar ; nd > >Subject: RE:

Re: [edk2-devel] [PATCH] MdePkg/UefiFileHandleLib: Tolerate more Root handle FileNames

2019-11-03 Thread Liming Gao
Push @ 6407186db9505f101ece4e1571eceed69b9fbdbe >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Liming Gao >Sent: Monday, November 04, 2019 8:50 AM >To: Gao, Zhichao ; devel@edk2.groups.io; >marvin.haeu...@outlook.com >Cc:

Re: [edk2-devel] [PATCH] MdePkg/UefiFileHandleLib: Tolerate more Root handle FileNames

2019-11-03 Thread Liming Gao
Reviewed-by: Liming Gao >-Original Message- >From: Gao, Zhichao >Sent: Thursday, October 24, 2019 10:52 AM >To: devel@edk2.groups.io; marvin.haeu...@outlook.com >Cc: vit9...@protonmail.com; Kinney, Michael D >; Gao, Liming >Subject: RE: [PATCH] MdePkg/UefiFileHandleLib: Tolerate more