From: "Chris Li OS via groups.io" <chrisli=os.amperecomputing.com@groups.io>
To: "Kinney, Michael D" <michael.d.kinney@intel.com>,
"Nong, Foster" <foster.nong@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>,
"Gao, Liming" <gaoliming@byosoft.com.cn>
Cc: "Yao, Jiewen" <jiewen.yao@intel.com>,
"Ni, Ray" <ray.ni@intel.com>,
Open Source Submission <patches@amperecomputing.com>
Subject: Re: [edk2-devel] [PATCH v3 1/1] MdePkg: Add Cxl20.h into IndustryStandard
Date: Wed, 23 Aug 2023 03:25:14 +0000 [thread overview]
Message-ID: <SN6PR01MB4656D7848EAF9E9B064298959F1CA@SN6PR01MB4656.prod.exchangelabs.com> (raw)
In-Reply-To: <SA2PR11MB493861F805BC20672ED95C11D21FA@SA2PR11MB4938.namprd11.prod.outlook.com>
Thanks and will send v4 soon.
________________________________________
From: Kinney, Michael D <michael.d.kinney@intel.com>
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] [PATCH v3 1/1] MdePkg: Add Cxl20.h into IndustryStandard
Yes.
Mike
> -----Original Message-----
> From: Chris Li OS <chrisli@os.amperecomputing.com>
> Sent: Tuesday, August 22, 2023 2:42 AM
> To: Nong, Foster <foster.nong@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; devel@edk2.groups.io; Gao, Liming
> <gaoliming@byosoft.com.cn>
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Ni, Ray <ray.ni@intel.com>; Open
> Source Submission <patches@amperecomputing.com>
> 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
>
> ________________________________________
> From: Nong, Foster <foster.nong@intel.com>
> Sent: Monday, 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 chapter 8.1.9, the number of register blocks need
> calculate from DVSEC length value of DVSEC header1 register.
>
> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Friday, August 18, 2023 11:30 PM
> To: Chris Li OS <chrisli@os.amperecomputing.com>; devel@edk2.groups.io;
> Nong, Foster <foster.nong@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Ni, Ray <ray.ni@intel.com>; Open
> Source Submission <patches@amperecomputing.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> 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: Chris Li OS <chrisli@os.amperecomputing.com>
> > Sent: Friday, August 18, 2023 2:29 AM
> > To: devel@edk2.groups.io; Nong, Foster <foster.nong@intel.com>; Gao,
> > Liming <gaoliming@byosoft.com.cn>
> > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Ni, Ray <ray.ni@intel.com>;
> > Kinney, Michael D <michael.d.kinney@intel.com>; Open Source Submission
> > <patches@amperecomputing.com>
> > 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'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_DVSEC_REGISTER_LOCATOR_REGISTER_BLOCK
> > RegisterBlock[0]; // offset 0x0C
> > + CXL_DVSEC_REGISTER_LOCATOR_REGISTER_BLOCK
> > RegisterBlock[1]; // offset 0x0C
> >
> >
> > Thanks all!
> >
> > -----
> > 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 <chrisli@os.amperecomputing.com>
> > ---
> > MdePkg/Include/IndustryStandard/Cxl.h | 2 +-
> > MdePkg/Include/IndustryStandard/Cxl20.h | 465
> > ++++++++++++++++++++++++
> > 2 files changed, 466 insertions(+), 1 deletion(-) create mode 100644
> > MdePkg/Include/IndustryStandard/Cxl20.h
> >
> > diff --git a/MdePkg/Include/IndustryStandard/Cxl.h
> > b/MdePkg/Include/IndustryStandard/Cxl.h
> > index 06c1230e3e..9ad3242e25 100644
> > --- a/MdePkg/Include/IndustryStandard/Cxl.h
> > +++ b/MdePkg/Include/IndustryStandard/Cxl.h
> > @@ -12,7 +12,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> > #ifndef _CXL_MAIN_H_ #define _CXL_MAIN_H_
> >
> > -#include <IndustryStandard/Cxl11.h>
> > +#include <IndustryStandard/Cxl20.h>
> > //
> > // CXL assigned new Vendor ID
> > //
> > diff --git a/MdePkg/Include/IndustryStandard/Cxl20.h
> > b/MdePkg/Include/IndustryStandard/Cxl20.h
> > new file mode 100644
> > index 0000000000..a5fe22b4a2
> > --- /dev/null
> > +++ b/MdePkg/Include/IndustryStandard/Cxl20.h
> > @@ -0,0 +1,465 @@
> > +/** @file
> > + CXL 2.0 Register definitions
> > +
> > + This file contains the register definitions based on the Compute
> > Express Link
> > + (CXL) Specification Revision 2.0.
> > +
> > + Copyright (c) 2023, Ampere Computing LLC. All rights reserved.<BR>
> > +
> > + SPDX-License-Identifier: BSD-2-Clause-Patent
> > +
> > +**/
> > +#ifndef CXL20_H_
> > +#define CXL20_H_
> > +
> > +#include <IndustryStandard/Cxl11.h>
> > +
> > +//
> > +// CXL DVSEC IDs
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.1.1 //
> > +#define CXL_DVSEC_ID_PCIE_DVSEC_FOR_CXL_DEVICE 0x0
> > +#define CXL_DVSEC_ID_NON_CXL_FUNCTION_MAP 0x2
> > +#define CXL_DVSEC_ID_CXL20_EXTENSIONS_DVSEC_FOR_PORTS 0x3
> > +#define CXL_DVSEC_ID_GPF_DVSEC_FOR_CXL_PORTS 0x4
> > +#define CXL_DVSEC_ID_GPF_DVSEC_FOR_CXL_DEVICES 0x5
> > +#define CXL_DVSEC_ID_PCIE_DVSEC_FOR_FLEX_BUS_PORT 0x7
> > +#define CXL_DVSEC_ID_REGISTER_LOCATOR 0x8
> > +#define CXL_DVSEC_ID_MLD 0x9
> > +#define CXL_DVSEC_ID_PCIE_DVSEC_FOR_TEST_CAPABILITY 0xA
> > +
> > +//
> > +// Register Block ID
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.1.9.1
> > +//
> > +#define CXL_REGISTER_BLOCK_ID_EMPTY 0x0
> > +#define CXL_REGISTER_BLOCK_ID_COMPONENT 0x1
> > +#define CXL_REGISTER_BLOCK_ID_BAR_VIRTUALIZATION_ACL 0x2
> > +#define CXL_REGISTER_BLOCK_ID_DEVICE 0x3
> > +
> > +//
> > +// CXL component register layout
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.2.4 //
> > +//|------------------------------------|
> > +//|--------- Range & Type -------------|
> > +//|------------------------------------| IO Base - 0KB
> > +//| (0KB - 4KB)IO Regs |
> > +//|------------------------------------| Cache and Mem Base - 4KB
> > +//| {4KB - 8KB)Cache & Mem Regs |
> > +//|------------------------------------| Implementation Spec Regs
> > +Base -
> > 8KB
> > +//| (8KB - 56KB)Implement Spec Regs|
> > +//|------------------------------------| ARB/Mux Regs Base - 56KB
> > +//| (56KB - 57KB)ARBMUX Regs |
> > +//|------------------------------------| Reserved Base - 57KB
> > +//| (57KB - 63KB)Reserved |
> > +//|------------------------------------| End 64KB // // Component
> > +Register Block Register Ranges Offset //
> > +#define CXL_COMPONENT_REGISTER_RANGE_OFFSET_IO 0x0
> > +#define CXL_COMPONENT_REGISTER_RANGE_OFFSET_CACHE_MEM 0x1000
> > +#define CXL_COMPONENT_REGISTER_RANGE_OFFSET_ARB_MUX 0xE000
> > +
> > +
> > +//
> > +// CXL Cache Memory Capability IDs
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.2.5 //
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_CXL 0x1
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_RAS 0x2
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_SECURITY 0x3
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_LINK 0x4
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_HDM_DECODER 0x5
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_EXTENDED_SECURITY 0x6
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_IDE 0x7
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_SNOOP_FILTER 0x8
> > +#define CXL_CACHE_MEM_CAPABILITY_ID_MASK 0xFFFF
> > +
> > +//
> > +// Generic CXL Device Capability IDs 0x0000 ~ 0x3FFF // Compute
> > +Express Link Specification Revision 2.0 - Chapter 8.2.8.2.1 //
> > +#define CXL_DEVICE_CAPABILITY_ID_CAPABILITIES_ARRAY_REGISTER 0x0000
> > +#define CXL_DEVICE_CAPABILITY_ID_DEVICE_STATUS 0x0001
> > +#define CXL_DEVICE_CAPABILITY_ID_PRIMARY_MAILBOX 0x0002
> > +#define CXL_DEVICE_CAPABILITY_ID_SECONDARY_MAILBOX 0x0003
> > +
> > +//
> > +// Specific CXL Device Capability IDs 0x4000 ~ 0x7FFF // Compute
> > +Express Link Specification Revision 2.0 - Chapter 8.2.8.2.1
> > and 8.2.8.5
> > +
> > +//
> > +#define CXL_DEVICE_CAPABILITY_ID_MEMORY_DEVICE_STATUS 0x4000
> > +#define CXL_DEVICE_CAPABILITY_ID_MASK 0xFFFF
> > +
> > +//
> > +// Memory Device Status
> > +// Compute Express Link Specification Revision 2.0 - Chapter
> > +8.2.8.5.1.1 //
> > +#define CXL_MEM_DEVICE_MEDIA_STATUS_NOT_READY 0x0
> > +#define CXL_MEM_DEVICE_MEDIA_STATUS_READY 0x1
> > +#define CXL_MEM_DEVICE_MEDIA_STATUS_ERROR 0x2
> > +#define CXL_MEM_DEVICE_MEDIA_STATUS_DISABLED 0x3
> > +
> > +//
> > +// Ensure proper structure formats
> > +//
> > +#pragma pack(1)
> > +
> > +//
> > +// PCIe DVSEC for CXL Device
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.1.3 //
> > +typedef union {
> > + struct {
> > + UINT16 CacheCapable : 1; // bit 0
> > + UINT16 IoCapable : 1; // bit 1
> > + UINT16 MemCapable : 1; // bit 2
> > + UINT16 MemHwInitMode : 1; // bit 3
> > + UINT16 HdmCount : 2; // bit
> 4..5
> > + UINT16 CacheWriteBackAndInvalidateCapable : 1; // bit 6
> > + UINT16 CxlResetCapable : 1; // bit 7
> > + UINT16 CxlResetTimeout : 3; // bit
> 8..10
> > + UINT16 CxlResetMemClrCapable : 1; // bit 11
> > + UINT16 Reserved : 1; // bit 12
> > + UINT16 MultipleLogicalDevice : 1; // bit 13
> > + UINT16 ViralCapable : 1; // bit 14
> > + UINT16 PmInitCompletionReportingCapable : 1; // bit 15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_CAPABILITY;
> > +
> > +typedef union {
> > + struct {
> > + UINT16 CacheEnable : 1; // bit 0
> > + UINT16 IoEnable : 1; // bit 1
> > + UINT16 MemEnable : 1; // bit 2
> > + UINT16 CacheSfCoverage : 5; // bit 3..7
> > + UINT16 CacheSfGranularity : 3; // bit 8..10
> > + UINT16 CacheCleanEviction : 1; // bit 11
> > + UINT16 Reserved1 : 2; // bit 12..13
> > + UINT16 ViralEnable : 1; // bit 14
> > + UINT16 Reserved2 : 1; // bit 15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_CONTROL;
> > +
> > +typedef union {
> > + struct {
> > + UINT16 Reserved1 : 14; // bit 0..13
> > + UINT16 ViralStatus : 1; // bit 14
> > + UINT16 Reserved2 : 1; // bit 15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_STATUS;
> > +
> > +typedef union {
> > + struct {
> > + UINT16 DisableCaching : 1; // bit 0
> > + UINT16 InitiateCacheWriteBackAndInvalidate : 1; // bit 1
> > + UINT16 InitiateCxlReset : 1; // bit 2
> > + UINT16 CxlResetMemClrEnable : 1; // bit 3
> > + UINT16 Reserved : 12; // bit
> 4..15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_CONTROL2;
> > +
> > +typedef union {
> > + struct {
> > + UINT16 CacheInvalid : 1; // bit
> 0
> > + UINT16 CxlResetComplete : 1; // bit
> 1
> > + UINT16 Reserved : 13; // bit
> > 2..14
> > + UINT16 PowerManagementInitialzationComplete : 1; // bit
> > 15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_STATUS2;
> > +
> > +typedef union {
> > + struct {
> > + UINT16 ConfigLock : 1; // bit 0
> > + UINT16 Reserved : 15; // bit 1..15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_LOCK;
> > +
> > +typedef union {
> > + struct {
> > + UINT16 CacheSizeUnit : 4; // bit 0..3
> > + UINT16 Reserved : 4; // bit 4..7
> > + UINT16 CacheSize : 8; // bit 8..15
> > + } Bits;
> > + UINT16 Uint16;
> > +} CXL_DVSEC_CXL_DEVICE_CAPABILITY2;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 MemorySizeHigh : 32; // bit 0..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_DVSEC_CXL_DEVICE_RANGE_SIZE_HIGH;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 MemoryInfoValid : 1; // bit 0
> > + UINT32 MemoryActive : 1; // bit 1
> > + UINT32 MediaType : 3; // bit 2..4
> > + UINT32 MemoryClass : 3; // bit 5..7
> > + UINT32 DesiredInterleave : 5; // bit 8..12
> > + UINT32 MemoryActiveTimeout : 3; // bit 13..15
> > + UINT32 Reserved : 12; // bit 16..27
> > + UINT32 MemorySizeLow : 4; // bit 28..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_DVSEC_CXL_DEVICE_RANGE_SIZE_LOW;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 MemoryBaseHigh : 32; // bit 0..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_DVSEC_CXL_DEVICE_RANGE_BASE_HIGH;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 Reserved : 28; // bit 0..27
> > + UINT32 MemoryBaseLow : 4; // bit 28..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_DVSEC_CXL_DEVICE_RANGE_BASE_LOW;
> > +
> > +
> > +typedef struct {
> > + PCI_EXPRESS_EXTENDED_CAPABILITIES_HEADER Header;
> > // offset 0x00
> > + PCI_EXPRESS_DESIGNATED_VENDOR_SPECIFIC_HEADER_1 DvsecHeader1;
> > // offset 0x04
> > + PCI_EXPRESS_DESIGNATED_VENDOR_SPECIFIC_HEADER_2 DvsecHeader2;
> > // offset 0x08
> > + CXL_DVSEC_CXL_DEVICE_CAPABILITY DeviceCapability;
> > // offset 0x0A
> > + CXL_DVSEC_CXL_DEVICE_CONTROL DeviceControl;
> > // offset 0x0C
> > + CXL_DVSEC_CXL_DEVICE_STATUS DeviceStatus;
> > // offset 0x0E
> > + CXL_DVSEC_CXL_DEVICE_CONTROL2 DeviceControl2;
> > // offset 0x10
> > + CXL_DVSEC_CXL_DEVICE_STATUS2 DeviceStatus2;
> > // offset 0x12
> > + CXL_DVSEC_CXL_DEVICE_LOCK DeviceLock;
> > // offset 0x14
> > + CXL_DVSEC_CXL_DEVICE_CAPABILITY2
> DeviceCapability2;
> > // offset 0x16
> > + CXL_DVSEC_CXL_DEVICE_RANGE_SIZE_HIGH
> > DeviceRange1SizeHigh; // offset 0x18
> > + CXL_DVSEC_CXL_DEVICE_RANGE_SIZE_LOW
> > DeviceRange1SizeLow; // offset 0x1C
> > + CXL_DVSEC_CXL_DEVICE_RANGE_BASE_HIGH
> > DeviceRange1BaseHigh; // offset 0x20
> > + CXL_DVSEC_CXL_DEVICE_RANGE_BASE_LOW
> > DeviceRange1BaseLow; // offset 0x24
> > + CXL_DVSEC_CXL_DEVICE_RANGE_SIZE_HIGH
> > DeviceRange2SizeHigh; // offset 0x28
> > + CXL_DVSEC_CXL_DEVICE_RANGE_SIZE_LOW
> > DeviceRange2SizeLow; // offset 0x2C
> > + CXL_DVSEC_CXL_DEVICE_RANGE_BASE_HIGH
> > DeviceRange2BaseHigh; // offset 0x30
> > + CXL_DVSEC_CXL_DEVICE_RANGE_BASE_LOW
> > DeviceRange2BaseLow; // offset 0x34
> > +} CXL_DVSEC_CXL_DEVICE;
> > +
> > +#define CXL_DVSEC_CXL_DEVICE_REVISION_1 0x1
> > +
> > +//
> > +// Register Locator DVSEC
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.1.9 //
> > +
> > +typedef union {
> > + struct {
> > + UINT32 RegisterBir : 3; // bit 0..2
> > + UINT32 Reserved : 5; // bit 3..7
> > + UINT32 RegisterBlockIdentifier : 8; // bit 8..15
> > + UINT32 RegisterBlockOffsetLow : 16; // bit 16..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_DVSEC_REGISTER_LOCATOR_REGISTER_OFFSET_LOW;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 RegisterBlockOffsetHigh : 32; // bit 0..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_DVSEC_REGISTER_LOCATOR_REGISTER_OFFSET_HIGH;
> > +
> > +typedef struct {
> > + CXL_DVSEC_REGISTER_LOCATOR_REGISTER_OFFSET_LOW OffsetLow;
> > + CXL_DVSEC_REGISTER_LOCATOR_REGISTER_OFFSET_HIGH OffsetHigh;
> > +} CXL_DVSEC_REGISTER_LOCATOR_REGISTER_BLOCK;
> > +
> > +typedef struct {
> > + PCI_EXPRESS_EXTENDED_CAPABILITIES_HEADER Header;
> > // offset 0x00
> > + PCI_EXPRESS_DESIGNATED_VENDOR_SPECIFIC_HEADER_1 DvsecHeader1;
> > // offset 0x04
> > + PCI_EXPRESS_DESIGNATED_VENDOR_SPECIFIC_HEADER_2 DvsecHeader2;
> > // offset 0x08
> > + UINT16 Reserved;
> > // offset 0x0A
> > + CXL_DVSEC_REGISTER_LOCATOR_REGISTER_BLOCK
> > RegisterBlock[1]; // offset 0x0C
> > +
> > +} CXL_DVSEC_REGISTER_LOCATOR;
> > +
> > +#define CXL_DVSEC_REGISTER_LOCATOR_REVISION_0 0x0
> > +
> > +//
> > +// CXL HDM Decoder Capability Header Register // Compute Express Link
> > +Specification Revision 2.0 - Chapter 8.2.5.5 // typedef union {
> > + struct {
> > + UINT32 CxlCapabilityId : 16; // bit 0..15
> > + UINT32 CxlCapabilityVersion : 4; // bit 16..19
> > + UINT32 CxlHdmDecoderCapabilityPointer : 12; // bit 20..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_CAPABILITY_HEADER_REGISTER;
> > +
> > +//
> > +// CXL HDM Decoder Capability Register // Compute Express Link
> > +Specification Revision 2.0 - Chapter 8.2.5.12 // typedef union {
> > + struct {
> > + UINT32 DecoderCount : 4; // bit 0..3
> > + UINT32 TargetCount : 4; // bit 4..7
> > + UINT32 InterleaveCapableA11to8 : 1; // bit 8
> > + UINT32 InterleaveCapableA14to12 : 1; // bit 9
> > + UINT32 PoisonOnDecodeErrorCapability : 1; // bit 10
> > + UINT32 Reserved : 21; // bit 11..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_CAPABILITY_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 PoisonOnDecodeErrorEnable : 1; // bit 0
> > + UINT32 HdmDecoderEnable : 1; // bit 1
> > + UINT32 Reserved : 30; // bit 2..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_GLOBAL_CONTROL_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 Reserved : 28; // bit 0..27
> > + UINT32 MemoryBaseLow : 4; // bit 28..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_BASE_LOW_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 MemoryBaseHigh : 32; // bit 0..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_BASE_HIGH_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 Reserved : 28; // bit 0..27
> > + UINT32 MemorySizeLow : 4; // bit 28..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_SIZE_LOW_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 MemorySizeHigh : 32; // bit 0..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_SIZE_HIGH_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 InterleaveGranularity : 4; // bit 0..3
> > + UINT32 InterleaveWays : 4; // bit 4..7
> > + UINT32 LockOnCommit : 1; // bit 8
> > + UINT32 Commit : 1; // bit 9
> > + UINT32 Committed : 1; // bit 10
> > + UINT32 ErrorNotCommitted : 1; // bit 11
> > + UINT32 TargetDeviceType : 1; // bit 12
> > + UINT32 Reserved : 19; // bit 13..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_CONTROL_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 TargetPortIdentiferWay0 : 8; // bit 0..7
> > + UINT32 TargetPortIdentiferWay1 : 8; // bit 8..15
> > + UINT32 TargetPortIdentiferWay2 : 8; // bit 16..23
> > + UINT32 TargetPortIdentiferWay3 : 8; // bit 24..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_TARGET_LIST_LOW_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 Reserved : 28; // bit 0..27
> > + UINT32 DpaSkipLow : 4; // bit 28..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_DPA_SKIP_LOW_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 TargetPortIdentiferWay4 : 8; // bit 0..7
> > + UINT32 TargetPortIdentiferWay5 : 8; // bit 8..15
> > + UINT32 TargetPortIdentiferWay6 : 8; // bit 16..23
> > + UINT32 TargetPortIdentiferWay7 : 8; // bit 24..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_TARGET_LIST_HIGH_REGISTER;
> > +
> > +typedef union {
> > + struct {
> > + UINT32 DpaSkipHigh : 32; // bit 0..31
> > + } Bits;
> > + UINT32 Uint32;
> > +} CXL_HDM_DECODER_DPA_SKIP_HIGH_REGISTER;
> > +
> > +typedef union {
> > + CXL_HDM_DECODER_TARGET_LIST_LOW_REGISTER TargetListLow;
> > + CXL_HDM_DECODER_DPA_SKIP_LOW_REGISTER DpaSkipLow;
> > +} CXL_HDM_DECODER_TARGET_LIST_OR_DPA_SKIP_LOW;
> > +
> > +typedef union {
> > + CXL_HDM_DECODER_TARGET_LIST_HIGH_REGISTER TargetListHigh;
> > + CXL_HDM_DECODER_DPA_SKIP_HIGH_REGISTER DpaSkipHigh;
> > +} CXL_HDM_DECODER_TARGET_LIST_OR_DPA_SKIP_HIGH;
> > +
> > +typedef struct {
> > + CXL_HDM_DECODER_BASE_LOW_REGISTER DecoderBaseLow;
> > // 0x10
> > + CXL_HDM_DECODER_BASE_HIGH_REGISTER DecoderBaseHigh;
> > // 0x14
> > + CXL_HDM_DECODER_SIZE_LOW_REGISTER DecoderSizeLow;
> > // 0x18
> > + CXL_HDM_DECODER_SIZE_HIGH_REGISTER DecoderSizeHigh;
> > // 0x1c
> > + CXL_HDM_DECODER_CONTROL_REGISTER DecoderControl;
> > // 0x20
> > + CXL_HDM_DECODER_TARGET_LIST_OR_DPA_SKIP_LOW
> > DecoderTargetListDpaSkipLow; // 0x24
> > + CXL_HDM_DECODER_TARGET_LIST_OR_DPA_SKIP_HIGH
> > DecoderTargetListDpaSkipHigh; // 0x28
> > + UINT32 Reserved;
> > // 0x2C
> > +} CXL_HDM_DECODER;
> > +
> > +//
> > +// CXL Device Capabilities Array Register // Compute Express Link
> > +Specification Revision 2.0 - Chapter 8.2.8.1 //
> > +
> > +typedef union {
> > + struct {
> > + UINT64 CxlDeviceCapabilityId : 16; // bit 0..15
> > + UINT64 CxlDeviceCapabilityVersion : 8; // bit 16..23
> > + UINT64 Reserved1 : 8; // bit 24..31
> > + UINT64 CxlDeviceCapabilitiesCount : 16; // bit 32..47
> > + UINT64 Reserved2 : 16; // bit 48..63
> > + } Bits;
> > + UINT64 Uint64;
> > +} CXL_DEVICE_CAPABILITIES_ARRAY_REGISTER;
> > +
> > +//
> > +// CXL Memory Status Register
> > +// Compute Express Link Specification Revision 2.0 - Chapter 8.2.8.5
> > +// typedef union {
> > + struct {
> > + UINT64 DeviceFatal : 1; // bit 0
> > + UINT64 FwHalt : 1; // bit 1
> > + UINT64 MediaStatus : 2; // bit 2..3
> > + UINT64 MailboxInterfacesReady : 1; // bit 4
> > + UINT64 ResetNeeded : 3; // bit 5..7
> > + UINT64 Reserved : 56; // bit 8..63
> > + } Bits;
> > + UINT64 Uint64;
> > +} CXL_MEMORY_DEVICE_STATUS_REGISTER;
> > +
> > +#pragma pack()
> > +
> > +#endif
> > --
> > 2.34.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107970): https://edk2.groups.io/g/devel/message/107970
Mute This Topic: https://groups.io/mt/100817312/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
next prev parent reply other threads:[~2023-08-23 3:25 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-28 5:28 [PATCH 1/1] MdePkg: Add Cxl20.h into IndustryStandard chrisli
2023-06-29 8:43 ` Yao, Jiewen
2023-06-30 6:07 ` Chris Li OS
2023-06-30 6:08 ` [edk2-devel] " Ni, Ray
2023-07-03 9:39 ` [edk2-devel] [PATCH v2 " Chris Li OS
2023-07-10 1:42 ` 回复: [ ** SPAMMAIL ** ][edk2-devel] " gaoliming
2023-08-18 9:28 ` [edk2-devel] [PATCH v3 " Chris Li OS via groups.io
2023-08-18 15:29 ` Michael D Kinney
2023-08-21 1:59 ` Nong, Foster
2023-08-22 9:42 ` Chris Li OS via groups.io
2023-08-22 17:47 ` Michael D Kinney
2023-08-23 3:25 ` Chris Li OS via groups.io [this message]
2023-08-25 3:48 ` [edk2-devel] [PATCH v4 " Chris Li OS via groups.io
2023-09-05 2:39 ` 回复: [ ** SPAMMAIL ** ][edk2-devel] " gaoliming via groups.io
2023-10-20 9:22 ` [edk2-devel] [PATCH v5 " Chris Li OS via groups.io
2023-10-26 0:56 ` 回复: [ ** SPAMMAIL ** ][edk2-devel] " gaoliming via groups.io
2023-10-26 7:10 ` [edk2-devel] " Chris Li OS via groups.io
2023-11-07 5:46 ` Huang, Yanbo
2023-07-18 3:14 ` [edk2-devel] [PATCH " Nong, Foster
2023-07-31 3:18 ` Nong, Foster
2023-08-11 7:24 ` Chris Li OS via groups.io
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=SN6PR01MB4656D7848EAF9E9B064298959F1CA@SN6PR01MB4656.prod.exchangelabs.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox