[PATCH] x86: remove obsolete comment in uapi/e820.h

[PATCH] x86: remove obsolete comment in uapi/e820.h

[Ross Zwisler]

A comment introduced by this commit:

028b785888c5 x86 boot: extend some internal memory map arrays to handle
	larger EFI input

had to do with some nested preprocessor directives.  The directives were
split into separate files by this commit:

af170c5061dd  UAPI: (Scripted) Disintegrate arch/x86/include/asm

The comment explaining their interaction was retained and is now present
in arch/x86/include/uapi/asm/e820.h.  This comment is no longer correct,
so delete it.

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>

Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: x86@kernel.org
---
 arch/x86/include/uapi/asm/e820.h | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/arch/x86/include/uapi/asm/e820.h b/arch/x86/include/uapi/asm/e820.h
index bbae024..d993e33 100644
--- a/arch/x86/include/uapi/asm/e820.h
+++ b/arch/x86/include/uapi/asm/e820.h
@@ -21,11 +21,6 @@
  * this size.
  */
 
-/*
- * Odd: 'make headers_check' complains about numa.h if I try
- * to collapse the next two #ifdef lines to a single line:
- *	#if defined(__KERNEL__) && defined(CONFIG_EFI)
- */
 #ifndef __KERNEL__
 #define E820_X_MAX E820MAX
 #endif
-- 
1.9.0

Re: [PATCH] x86: remove obsolete comment in uapi/e820.h

[Ross Zwisler]

On Mon, 2014-05-19 at 11:50 -0600, Ross Zwisler wrote:
> A comment introduced by this commit:
> 
> 028b785888c5 x86 boot: extend some internal memory map arrays to handle
> 	larger EFI input
> 
> had to do with some nested preprocessor directives.  The directives were
> split into separate files by this commit:
> 
> af170c5061dd  UAPI: (Scripted) Disintegrate arch/x86/include/asm
> 
> The comment explaining their interaction was retained and is now present
> in arch/x86/include/uapi/asm/e820.h.  This comment is no longer correct,
> so delete it.
> 
> Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> 
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: "H. Peter Anvin" <hpa@zytor.com>
> Cc: x86@kernel.org
> ---
>  arch/x86/include/uapi/asm/e820.h | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/arch/x86/include/uapi/asm/e820.h b/arch/x86/include/uapi/asm/e820.h
> index bbae024..d993e33 100644
> --- a/arch/x86/include/uapi/asm/e820.h
> +++ b/arch/x86/include/uapi/asm/e820.h
> @@ -21,11 +21,6 @@
>   * this size.
>   */
>  
> -/*
> - * Odd: 'make headers_check' complains about numa.h if I try
> - * to collapse the next two #ifdef lines to a single line:
> - *	#if defined(__KERNEL__) && defined(CONFIG_EFI)
> - */
>  #ifndef __KERNEL__
>  #define E820_X_MAX E820MAX
>  #endif

Ping.  :)

[tip:x86/cleanups] x86: Remove obsolete comment in uapi/e820.h

[tip-bot for Ross Zwisler]

Commit-ID:  448466b723cc7b44016547116aa2b55a3736f1f0
Gitweb:     http://git.kernel.org/tip/448466b723cc7b44016547116aa2b55a3736f1f0
Author:     Ross Zwisler <ross.zwisler@linux.intel.com>
AuthorDate: Mon, 19 May 2014 11:50:24 -0600
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Thu, 21 Aug 2014 08:43:39 +0200

x86: Remove obsolete comment in uapi/e820.h

A comment introduced by this old commit:

  028b785888c5 ("x86 boot: extend some internal memory map arrays to handle larger EFI input")

had to do with some nested preprocessor directives.  The
directives were split into separate files by this commit:

  af170c5061dd ("UAPI: (Scripted) Disintegrate arch/x86/include/asm")

The comment explaining their interaction was retained and is now
present in arch/x86/include/uapi/asm/e820.h.  This comment is no
longer correct, so delete it.

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
Link: http://lkml.kernel.org/r/1400521824-21040-1-git-send-email-ross.zwisler@linux.intel.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 arch/x86/include/uapi/asm/e820.h | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/arch/x86/include/uapi/asm/e820.h b/arch/x86/include/uapi/asm/e820.h
index bbae024..d993e33 100644
--- a/arch/x86/include/uapi/asm/e820.h
+++ b/arch/x86/include/uapi/asm/e820.h
@@ -21,11 +21,6 @@
  * this size.
  */
 
-/*
- * Odd: 'make headers_check' complains about numa.h if I try
- * to collapse the next two #ifdef lines to a single line:
- *	#if defined(__KERNEL__) && defined(CONFIG_EFI)
- */
 #ifndef __KERNEL__
 #define E820_X_MAX E820MAX
 #endif