n
> Source Submission
> Subject: Re: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into
> IndustryStandard
>
> Hi Mike and Foster,
> So the preferred definition is as below?
>
> RegisterBlock[]; // offset 0x0C
>
> __
August 21, 2023 9:59 AM
> To: Kinney, Michael D; Chris Li OS; devel@edk2.groups.io; Gao, Liming
> Cc: Yao, Jiewen; Ni, Ray; Open Source Submission
> Subject: RE: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into
> IndustryStandard
>
> Hi Mike,
>
> Yes. In CXL SPEC chapte
, Jiewen; Ni, Ray; Open Source Submission
Subject: RE: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into
IndustryStandard
Hi Mike,
Yes. In CXL SPEC chapter 8.1.9, the number of register blocks need calculate
from DVSEC length value of DVSEC header1 register.
-Original Message-
From
, Liming
Cc: Yao, Jiewen ; Ni, Ray ; Open Source
Submission ; Kinney, Michael D
Subject: RE: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into
IndustryStandard
Is RegisterBlock a flexible array member?
If so, then [] is the compatible syntax.
Mike
> -Original Message-
> From:
ney, Michael D ; Open Source Submission
>
> Subject: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into
> IndustryStandard
>
> Absorbed Foster's input with minor delta.
> @Nong, Foster @Liming and all kindly help review/vote again.
>
> One change compared with Intel&#
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
