Sorry, I did not express my thought clearly. I mean union like below:
union { struct { UINT8 SegmentGroup; UINT8 Bus; UINT8 Device; UINT8 Function; } Pci; UINT32 Uid; } DeviceId; > -----Original Message----- > From: Wu, Hao A > Sent: Friday, March 23, 2018 9:20 AM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A <hao.a...@intel.com>; Younas Khan > <pmdyounaskhan...@gmail.com>; Kinney, Michael D > <michael.d.kin...@intel.com>; Gao, Liming <liming....@intel.com>; Yao, > Jiewen <jiewen....@intel.com> > Subject: [PATCH v2] MdePkg/IndustryStandard: Add header file for SPMI ACPI > table > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=840 > > Add the header file for Service Processor Management Interface ACPI table > definition. > > Cc: Younas Khan <pmdyounaskhan...@gmail.com> > Cc: Michael Kinney <michael.d.kin...@intel.com> > Cc: Liming Gao <liming....@intel.com> > Cc: Jiewen Yao <jiewen....@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Hao Wu <hao.a...@intel.com> > --- > .../ServiceProcessorManagementInterfaceTable.h | 109 > +++++++++++++++++++++ > 1 file changed, 109 insertions(+) > create mode 100644 > MdePkg/Include/IndustryStandard/ServiceProcessorManagementInterfaceTable > .h > > diff --git > a/MdePkg/Include/IndustryStandard/ServiceProcessorManagementInterfaceTa > ble.h > b/MdePkg/Include/IndustryStandard/ServiceProcessorManagementInterfaceTa > ble.h > new file mode 100644 > index 0000000000..bda32c8c94 > --- /dev/null > +++ > b/MdePkg/Include/IndustryStandard/ServiceProcessorManagementInterfaceTa > ble.h > @@ -0,0 +1,109 @@ > +/** @file > + Service Processor Management Interface (SPMI) ACPI table definition from > + Intelligent Platform Management Interface Specification Second Generation. > + > + Copyright (c) 2018, Intel Corporation. All rights reserved.<BR> > + This program and the accompanying materials > + are licensed and made available under the terms and conditions of the BSD > License > + which accompanies this distribution. The full text of the license may be > found at > + http://opensource.org/licenses/bsd-license.php > + > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" > BASIS, > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER > EXPRESS OR IMPLIED. > + > + @par Revision Reference: > + - Intelligent Platform Management Interface Specification Second > Generation > + v2.0 Revision 1.1, Dated October 2013. > + > https://www.intel.com/content/dam/www/public/us/en/documents/specificati > on-updates/ipmi-intelligent-platform-mgt-interface-spec-2nd-gen-v2-0-spec-up > date.pdf > +**/ > +#ifndef _SERVICE_PROCESSOR_MANAGEMENT_INTERFACE_TABLE_H_ > +#define _SERVICE_PROCESSOR_MANAGEMENT_INTERFACE_TABLE_H_ > + > +#include <IndustryStandard/Acpi.h> > + > +#pragma pack(1) > + > +/// > +/// Definition for Service Processor Management Interface Description Table > +/// > +typedef struct { > + EFI_ACPI_DESCRIPTION_HEADER Header; > + /// > + /// Indicates the type of IPMI interface. > + /// > + UINT8 InterfaceType; > + /// > + /// This field must always be 01h to be compatible with any software that > + /// implements previous versions of this spec. > + /// > + UINT8 Reserved1; > + /// > + /// Identifies the IPMI specification revision, in BCD format. > + /// > + UINT16 SpecificationRevision; > + /// > + /// Interrupt type(s) used by the interface. > + /// > + UINT8 InterruptType; > + /// > + /// The bit assignment of the SCI interrupt within the GPEx_STS register > of a > + /// GPE described if the FADT that the interface triggers. > + /// > + UINT8 Gpe; > + /// > + /// Reserved, must be 00h. > + /// > + UINT8 Reserved2; > + /// > + /// PCI Device Flag. > + /// > + UINT8 PciDeviceFlag; > + /// > + /// The I/O APIC or I/O SAPIC Global System Interrupt used by the > interface. > + /// > + UINT32 GlobalSystemInterrupt; > + /// > + /// The base address of the interface register set described using the > + /// Generic Address Structure (GAS, See [ACPI 2.0] for the definition). > + /// > + EFI_ACPI_2_0_GENERIC_ADDRESS_STRUCTURE BaseAddress; > + /// > + /// PCI Segment Group Number, if the IPMI device is a PCI device. > + /// Otherwise, this field is byte 1 of a UID. > + /// > + union { > + UINT8 PciSegmentGroup; > + UINT8 Uid1; > + } Byte60; > + /// > + /// PCI Bus Number, if the IPMI device is a PCI device. > + /// Otherwise, this field is byte 2 of a UID. > + /// > + union { > + UINT8 PciBus; > + UINT8 Uid2; > + } Byte61; > + /// > + /// PCI Device fields or byte 3 of a UID. > + /// > + union { > + UINT8 PciDevice; > + UINT8 Uid3; > + } Byte62; > + /// > + /// PCI Function fields or byte 4 of a UID. > + /// > + union { > + UINT8 PciFunction; > + UINT8 Uid4; > + } Byte63; > + /// > + /// This field must always be null (0x00) to be compatible with any > software > + /// that implements previous versions of this spec. > + /// > + UINT8 Reserved3; > +} EFI_ACPI_SERVICE_PROCESSOR_MANAGEMENT_INTERFACE_TABLE; > + > +#pragma pack() > + > +#endif > -- > 2.12.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel