Use an enum instead of CPP #define. Also fix various small annoyances in the descriptions of the various E820 types.
No change in functionality. Cc: Alex Thorlton <athorl...@sgi.com> Cc: Andy Lutomirski <l...@kernel.org> Cc: Borislav Petkov <b...@alien8.de> Cc: Brian Gerst <brge...@gmail.com> Cc: Dan Williams <dan.j.willi...@intel.com> Cc: Denys Vlasenko <dvlas...@redhat.com> Cc: H. Peter Anvin <h...@zytor.com> Cc: Huang, Ying <ying.hu...@intel.com> Cc: Josh Poimboeuf <jpoim...@redhat.com> Cc: Juergen Gross <jgr...@suse.com> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Paul Jackson <p...@sgi.com> Cc: Peter Zijlstra <pet...@infradead.org> Cc: Rafael J. Wysocki <r...@sisk.pl> Cc: Tejun Heo <t...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Wei Yang <richard.weiy...@gmail.com> Cc: Yinghai Lu <ying...@kernel.org> Cc: linux-kernel@vger.kernel.org Signed-off-by: Ingo Molnar <mi...@kernel.org> --- arch/x86/include/asm/e820/types.h | 57 +++++++++++++++++++++++++++++++-------------------------- 1 file changed, 31 insertions(+), 26 deletions(-) diff --git a/arch/x86/include/asm/e820/types.h b/arch/x86/include/asm/e820/types.h index da9477d78214..1c3426825535 100644 --- a/arch/x86/include/asm/e820/types.h +++ b/arch/x86/include/asm/e820/types.h @@ -1,6 +1,37 @@ #ifndef _ASM_E820_TYPES_H #define _ASM_E820_TYPES_H +enum e820_type { + E820_RAM = 1, + E820_RESERVED = 2, + E820_ACPI = 3, + E820_NVS = 4, + E820_UNUSABLE = 5, + E820_PMEM = 7, + + /* + * This is a non-standardized way to represent ADR or + * NVDIMM regions that persist over a reboot. + * + * The kernel will ignore their special capabilities + * unless the CONFIG_X86_PMEM_LEGACY=y option is set. + * + * ( Note that older platforms also used 6 for the same + * type of memory, but newer versions switched to 12 as + * 6 was assigned differently. Some time they will learn... ) + */ + E820_PRAM = 12, + + /* + * Reserved RAM used by the kernel itself if + * CONFIG_INTEL_TXT=y is enabled, memory of this type + * will be included in the S3 integrity calculation + * and so should not include any memory that the BIOS + * might alter over the S3 transition: + */ + E820_RESERVED_KERN = 128, +}; + #include <uapi/asm/e820/types.h> /* @@ -39,32 +70,6 @@ /* Number of entries in E820MAP: */ #define E820NR 0x1e8 -#define E820_RAM 1 -#define E820_RESERVED 2 -#define E820_ACPI 3 -#define E820_NVS 4 -#define E820_UNUSABLE 5 -#define E820_PMEM 7 - -/* - * This is a non-standardized way to represent ADR or NVDIMM regions that - * persist over a reboot. The kernel will ignore their special capabilities - * unless the CONFIG_X86_PMEM_LEGACY option is set. - * - * ( Note that older platforms also used 6 for the same type of memory, - * but newer versions switched to 12 as 6 was assigned differently. Some - * time they will learn... ) - */ -#define E820_PRAM 12 - -/* - * reserved RAM used by kernel itself - * if CONFIG_INTEL_TXT is enabled, memory of this type will be - * included in the S3 integrity calculation and so should not include - * any memory that BIOS might alter over the S3 transition - */ -#define E820_RESERVED_KERN 128 - /* * The whole array of E820 entries: */ -- 2.7.4