public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH 1/1] OvmfPkg/IoMmuDxe: Fix header guard macro
@ 2017-08-04 19:18 Thomas Palmer
  2017-08-04 20:00 ` Laszlo Ersek
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Palmer @ 2017-08-04 19:18 UTC (permalink / raw)
  To: edk2-devel
  Cc: jordan.l.justen, lersek, brijesh.singh, joseph.shifflett,
	Thomas Palmer

Correct the header guard macro

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Palmer <thomas.palmer@hpe.com>
---
 OvmfPkg/IoMmuDxe/AmdSevIoMmu.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
index 8b3962a8c395..47428c235090 100644
--- a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
+++ b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
@@ -7,6 +7,7 @@
 
   Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
   Copyright (c) 2017, AMD Inc. All rights reserved.<BR>
+  (C) Copyright 2017 Hewlett Packard Enterprise Development LP<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
@@ -18,7 +19,7 @@
 **/
 
 #ifndef __AMD_SEV_IOMMU_H_
-#define __AMD_SEV_IOMMU_H
+#define __AMD_SEV_IOMMU_H_
 
 #include <Protocol/IoMmu.h>
 
-- 
2.7.4



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

* Re: [PATCH 1/1] OvmfPkg/IoMmuDxe: Fix header guard macro
  2017-08-04 19:18 [PATCH 1/1] OvmfPkg/IoMmuDxe: Fix header guard macro Thomas Palmer
@ 2017-08-04 20:00 ` Laszlo Ersek
  2017-08-04 20:02   ` Palmer, Thomas
  0 siblings, 1 reply; 3+ messages in thread
From: Laszlo Ersek @ 2017-08-04 20:00 UTC (permalink / raw)
  To: Thomas Palmer, edk2-devel
  Cc: jordan.l.justen, brijesh.singh, joseph.shifflett

On 08/04/17 21:18, Thomas Palmer wrote:
> Correct the header guard macro
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Thomas Palmer <thomas.palmer@hpe.com>
> ---
>  OvmfPkg/IoMmuDxe/AmdSevIoMmu.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
> index 8b3962a8c395..47428c235090 100644
> --- a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
> +++ b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
> @@ -7,6 +7,7 @@
>  
>    Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
>    Copyright (c) 2017, AMD Inc. All rights reserved.<BR>
> +  (C) Copyright 2017 Hewlett Packard Enterprise Development LP<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
> @@ -18,7 +19,7 @@
>  **/
>  
>  #ifndef __AMD_SEV_IOMMU_H_
> -#define __AMD_SEV_IOMMU_H
> +#define __AMD_SEV_IOMMU_H_
>  
>  #include <Protocol/IoMmu.h>
>  
> 

Thank you, Thomas. Could you please submit a version that changes all
instances of the macro to:

  __AMD_SEV_IOMMU_H__

or

  _AMD_SEV_IOMMU_H_

?

Both are conventional in edk2. The number of underscores on the two
sides should match.

Thank you,
Laszlo



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

* Re: [PATCH 1/1] OvmfPkg/IoMmuDxe: Fix header guard macro
  2017-08-04 20:00 ` Laszlo Ersek
@ 2017-08-04 20:02   ` Palmer, Thomas
  0 siblings, 0 replies; 3+ messages in thread
From: Palmer, Thomas @ 2017-08-04 20:02 UTC (permalink / raw)
  To: Laszlo Ersek, edk2-devel@lists.01.org
  Cc: jordan.l.justen@intel.com, brijesh.singh@amd.com,
	Shifflett, Joseph

roger


Regards,

Thomas Palmer

“I have only made this letter longer because I have not had the time to make it shorter” - Blaise Pascal

-----Original Message-----
From: Laszlo Ersek [mailto:lersek@redhat.com] 
Sent: Friday, August 4, 2017 3:00 PM
To: Palmer, Thomas <thomas.palmer@hpe.com>; edk2-devel@lists.01.org
Cc: jordan.l.justen@intel.com; brijesh.singh@amd.com; Shifflett, Joseph <joseph.shifflett@hpe.com>
Subject: Re: [PATCH 1/1] OvmfPkg/IoMmuDxe: Fix header guard macro

On 08/04/17 21:18, Thomas Palmer wrote:
> Correct the header guard macro
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Thomas Palmer <thomas.palmer@hpe.com>
> ---
>  OvmfPkg/IoMmuDxe/AmdSevIoMmu.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h 
> b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h index 8b3962a8c395..47428c235090 
> 100644
> --- a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
> +++ b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.h
> @@ -7,6 +7,7 @@
>  
>    Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
>    Copyright (c) 2017, AMD Inc. All rights reserved.<BR>
> +  (C) Copyright 2017 Hewlett Packard Enterprise Development LP<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 @@ 
> -18,7 +19,7 @@  **/
>  
>  #ifndef __AMD_SEV_IOMMU_H_
> -#define __AMD_SEV_IOMMU_H
> +#define __AMD_SEV_IOMMU_H_
>  
>  #include <Protocol/IoMmu.h>
>  
> 

Thank you, Thomas. Could you please submit a version that changes all instances of the macro to:

  __AMD_SEV_IOMMU_H__

or

  _AMD_SEV_IOMMU_H_

?

Both are conventional in edk2. The number of underscores on the two sides should match.

Thank you,
Laszlo


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

end of thread, other threads:[~2017-08-04 20:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-04 19:18 [PATCH 1/1] OvmfPkg/IoMmuDxe: Fix header guard macro Thomas Palmer
2017-08-04 20:00 ` Laszlo Ersek
2017-08-04 20:02   ` Palmer, Thomas

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