* [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data @ 2016-10-21 16:17 Giri P Mudusuru 2016-10-21 23:42 ` Yao, Jiewen 2016-10-22 10:45 ` Zeng, Star 0 siblings, 2 replies; 4+ messages in thread From: Giri P Mudusuru @ 2016-10-21 16:17 UTC (permalink / raw) To: edk2-devel; +Cc: Jiewen Yao, Star Zeng, Michael Kinney Add gIntelSmbiosDataHobGuid and gIntelGraphicsVbtDataGuid GUIDs Cc: Jiewen Yao <jiewen.yao@intel.com> Cc: Star Zeng <star.zeng@intel.com> Cc: Michael Kinney <michael.d.kinney@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Giri P Mudusuru <giri.p.mudusuru@intel.com> --- IntelSiliconPkg/IntelSiliconPkg.dec | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/IntelSiliconPkg/IntelSiliconPkg.dec b/IntelSiliconPkg/IntelSiliconPkg.dec index e42dbad..556fe76 100644 --- a/IntelSiliconPkg/IntelSiliconPkg.dec +++ b/IntelSiliconPkg/IntelSiliconPkg.dec @@ -1,5 +1,5 @@ ## @file -# IntelSilicon Package +# Intel Silicon Package # # This package provides common open source Intel silicon modules. # @@ -15,10 +15,19 @@ ## [Defines] - DEC_SPECIFICATION = 0x00010005 - PACKAGE_NAME = IntelSiliconPkg - PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 - PACKAGE_VERSION = 0.1 + DEC_SPECIFICATION = 0x00010005 + PACKAGE_NAME = IntelSiliconPkg + PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 + PACKAGE_VERSION = 0.1 [Includes] Include + +[Guids] + ## GUID for Graphics Video BIOS Table (VBT) Data + gIntelGraphicsVbtDataGuid = { 0x56752da9, 0xde6b, 0x4895, { 0x88, 0x19, 0x19, 0x45, 0xb6, 0xb7, 0x6c, 0x22 }} + + ## HOB GUID to publish SMBIOS data records from PEI phase + # Generic DXE Library / Driver can locate HOB(s) and add SMBIOS records into SMBIOS table + gIntelSmbiosDataHobGuid = { 0x798e722e, 0x15b2, 0x4e13, { 0x8a, 0xe9, 0x6b, 0xa3, 0x0f, 0xf7, 0xf1, 0x67 }} + -- 2.9.0.windows.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data 2016-10-21 16:17 [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data Giri P Mudusuru @ 2016-10-21 23:42 ` Yao, Jiewen 2016-10-23 13:47 ` Mudusuru, Giri P 2016-10-22 10:45 ` Zeng, Star 1 sibling, 1 reply; 4+ messages in thread From: Yao, Jiewen @ 2016-10-21 23:42 UTC (permalink / raw) To: Mudusuru, Giri P, edk2-devel@lists.01.org Cc: Kinney, Michael D, Zeng, Star, Yao, Jiewen Thanks for the patch. I have 2 minor comments: 1) below update seems unnecessary. It makes me confused. We did not change anything right? Can we skip it? - DEC_SPECIFICATION = 0x00010005 - PACKAGE_NAME = IntelSiliconPkg - PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 - PACKAGE_VERSION = 0.1 + DEC_SPECIFICATION = 0x00010005 + PACKAGE_NAME = IntelSiliconPkg + PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 + PACKAGE_VERSION = 0.1 2) Can we add one line to describe the VBT in more detail? If this is FFS GUID, or HOB GUID, PROTOCOL GUID? How is produced and consumed? > + ## GUID for Graphics Video BIOS Table (VBT) Data > + gIntelGraphicsVbtDataGuid = { 0x56752da9, 0xde6b, 0x4895, > { 0x88, 0x19, 0x19, 0x45, 0xb6, 0xb7, 0x6c, 0x22 }} I think below example is much better: > + ## HOB GUID to publish SMBIOS data records from PEI phase > + # Generic DXE Library / Driver can locate HOB(s) and add SMBIOS records > into SMBIOS table > + gIntelSmbiosDataHobGuid = { 0x798e722e, 0x15b2, 0x4e13, > { 0x8a, 0xe9, 0x6b, 0xa3, 0x0f, 0xf7, 0xf1, 0x67 }} Thank you Yao Jiewen > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Giri P Mudusuru > Sent: Saturday, October 22, 2016 12:18 AM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Yao, Jiewen > <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com> > Subject: [edk2] [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB > and VBT data > > Add gIntelSmbiosDataHobGuid and gIntelGraphicsVbtDataGuid GUIDs > > Cc: Jiewen Yao <jiewen.yao@intel.com> > Cc: Star Zeng <star.zeng@intel.com> > Cc: Michael Kinney <michael.d.kinney@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Giri P Mudusuru <giri.p.mudusuru@intel.com> > --- > IntelSiliconPkg/IntelSiliconPkg.dec | 19 ++++++++++++++----- > 1 file changed, 14 insertions(+), 5 deletions(-) > > diff --git a/IntelSiliconPkg/IntelSiliconPkg.dec > b/IntelSiliconPkg/IntelSiliconPkg.dec > index e42dbad..556fe76 100644 > --- a/IntelSiliconPkg/IntelSiliconPkg.dec > +++ b/IntelSiliconPkg/IntelSiliconPkg.dec > @@ -1,5 +1,5 @@ > ## @file > -# IntelSilicon Package > +# Intel Silicon Package > # > # This package provides common open source Intel silicon modules. > # > @@ -15,10 +15,19 @@ > ## > > [Defines] > - DEC_SPECIFICATION = 0x00010005 > - PACKAGE_NAME = IntelSiliconPkg > - PACKAGE_GUID = > F7A58914-FA0E-4F71-BD6A-220FDF824A49 > - PACKAGE_VERSION = 0.1 > + DEC_SPECIFICATION = 0x00010005 > + PACKAGE_NAME = IntelSiliconPkg > + PACKAGE_GUID = > F7A58914-FA0E-4F71-BD6A-220FDF824A49 > + PACKAGE_VERSION = 0.1 > > [Includes] > Include > + > +[Guids] > + ## GUID for Graphics Video BIOS Table (VBT) Data > + gIntelGraphicsVbtDataGuid = { 0x56752da9, 0xde6b, 0x4895, > { 0x88, 0x19, 0x19, 0x45, 0xb6, 0xb7, 0x6c, 0x22 }} > + > + ## HOB GUID to publish SMBIOS data records from PEI phase > + # Generic DXE Library / Driver can locate HOB(s) and add SMBIOS records > into SMBIOS table > + gIntelSmbiosDataHobGuid = { 0x798e722e, 0x15b2, 0x4e13, > { 0x8a, 0xe9, 0x6b, 0xa3, 0x0f, 0xf7, 0xf1, 0x67 }} > + > -- > 2.9.0.windows.1 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data 2016-10-21 23:42 ` Yao, Jiewen @ 2016-10-23 13:47 ` Mudusuru, Giri P 0 siblings, 0 replies; 4+ messages in thread From: Mudusuru, Giri P @ 2016-10-23 13:47 UTC (permalink / raw) To: Yao, Jiewen, edk2-devel@lists.01.org; +Cc: Kinney, Michael D, Zeng, Star Thanks for the feedback Jiewen, Star. Sure will send separate patches. The cosmetic change was to indent it at odd columns so it aligns with tabs (two spaces) will skip this for now. Will add description to the VBT. This is FFS GUID to store the GOP VBT data in RAW format. Thanks, -Giri > -----Original Message----- > From: Yao, Jiewen > Sent: Friday, October 21, 2016 4:43 PM > To: Mudusuru, Giri P <giri.p.mudusuru@intel.com>; edk2-devel@lists.01.org > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Zeng, Star > <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com> > Subject: RE: [edk2] [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB > and VBT data > > Thanks for the patch. I have 2 minor comments: > > 1) below update seems unnecessary. It makes me confused. We did not change > anything right? Can we skip it? > - DEC_SPECIFICATION = 0x00010005 > - PACKAGE_NAME = IntelSiliconPkg > - PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 > - PACKAGE_VERSION = 0.1 > + DEC_SPECIFICATION = 0x00010005 > + PACKAGE_NAME = IntelSiliconPkg > + PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 > + PACKAGE_VERSION = 0.1 > > 2) Can we add one line to describe the VBT in more detail? If this is FFS GUID, or > HOB GUID, PROTOCOL GUID? How is produced and consumed? > > + ## GUID for Graphics Video BIOS Table (VBT) Data > > + gIntelGraphicsVbtDataGuid = { 0x56752da9, 0xde6b, 0x4895, > > { 0x88, 0x19, 0x19, 0x45, 0xb6, 0xb7, 0x6c, 0x22 }} > > I think below example is much better: > > + ## HOB GUID to publish SMBIOS data records from PEI phase > > + # Generic DXE Library / Driver can locate HOB(s) and add SMBIOS records > > into SMBIOS table > > + gIntelSmbiosDataHobGuid = { 0x798e722e, 0x15b2, 0x4e13, > > { 0x8a, 0xe9, 0x6b, 0xa3, 0x0f, 0xf7, 0xf1, 0x67 }} > > > > Thank you > Yao Jiewen > > > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > > Giri P Mudusuru > > Sent: Saturday, October 22, 2016 12:18 AM > > To: edk2-devel@lists.01.org > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Yao, Jiewen > > <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com> > > Subject: [edk2] [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB > > and VBT data > > > > Add gIntelSmbiosDataHobGuid and gIntelGraphicsVbtDataGuid GUIDs > > > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > Cc: Star Zeng <star.zeng@intel.com> > > Cc: Michael Kinney <michael.d.kinney@intel.com> > > Contributed-under: TianoCore Contribution Agreement 1.0 > > Signed-off-by: Giri P Mudusuru <giri.p.mudusuru@intel.com> > > --- > > IntelSiliconPkg/IntelSiliconPkg.dec | 19 ++++++++++++++----- > > 1 file changed, 14 insertions(+), 5 deletions(-) > > > > diff --git a/IntelSiliconPkg/IntelSiliconPkg.dec > > b/IntelSiliconPkg/IntelSiliconPkg.dec > > index e42dbad..556fe76 100644 > > --- a/IntelSiliconPkg/IntelSiliconPkg.dec > > +++ b/IntelSiliconPkg/IntelSiliconPkg.dec > > @@ -1,5 +1,5 @@ > > ## @file > > -# IntelSilicon Package > > +# Intel Silicon Package > > # > > # This package provides common open source Intel silicon modules. > > # > > @@ -15,10 +15,19 @@ > > ## > > > > [Defines] > > - DEC_SPECIFICATION = 0x00010005 > > - PACKAGE_NAME = IntelSiliconPkg > > - PACKAGE_GUID = > > F7A58914-FA0E-4F71-BD6A-220FDF824A49 > > - PACKAGE_VERSION = 0.1 > > + DEC_SPECIFICATION = 0x00010005 > > + PACKAGE_NAME = IntelSiliconPkg > > + PACKAGE_GUID = > > F7A58914-FA0E-4F71-BD6A-220FDF824A49 > > + PACKAGE_VERSION = 0.1 > > > > [Includes] > > Include > > + > > +[Guids] > > + ## GUID for Graphics Video BIOS Table (VBT) Data > > + gIntelGraphicsVbtDataGuid = { 0x56752da9, 0xde6b, 0x4895, > > { 0x88, 0x19, 0x19, 0x45, 0xb6, 0xb7, 0x6c, 0x22 }} > > + > > + ## HOB GUID to publish SMBIOS data records from PEI phase > > + # Generic DXE Library / Driver can locate HOB(s) and add SMBIOS records > > into SMBIOS table > > + gIntelSmbiosDataHobGuid = { 0x798e722e, 0x15b2, 0x4e13, > > { 0x8a, 0xe9, 0x6b, 0xa3, 0x0f, 0xf7, 0xf1, 0x67 }} > > + > > -- > > 2.9.0.windows.1 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data 2016-10-21 16:17 [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data Giri P Mudusuru 2016-10-21 23:42 ` Yao, Jiewen @ 2016-10-22 10:45 ` Zeng, Star 1 sibling, 0 replies; 4+ messages in thread From: Zeng, Star @ 2016-10-22 10:45 UTC (permalink / raw) To: Mudusuru, Giri P, edk2-devel@lists.01.org Cc: Yao, Jiewen, Kinney, Michael D, Zeng, Star My suggestion is to split the patch to two different patches since it is adding two GUIDs. Thanks, Star -----Original Message----- From: Mudusuru, Giri P Sent: Saturday, October 22, 2016 12:18 AM To: edk2-devel@lists.01.org Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com> Subject: [edk2] [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data Add gIntelSmbiosDataHobGuid and gIntelGraphicsVbtDataGuid GUIDs Cc: Jiewen Yao <jiewen.yao@intel.com> Cc: Star Zeng <star.zeng@intel.com> Cc: Michael Kinney <michael.d.kinney@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Giri P Mudusuru <giri.p.mudusuru@intel.com> --- IntelSiliconPkg/IntelSiliconPkg.dec | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/IntelSiliconPkg/IntelSiliconPkg.dec b/IntelSiliconPkg/IntelSiliconPkg.dec index e42dbad..556fe76 100644 --- a/IntelSiliconPkg/IntelSiliconPkg.dec +++ b/IntelSiliconPkg/IntelSiliconPkg.dec @@ -1,5 +1,5 @@ ## @file -# IntelSilicon Package +# Intel Silicon Package # # This package provides common open source Intel silicon modules. # @@ -15,10 +15,19 @@ ## [Defines] - DEC_SPECIFICATION = 0x00010005 - PACKAGE_NAME = IntelSiliconPkg - PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 - PACKAGE_VERSION = 0.1 + DEC_SPECIFICATION = 0x00010005 + PACKAGE_NAME = IntelSiliconPkg + PACKAGE_GUID = F7A58914-FA0E-4F71-BD6A-220FDF824A49 + PACKAGE_VERSION = 0.1 [Includes] Include + +[Guids] + ## GUID for Graphics Video BIOS Table (VBT) Data + gIntelGraphicsVbtDataGuid = { 0x56752da9, 0xde6b, 0x4895, { 0x88, 0x19, 0x19, 0x45, 0xb6, 0xb7, 0x6c, 0x22 }} + + ## HOB GUID to publish SMBIOS data records from PEI phase # Generic + DXE Library / Driver can locate HOB(s) and add SMBIOS records into SMBIOS table + gIntelSmbiosDataHobGuid = { 0x798e722e, 0x15b2, 0x4e13, { 0x8a, 0xe9, 0x6b, 0xa3, 0x0f, 0xf7, 0xf1, 0x67 }} + -- 2.9.0.windows.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-10-23 13:47 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-10-21 16:17 [PATCH] IntelSiliconPkg: Add GUIDs for SMBIOS data HOB and VBT data Giri P Mudusuru 2016-10-21 23:42 ` Yao, Jiewen 2016-10-23 13:47 ` Mudusuru, Giri P 2016-10-22 10:45 ` Zeng, Star
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox