Re: [edk2-devel] [PATCH V2 3/8] MdePkg/Smbios.h: SMBIOS 3.3.0 Add value HBM and Die for type 17

["Liming Gao" <liming.gao@intel.com>]

Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gao, Zhichao
> Sent: Monday, November 4, 2019 6:13 PM
> To: devel@edk2.groups.io
> Subject: [edk2-devel] [PATCH V2 3/8] MdePkg/Smbios.h: SMBIOS 3.3.0 Add value HBM and Die for type 17
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2305
> 
> Memory Device (Type 17):
> – SMBIOSCR00178: add new memory device type value (HBM) and new form
> factor value (Die)
> 
> Signed-off-by: Zhichao Gao <zhichao.gao@intel.com>
> ---
>  MdePkg/Include/IndustryStandard/SmBios.h | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
> index 54ec837671..9aa8cf3e72 100644
> --- a/MdePkg/Include/IndustryStandard/SmBios.h
> +++ b/MdePkg/Include/IndustryStandard/SmBios.h
> @@ -1681,7 +1681,8 @@ typedef enum {
>    MemoryFormFactorRimm                     = 0x0C,
>    MemoryFormFactorSodimm                   = 0x0D,
>    MemoryFormFactorSrimm                    = 0x0E,
> -  MemoryFormFactorFbDimm                   = 0x0F
> +  MemoryFormFactorFbDimm                   = 0x0F,
> +  MemoryFormFactorDie                      = 0x10
>  } MEMORY_FORM_FACTOR;
> 
>  ///
> @@ -1715,7 +1716,9 @@ typedef enum {
>    MemoryTypeLpddr2                         = 0x1C,
>    MemoryTypeLpddr3                         = 0x1D,
>    MemoryTypeLpddr4                         = 0x1E,
> -  MemoryTypeLogicalNonVolatileDevice       = 0x1F
> +  MemoryTypeLogicalNonVolatileDevice       = 0x1F,
> +  MemoryTypeHBM                            = 0x20,
> +  MemoryTypeHBM2                           = 0x21
>  } MEMORY_DEVICE_TYPE;
> 
>  ///
> --
> 2.21.0.windows.1
> 
> 
>