Cc: Jeff Fan <jeff.fan@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Eric Dong <eric.dong@intel.com>
---
 MdePkg/Include/Protocol/SmmAccess2.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/MdePkg/Include/Protocol/SmmAccess2.h b/MdePkg/Include/Protocol/SmmAccess2.h
index eb315db..5904266 100644
--- a/MdePkg/Include/Protocol/SmmAccess2.h
+++ b/MdePkg/Include/Protocol/SmmAccess2.h
@@ -108,8 +108,9 @@ EFI_STATUS
 
 ///
 ///  EFI SMM Access2 Protocol is used to control the visibility of the SMRAM on the platform.
-///  It abstracts the location and characteristics of SMRAM.  The expectation is
-///  that the north bridge or memory controller would publish this protocol.
+///  It abstracts the location and characteristics of SMRAM. The platform should report all 
+///  MMRAM via EFI_MM_ACCESS_PROTOCOL. The expectation is that the north bridge or memory 
+///  controller would publish this protocol.
 /// 
 struct _EFI_SMM_ACCESS2_PROTOCOL {
   EFI_SMM_OPEN2          Open;
-- 
2.7.0.windows.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

Thanks/Ray

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Eric Dong
> Sent: Wednesday, July 19, 2017 10:38 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu <ruiyu.ni@intel.com>; Fan, Jeff <jeff.fan@intel.com>
> Subject: [edk2] [Patch 1/2] MdePkg SmmAccess2: Update comments to
> follow PI spec.
> 
> Cc: Jeff Fan <jeff.fan@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Eric Dong <eric.dong@intel.com>
> ---
>  MdePkg/Include/Protocol/SmmAccess2.h | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/MdePkg/Include/Protocol/SmmAccess2.h
> b/MdePkg/Include/Protocol/SmmAccess2.h
> index eb315db..5904266 100644
> --- a/MdePkg/Include/Protocol/SmmAccess2.h
> +++ b/MdePkg/Include/Protocol/SmmAccess2.h
> @@ -108,8 +108,9 @@ EFI_STATUS
> 
>  ///
>  ///  EFI SMM Access2 Protocol is used to control the visibility of the SMRAM
> on the platform.
> -///  It abstracts the location and characteristics of SMRAM.  The expectation
> is
> -///  that the north bridge or memory controller would publish this protocol.
> +///  It abstracts the location and characteristics of SMRAM. The platform
> should report all
> +///  MMRAM via EFI_MM_ACCESS_PROTOCOL. The expectation is that the
> north bridge or memory
> +///  controller would publish this protocol.
>  ///
>  struct _EFI_SMM_ACCESS2_PROTOCOL {
>    EFI_SMM_OPEN2          Open;
> --
> 2.7.0.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel