public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Marc W Chen" <marc.w.chen@intel.com>
To: "Gao, Liming" <liming.gao@intel.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"Ni, Ray" <ray.ni@intel.com>
Subject: Re: [edk2-devel] [PATCH V2] MdePkg: Add PI 1.5 SmramMemoryReserve HOB file
Date: Mon, 29 Jul 2019 08:40:27 +0000	[thread overview]
Message-ID: <AF60760DA41C634EBADDE512AA3D4E537E302145@PGSMSX108.gar.corp.intel.com> (raw)
In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4C455E@SHSMSX104.ccr.corp.intel.com>

Sure.

Thanks,
Marc

> -----Original Message-----
> From: Gao, Liming
> Sent: Monday, July 29, 2019 4:29 PM
> To: devel@edk2.groups.io; Chen, Marc W <marc.w.chen@intel.com>
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray
> <ray.ni@intel.com>
> Subject: RE: [edk2-devel] [PATCH V2] MdePkg: Add PI 1.5
> SmramMemoryReserve HOB file
> 
> Marc:
>   Normally, gEfiGuidCName matches GUID C Macro name. Can you follow this
> rule?
> 
> Thanks
> Liming
> >-----Original Message-----
> >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> >Marc W Chen
> >Sent: Monday, July 29, 2019 4:05 PM
> >To: devel@edk2.groups.io
> >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> ><liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>
> >Subject: [edk2-devel] [PATCH V2] MdePkg: Add PI 1.5
> SmramMemoryReserve
> >HOB file
> >
> >This is a special GUID extension Hob to describe SMRAM memory regions
> >
> >Cc: Michael D Kinney <michael.d.kinney@intel.com>
> >Cc: Liming Gao <liming.gao@intel.com>
> >Cc: Ray Ni <ray.ni@intel.com>
> >Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2020
> >Signed-off-by: Marc W Chen <marc.w.chen@intel.com>
> >---
> > MdePkg/Include/Guid/SmramMemoryReserve.h | 45
> >++++++++++++++++++++++++++++++++
> > MdePkg/MdePkg.dec                        |  3 +++
> > 2 files changed, 48 insertions(+)
> > create mode 100644 MdePkg/Include/Guid/SmramMemoryReserve.h
> >
> >diff --git a/MdePkg/Include/Guid/SmramMemoryReserve.h
> >b/MdePkg/Include/Guid/SmramMemoryReserve.h
> >new file mode 100644
> >index 0000000000..3ba8c6f526
> >--- /dev/null
> >+++ b/MdePkg/Include/Guid/SmramMemoryReserve.h
> >@@ -0,0 +1,45 @@
> >+/** @file
> >+  This is a special GUID extension Hob to describe SMRAM memory regions.
> >+
> >+  This file defines:
> >+  * the GUID used to identify the GUID HOB for reserving SMRAM regions.
> >+  * the data structure of SMRAM descriptor to describe SMRAM candidate
> >regions
> >+  * values of state of SMRAM candidate regions
> >+  * the GUID specific data structure of HOB for reserving SMRAM regions.
> >+
> >+  Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> >+  SPDX-License-Identifier: BSD-2-Clause-Patent
> >+
> >+  @par Revision Reference:
> >+  GUIDs defined in PI SPEC version 1.5.
> >+
> >+**/
> >+
> >+#ifndef _SMRAM_MEMORY_RESERVE_H_
> >+#define _SMRAM_MEMORY_RESERVE_H_
> >+
> >+#define EFI_SMM_SMRAM_MEMORY_GUID \
> >+  { \
> >+    0x6dadf1d1, 0xd4cc, 0x4910, {0xbb, 0x6e, 0x82, 0xb1, 0xfd, 0x80, 0xff,
> 0x3d }
> >\
> >+  }
> >+
> >+/**
> >+* The GUID extension hob is to describe SMRAM memory regions
> supported
> >by the platform.
> >+**/
> >+typedef struct {
> >+  ///
> >+  /// Designates the number of possible regions in the system
> >+  /// that can be usable for SMRAM.
> >+  ///
> >+  UINT32                NumberOfSmmReservedRegions;
> >+  ///
> >+  /// Used throughout this protocol to describe the candidate
> >+  /// regions for SMRAM that are supported by this platform.
> >+  ///
> >+  EFI_SMRAM_DESCRIPTOR  Descriptor[1];
> >+} EFI_SMRAM_HOB_DESCRIPTOR_BLOCK;
> >+
> >+extern EFI_GUID gEfiSmmSmramMemoryReserveGuid;
> >+
> >+#endif
> >+
> >diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec
> >index b382efd578..ccfa220a02 100644
> >--- a/MdePkg/MdePkg.dec
> >+++ b/MdePkg/MdePkg.dec
> >@@ -749,6 +749,9 @@
> >   ## Include/Guid/GraphicsInfoHob.h
> >   gEfiGraphicsDeviceInfoHobGuid     = { 0xe5cb2ac9, 0xd35d, 0x4430, { 0x93,
> >0x6e, 0x1d, 0xe3, 0x32, 0x47, 0x8d, 0xe7 }}
> >
> >+  ## Include/Guid/SmramMemoryReserve.h
> >+  gEfiSmmSmramMemoryReserveGuid     = { 0x6dadf1d1, 0xd4cc, 0x4910,
> >{ 0xbb, 0x6e, 0x82, 0xb1, 0xfd, 0x80, 0xff, 0x3d }}
> >+
> >   #
> >   # GUID defined in PI1.6
> >   #
> >--
> >2.16.2.windows.1
> >
> >
> >


      reply	other threads:[~2019-07-29  8:52 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-29  8:04 [PATCH V2] MdePkg: Add PI 1.5 SmramMemoryReserve HOB file Marc W Chen
2019-07-29  8:29 ` [edk2-devel] " Liming Gao
2019-07-29  8:40   ` Marc W Chen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AF60760DA41C634EBADDE512AA3D4E537E302145@PGSMSX108.gar.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox