* [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 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
* 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
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