Re: [edk2-devel] [PATCH v5 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-10-26 Thread Chris Li OS via groups.io
27;Ni, Ray'; Open Source Submission Subject: 回复: [ ** SPAMMAIL ** ][edk2-devel] [PATCH v5 1/1] MdePkg: Add Cxl20.h into IndustryStandard Chris: I just add push label for this PR https://github.com/tianocore/edk2/pull/4932. Thanks Liming > -邮件原件----- > 发件人: devel@edk2.groups.io 代

[edk2-devel] [PATCH v5 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-10-20 Thread Chris Li OS via groups.io
@Liming, The CI looks good (see https://github.com/tianocore/edk2/pull/4932), and this v5 patch only contains space adjustment compared with v4 to make CI happy. Kindly help merge this patch. Thanks a lot! -- 1

[edk2-devel] [PATCH v4 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-08-24 Thread Chris Li OS via groups.io
@Foster @Mike @Liming and all kindly help review/vote again. RegisterBlock[] is used. 1) Add CXL 2.0 header file to comply with CXL 2.0 specification 2) CXL 2.0 header will embed Cxl11.h 3) Updated Cxl.h to point to 2.0 header file Signed-off-by: Chris Li --- MdePkg/Include/IndustrySt

Re: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-08-22 Thread Chris Li OS via groups.io
Thanks and will send v4 soon. From: Kinney, Michael D Sent: Wednesday, August 23, 2023 1:47 AM To: Chris Li OS; Nong, Foster; devel@edk2.groups.io; Gao, Liming Cc: Yao, Jiewen; Ni, Ray; Open Source Submission; Kinney, Michael D Subject: RE: [edk2-devel] [P

Re: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-08-22 Thread Chris Li OS via groups.io
Hi Mike and Foster, So the preferred definition is as below? RegisterBlock[]; // offset 0x0C From: Nong, Foster Sent: Monday, August 21, 2023 9:59 AM To: Kinney, Michael D; Chris Li OS; devel@edk2.groups.io; Gao, Liming Cc: Yao, J

[edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-08-18 Thread Chris Li OS via groups.io
Absorbed Foster's input with minor delta. @Nong, Foster @Liming and all kindly help review/vote again. One change compared with Intel's proposal is the below line, we prefer [1] style to be safer to work across all compilers. Let us know if you folks have different suggestions here. - CXL_DV

Re: [edk2-devel] [PATCH 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-08-11 Thread Chris Li OS via groups.io
June 30, 2023 2:08 PM To: devel@edk2.groups.io; chri...@os.amperecomputing.com; Yao, Jiewen ; Nong, Foster Subject: RE: [edk2-devel] [PATCH 1/1] MdePkg: Add Cxl20.h into IndustryStandard +@Nong, Foster > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chris

[edk2-devel] [PATCH v2 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-07-03 Thread Chris Li OS via groups.io
into IndustryStandard +@Nong, Foster > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chris Li OS > via groups.io > Sent: Friday, June 30, 2023 2:07 PM > To: Yao, Jiewen ; devel@edk2.groups.io > Subject: Re: [edk2-devel] [PATCH 1/1] MdePkg

Re: [edk2-devel] [PATCH 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-06-29 Thread Chris Li OS via groups.io
Thanks Jiewen, will soon send an v2 for it. From: Yao, Jiewen Sent: Thursday, June 29, 2023 4:43 PM To: Chris Li OS; devel@edk2.groups.io Cc: Yao, Jiewen Subject: RE: [PATCH 1/1] MdePkg: Add Cxl20.h into IndustryStandard Thanks for the update. +} HDM_DEC

[edk2-devel] [PATCH 1/1] MdePkg: Add Cxl20.h into IndustryStandard

2023-06-28 Thread Chris Li OS via groups.io
1) Add CXL 2.0 header file to comply with CXL 2.0 specification 2) CXL 2.0 header will embed Cxl11.h 3) Updated Cxl.h to point to 2.0 header file Signed-off-by: Chris Li --- MdePkg/Include/IndustryStandard/Cxl.h | 2 +- MdePkg/Include/IndustryStandard/Cxl20.h | 477