public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa
@ 2017-06-21 13:34 Star Zeng
  2017-06-21 13:48 ` Gao, Liming
  2017-06-22  2:11 ` Ni, Ruiyu
  0 siblings, 2 replies; 3+ messages in thread
From: Star Zeng @ 2017-06-21 13:34 UTC (permalink / raw)
  To: edk2-devel; +Cc: Star Zeng, Liming Gao, Ruiyu Ni

e286e118bafa updated *.c file, but forgot to update *.h file.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
 DuetPkg/FSVariable/FSVariable.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/DuetPkg/FSVariable/FSVariable.h b/DuetPkg/FSVariable/FSVariable.h
index cc39323e97d8..cb37c29cc2a5 100644
--- a/DuetPkg/FSVariable/FSVariable.h
+++ b/DuetPkg/FSVariable/FSVariable.h
@@ -6,7 +6,7 @@ disk. They can be changed by user. BIOS is not able to protoect those.
 Duet trusts all meta data from disk. If variable code, variable metadata and variable
 data is modified in inproper way, the behavior is undefined.
 
-Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials                          
 are licensed and made available under the terms and conditions of the BSD License         
 which accompanies this distribution.  The full text of the license may be found at        
@@ -129,7 +129,7 @@ DuetGetVariable (
   IN      EFI_GUID          *VendorGuid,
   OUT     UINT32            *Attributes OPTIONAL,
   IN OUT  UINTN             *DataSize,
-  OUT     VOID              *Data
+  OUT     VOID              *Data OPTIONAL
   );
 
 EFI_STATUS
-- 
2.7.0.windows.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa
  2017-06-21 13:34 [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa Star Zeng
@ 2017-06-21 13:48 ` Gao, Liming
  2017-06-22  2:11 ` Ni, Ruiyu
  1 sibling, 0 replies; 3+ messages in thread
From: Gao, Liming @ 2017-06-21 13:48 UTC (permalink / raw)
  To: Zeng, Star, edk2-devel@lists.01.org; +Cc: Ni, Ruiyu

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

> -----Original Message-----
> From: Zeng, Star
> Sent: Wednesday, June 21, 2017 9:34 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>; Gao, Liming <liming.gao@intel.com>; Ni, Ruiyu <ruiyu.ni@intel.com>
> Subject: [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa
> 
> e286e118bafa updated *.c file, but forgot to update *.h file.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
>  DuetPkg/FSVariable/FSVariable.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/DuetPkg/FSVariable/FSVariable.h b/DuetPkg/FSVariable/FSVariable.h
> index cc39323e97d8..cb37c29cc2a5 100644
> --- a/DuetPkg/FSVariable/FSVariable.h
> +++ b/DuetPkg/FSVariable/FSVariable.h
> @@ -6,7 +6,7 @@ disk. They can be changed by user. BIOS is not able to protoect those.
>  Duet trusts all meta data from disk. If variable code, variable metadata and variable
>  data is modified in inproper way, the behavior is undefined.
> 
> -Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
>  This program and the accompanying materials
>  are licensed and made available under the terms and conditions of the BSD License
>  which accompanies this distribution.  The full text of the license may be found at
> @@ -129,7 +129,7 @@ DuetGetVariable (
>    IN      EFI_GUID          *VendorGuid,
>    OUT     UINT32            *Attributes OPTIONAL,
>    IN OUT  UINTN             *DataSize,
> -  OUT     VOID              *Data
> +  OUT     VOID              *Data OPTIONAL
>    );
> 
>  EFI_STATUS
> --
> 2.7.0.windows.1



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa
  2017-06-21 13:34 [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa Star Zeng
  2017-06-21 13:48 ` Gao, Liming
@ 2017-06-22  2:11 ` Ni, Ruiyu
  1 sibling, 0 replies; 3+ messages in thread
From: Ni, Ruiyu @ 2017-06-22  2:11 UTC (permalink / raw)
  To: Zeng, Star, edk2-devel@lists.01.org; +Cc: Gao, Liming, Zeng, Star

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
> Star Zeng
> Sent: Wednesday, June 21, 2017 9:34 PM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu <ruiyu.ni@intel.com>; Gao, Liming <liming.gao@intel.com>;
> Zeng, Star <star.zeng@intel.com>
> Subject: [edk2] [PATCH] DuetPkg FSVariable: Add missing change in
> e286e118bafa
> 
> e286e118bafa updated *.c file, but forgot to update *.h file.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
>  DuetPkg/FSVariable/FSVariable.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/DuetPkg/FSVariable/FSVariable.h
> b/DuetPkg/FSVariable/FSVariable.h index cc39323e97d8..cb37c29cc2a5
> 100644
> --- a/DuetPkg/FSVariable/FSVariable.h
> +++ b/DuetPkg/FSVariable/FSVariable.h
> @@ -6,7 +6,7 @@ disk. They can be changed by user. BIOS is not able to
> protoect those.
>  Duet trusts all meta data from disk. If variable code, variable metadata and
> variable  data is modified in inproper way, the behavior is undefined.
> 
> -Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
>  This program and the accompanying materials
>  are licensed and made available under the terms and conditions of the BSD
> License
>  which accompanies this distribution.  The full text of the license may be
> found at
> @@ -129,7 +129,7 @@ DuetGetVariable (
>    IN      EFI_GUID          *VendorGuid,
>    OUT     UINT32            *Attributes OPTIONAL,
>    IN OUT  UINTN             *DataSize,
> -  OUT     VOID              *Data
> +  OUT     VOID              *Data OPTIONAL
>    );
> 
>  EFI_STATUS
> --
> 2.7.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] 3+ messages in thread

end of thread, other threads:[~2017-06-22  2:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-21 13:34 [PATCH] DuetPkg FSVariable: Add missing change in e286e118bafa Star Zeng
2017-06-21 13:48 ` Gao, Liming
2017-06-22  2:11 ` Ni, Ruiyu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox