[edk2-devel] [PATCH v1] IntelSiliconPkg: Add FVI_SMBIOS_TYPE definition in FirmwareVersionInfo.h

[Isaac Oram]

Ray,

I think it is ok because this file is "Intel common definitions for an OEM defined SMBIOS table structure".  We could say that the folder name is wrong or the whole file is in the wrong place.  But this addition seems consistent with the rest of the file contents.

Regards,
Isaac

-----Original Message-----
From: Ni, Ray <ray.ni at intel.com> 
Sent: Thursday, January 5, 2023 1:39 AM
To: Chang, Hunter <hunter.chang at intel.com>; devel at edk2.groups.io
Cc: Chaganty, Rangasai V <rangasai.v.chaganty at intel.com>; Oram, Isaac W <isaac.w.oram at intel.com>; S, Ashraf Ali <ashraf.ali.s at intel.com>
Subject: RE: [PATCH v1] IntelSiliconPkg: Add FVI_SMBIOS_TYPE definition in FirmwareVersionInfo.h

If it's not defined by any spec, it's probably not a good idea to put the definition in an industry header file.


> -----Original Message-----
> From: Chang, Hunter <hunter.chang at intel.com>
> Sent: Thursday, January 5, 2023 3:54 PM
> To: Ni, Ray <ray.ni at intel.com>; devel at edk2.groups.io
> Cc: Chaganty, Rangasai V <rangasai.v.chaganty at intel.com>; Oram, Isaac 
> W <isaac.w.oram at intel.com>; S, Ashraf Ali <ashraf.ali.s at intel.com>
> Subject: RE: [PATCH v1] IntelSiliconPkg: Add FVI_SMBIOS_TYPE 
> definition in FirmwareVersionInfo.h
> 
> It is from Sipkg.dec
> gSiPkgTokenSpaceGuid.PcdFviSmbiosType|0xDD
> 
> 0x00~0x7F are reserved for and defined by smbios spec 0x80~0xFF are 
> available for system- and OEM-specific information
> 
> I think there is no spec define 0xDD.
> 
> -----Original Message-----
> From: Ni, Ray <ray.ni at intel.com>
> Sent: Thursday, January 5, 2023 3:05 PM
> To: Chang, Hunter <hunter.chang at intel.com>; devel at edk2.groups.io
> Cc: Chaganty, Rangasai V <rangasai.v.chaganty at intel.com>; Oram, Isaac 
> W <isaac.w.oram at intel.com>; S, Ashraf Ali <ashraf.ali.s at intel.com>
> Subject: RE: [PATCH v1] IntelSiliconPkg: Add FVI_SMBIOS_TYPE 
> definition in FirmwareVersionInfo.h
> 
> Which spec defines 0xDD?
> 
> > -----Original Message-----
> > From: Chang, Hunter <hunter.chang at intel.com>
> > Sent: Wednesday, January 4, 2023 3:47 PM
> > To: devel at edk2.groups.io
> > Cc: Chang, Hunter <hunter.chang at intel.com>; Ni, Ray 
> > <ray.ni at intel.com>; Chaganty, Rangasai V 
> > <rangasai.v.chaganty at intel.com>; Oram, Isaac W 
> > <isaac.w.oram at intel.com>; S, Ashraf Ali <ashraf.ali.s at intel.com>
> > Subject: [PATCH v1] IntelSiliconPkg: Add FVI_SMBIOS_TYPE definition 
> > in FirmwareVersionInfo.h
> >
> > From: Hunter Chang <hunter.chang at intel.com>
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4242
> >
> > Define a macro for SmbiosFeaturePkg usage which named 
> > INTEL_FVI_SMBIOS_TYPE and initialized to 0xDD in 
> > IndustryStandard/FirmwareVersionInfo.h
> >
> > Signed-off-by: Hunter Chang <hunter.chang at intel.com>
> >
> > Cc: Ray Ni <ray.ni at intel.com>
> > Cc: Rangasai V Chaganty <rangasai.v.chaganty at intel.com>
> > Cc: Isaac Oram <isaac.w.oram at intel.com>
> > Cc: Ashraf Ali S <ashraf.ali.s at intel.com>
> > ---
> >
> > Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVersi
> > on
> > Info.h | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git
> > a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVer
> > si
> > onInfo.h
> > b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVer
> > si
> > onInfo.h
> > index b30bc3f9e7..466cb8e7d2 100644
> > ---
> > a/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/FirmwareVer
> > si
> > onInfo.h
> > +++ b/Silicon/Intel/IntelSiliconPkg/Include/IndustryStandard/Firmwar
> > +++ eV
> > +++ ersionInfo.h
> > @@ -18,6 +18,7 @@
> >  #include <IndustryStandard/SmBios.h>
> >
> >
> >
> >  #define INTEL_FIRMWARE_VERSION_INFO_GROUP_NAME    "Firmware Version Info"
> >
> > +#define INTEL_FVI_SMBIOS_TYPE                     0xDD
> >
> >
> >
> >  #pragma pack(1)
> >
> >
> >
> > --
> > 2.26.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#98034): https://edk2.groups.io/g/devel/message/98034
Mute This Topic: https://groups.io/mt/96046622/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-