From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id A968C1A1E08 for ; Thu, 1 Sep 2016 19:32:45 -0700 (PDT) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP; 01 Sep 2016 19:32:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,269,1470726000"; d="scan'208";a="1034506369" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga001.fm.intel.com with ESMTP; 01 Sep 2016 19:32:42 -0700 Received: from fmsmsx102.amr.corp.intel.com (10.18.124.200) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.248.2; Thu, 1 Sep 2016 19:32:42 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by FMSMSX102.amr.corp.intel.com (10.18.124.200) with Microsoft SMTP Server (TLS) id 14.3.248.2; Thu, 1 Sep 2016 19:32:19 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.109]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.102]) with mapi id 14.03.0248.002; Fri, 2 Sep 2016 10:32:17 +0800 From: "Yao, Jiewen" To: "Mudusuru, Giri P" , "edk2-devel@lists.01.org" Thread-Topic: [edk2] [PATCH] IntelSiliconPkg: Rename IGD structures to make it consistent Thread-Index: AQHSAxJG7ydA5tItVECcBD7a+trKdaBlfkCg Date: Fri, 2 Sep 2016 02:32:17 +0000 Message-ID: <74D8A39837DF1E4DA445A8C0B3885C50385F885F@shsmsx102.ccr.corp.intel.com> References: In-Reply-To: Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] IntelSiliconPkg: Rename IGD structures to make it consistent X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Sep 2016 02:32:45 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed by: Jiewen.yao@intel.com > -----Original Message----- > From: Mudusuru, Giri P > Sent: Wednesday, August 31, 2016 6:58 AM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen > Subject: [edk2] [PATCH] IntelSiliconPkg: Rename IGD structures to make it > consistent >=20 > Renamed INTEL_IGD_* to IGD_* and IGD_OPREGION_VBT to > IGD_OPREGION_MBOX4 > to make it consistent with file name and other mailbox naming. >=20 > Cc: Jiewen Yao > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Giri P Mudusuru > --- > .../Include/IndustryStandard/IgdOpRegion.h | 39 > ++++++++++++---------- > 1 file changed, 22 insertions(+), 17 deletions(-) >=20 > diff --git a/IntelSiliconPkg/Include/IndustryStandard/IgdOpRegion.h > b/IntelSiliconPkg/Include/IndustryStandard/IgdOpRegion.h > index 9982c9c..7f76c09 100644 > --- a/IntelSiliconPkg/Include/IndustryStandard/IgdOpRegion.h > +++ b/IntelSiliconPkg/Include/IndustryStandard/IgdOpRegion.h > @@ -36,8 +36,9 @@ > **/ > #pragma pack(1) > /// > -/// OpRegion header (mailbox 0) structure. The OpRegion Header is used t= o > +/// OpRegion Mailbox 0 Header structure. The OpRegion Header is used to > /// identify a block of memory as the graphics driver OpRegion. > +/// Offset 0x0, Size 0x100 > /// > typedef struct { > CHAR8 SIGN[0x10]; ///< Offset 0x00 OpRegion Signature > @@ -49,16 +50,17 @@ typedef struct { > UINT32 MBOX; ///< Offset 0x58 Supported Mailboxes > UINT32 DMOD; ///< Offset 0x5C Driver Model > UINT8 RSV1[0xA0]; ///< Offset 0x60 Reserved > -} INTEL_IGD_OPREGION_HEADER; > +} IGD_OPREGION_HEADER; >=20 > /// > -/// OpRegion mailbox 1 (public ACPI Methods) > +/// OpRegion Mailbox 1 - Public ACPI Methods > +/// Offset 0x100, Size 0x100 > /// > typedef struct { > UINT32 DRDY; ///< Offset 0x100 Driver Readiness > UINT32 CSTS; ///< Offset 0x104 Status > UINT32 CEVT; ///< Offset 0x108 Current Event > - UINT8 RSV2[0x14]; ///< Offset 0x10C Reserved > + UINT8 RSVD[0x14]; ///< Offset 0x10C Reserved Must be Zero > UINT32 DIDL[8]; ///< Offset 0x120 Supported Display Devices ID > List > UINT32 CPDL[8]; ///< Offset 0x140 Currently Attached Display > Devices List > UINT32 CADL[8]; ///< Offset 0x160 Currently Active Display > Devices List > @@ -73,20 +75,22 @@ typedef struct { > UINT32 CNOT; ///< Offset 0x1BC Current OS Notification > UINT32 NRDY; ///< Offset 0x1C0 Driver Status > UINT8 RSV3[0x3C]; ///< Offset 0x1C4 - 0x1FF Reserved > -} INTEL_IGD_OPREGION_MBOX1; > +} IGD_OPREGION_MBOX1; >=20 > /// > -/// OpRegion mailbox 2 (Software SCI Interface). > +/// OpRegion Mailbox 2 - Software SCI Interface > +/// Offset 0x200, Size 0x100 > /// > typedef struct { > UINT32 SCIC; ///< Offset 0x200 Software SCI Command / > Status / Data > UINT32 PARM; ///< Offset 0x204 Software SCI Parameters > UINT32 DSLP; ///< Offset 0x208 Driver Sleep Time Out > UINT8 RSV4[0xF4]; ///< Offset 0x20C - 0x2FF Reserved > -} INTEL_IGD_OPREGION_MBOX2; > +} IGD_OPREGION_MBOX2; >=20 > /// > -/// OpRegion mailbox 3 (BIOS/Driver Communication - ASLE Support). > +/// OpRegion Mailbox 3 - BIOS/Driver Notification - ASLE Support > +/// Offset 0x300, Size 0x100 > /// > typedef struct { > UINT32 ARDY; ///< Offset 0x300 Driver Readiness > @@ -103,25 +107,26 @@ typedef struct { > UINT32 PFMB; ///< Offset 0x396 PWM Frequency and > Minimum Brightness > UINT32 CCDV; ///< Offset 0x39A Color Correction Default > Values > UINT8 RSV5[0x62]; ///< Offset 0x39E - 0x3FF Reserved > -} INTEL_IGD_OPREGION_MBOX3; > +} IGD_OPREGION_MBOX3; >=20 > /// > -/// OpRegion mailbox 4 (VBT). > +/// OpRegion Mailbox 4 - VBT Video BIOS Table > +/// Offset 0x400, Size 0x1800 > /// > typedef struct { > UINT8 RVBT[0x1800]; ///< Offset 0x400 - 0x1BFF Raw VBT Data > -} INTEL_IGD_OPREGION_VBT; > +} IGD_OPREGION_MBOX4; >=20 > /// > /// IGD OpRegion Structure > /// > typedef struct { > - INTEL_IGD_OPREGION_HEADER Header; ///< OpRegion header (Offset > 0x0, Size 0x100) > - INTEL_IGD_OPREGION_MBOX1 MBox1; ///< Mailbox 1: Public ACPI > Methods (Offset 0x100, Size 0x100) > - INTEL_IGD_OPREGION_MBOX2 MBox2; ///< Mailbox 2: Software SCI > Interface (Offset 0x200, Size 0x100) > - INTEL_IGD_OPREGION_MBOX3 MBox3; ///< Mailbox 3: BIOS to > Driver Communication (Offset 0x300, Size 0x100) > - INTEL_IGD_OPREGION_VBT VBT; ///< Mailbox 4: Video BIOS > Table (VBT) (Offset 0x400, Size 0x1200) > -} IGD_IGD_OPREGION_STRUCTURE; > + IGD_OPREGION_HEADER Header; ///< OpRegion header (Offset 0x0, Size > 0x100) > + IGD_OPREGION_MBOX1 MBox1; ///< Mailbox 1: Public ACPI > Methods (Offset 0x100, Size 0x100) > + IGD_OPREGION_MBOX2 MBox2; ///< Mailbox 2: Software SCI > Interface (Offset 0x200, Size 0x100) > + IGD_OPREGION_MBOX3 MBox3; ///< Mailbox 3: BIOS to Driver > Notification (Offset 0x300, Size 0x100) > + IGD_OPREGION_MBOX4 MBox4; ///< Mailbox 4: Video BIOS Table > (VBT) (Offset 0x400, Size 0x1800) > +} IGD_OPREGION_STRUCTURE; > #pragma pack() >=20 > #endif > -- > 2.9.0.windows.1