From: Franck Bui-Huu <vagabon.xyz@gmail.com>
To: ralf@linux-mips.org
Cc: linux-mips@linux-mips.org
Subject: [PATCH 2/5] Clean up asm-mips/mach-generic/spaces.h
Date: Mon, 4 Jun 2007 17:46:32 +0200 [thread overview]
Message-ID: <118097199516-git-send-email-fbuihuu@gmail.com> (raw)
In-Reply-To: <1180971995757-git-send-email-fbuihuu@gmail.com>
From: Franck Bui-Huu <fbuihuu@gmail.com>
PAGE_OFFSET definition is now using CAC_BASE by default.
This patch also reorder some macros to make them appear
in the same order for both 32 and 64 bits configs.
It also makes use of const.h generic header file to
annotate constants.
Signed-off-by: Franck Bui-Huu <fbuihuu@gmail.com>
---
include/asm-mips/mach-generic/spaces.h | 66 +++++++++++++------------------
1 files changed, 28 insertions(+), 38 deletions(-)
diff --git a/include/asm-mips/mach-generic/spaces.h b/include/asm-mips/mach-generic/spaces.h
index 9a3c521..c90900b 100644
--- a/include/asm-mips/mach-generic/spaces.h
+++ b/include/asm-mips/mach-generic/spaces.h
@@ -10,74 +10,56 @@
#ifndef _ASM_MACH_GENERIC_SPACES_H
#define _ASM_MACH_GENERIC_SPACES_H
+#include <linux/const.h>
#ifdef CONFIG_32BIT
-#define CAC_BASE 0x80000000
-#define IO_BASE 0xa0000000
-#define UNCAC_BASE 0xa0000000
+#define CAC_BASE _AC(0x80000000,UL)
+#define IO_BASE _AC(0xa0000000,UL)
+#define UNCAC_BASE _AC(0xa0000000,UL)
#ifndef MAP_BASE
-#define MAP_BASE 0xc0000000
-#endif
-
-/*
- * This handles the memory map.
- * We handle pages at KSEG0 for kernels with 32 bit address space.
- */
-#ifndef PAGE_OFFSET
-#define PAGE_OFFSET 0x80000000UL
+#define MAP_BASE _AC(0xc0000000,UL)
#endif
/*
* Memory above this physical address will be considered highmem.
*/
#ifndef HIGHMEM_START
-#define HIGHMEM_START 0x20000000UL
+#define HIGHMEM_START _AC(0x20000000,UL)
#endif
#endif /* CONFIG_32BIT */
#ifdef CONFIG_64BIT
-/*
- * This handles the memory map.
- */
-#ifndef PAGE_OFFSET
-#ifdef CONFIG_DMA_NONCOHERENT
-#define PAGE_OFFSET 0x9800000000000000UL
-#else
-#define PAGE_OFFSET 0xa800000000000000UL
-#endif
-#endif
-
-/*
- * Memory above this physical address will be considered highmem.
- * Fixme: 59 bits is a fictive number and makes assumptions about processors
- * in the distant future. Nobody will care for a few years :-)
- */
-#ifndef HIGHMEM_START
-#define HIGHMEM_START (1UL << 59UL)
-#endif
-
#ifndef CAC_BASE
#ifdef CONFIG_DMA_NONCOHERENT
-#define CAC_BASE 0x9800000000000000UL
+#define CAC_BASE _AC(0x9800000000000000,UL)
#else
-#define CAC_BASE 0xa800000000000000UL
+#define CAC_BASE _AC(0xa800000000000000,UL)
#endif
#endif
#ifndef IO_BASE
-#define IO_BASE 0x9000000000000000UL
+#define IO_BASE _AC(0x9000000000000000,UL)
#endif
#ifndef UNCAC_BASE
-#define UNCAC_BASE 0x9000000000000000UL
+#define UNCAC_BASE _AC(0x9000000000000000,UL)
#endif
#ifndef MAP_BASE
-#define MAP_BASE 0xc000000000000000UL
+#define MAP_BASE _AC(0xc000000000000000,UL)
+#endif
+
+/*
+ * Memory above this physical address will be considered highmem.
+ * Fixme: 59 bits is a fictive number and makes assumptions about processors
+ * in the distant future. Nobody will care for a few years :-)
+ */
+#ifndef HIGHMEM_START
+#define HIGHMEM_START (_AC(1,UL) << _AC(59,UL))
#endif
#define TO_PHYS(x) ( ((x) & TO_PHYS_MASK))
@@ -86,4 +68,12 @@
#endif /* CONFIG_64BIT */
+/*
+ * This handles the memory map.
+ */
+#ifndef PAGE_OFFSET
+#define PAGE_OFFSET CAC_BASE
+#endif
+
+
#endif /* __ASM_MACH_GENERIC_SPACES_H */
--
1.5.1.4
next prev parent reply other threads:[~2007-06-04 15:41 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-04 15:46 [PATCH 0/5] PHYS_OFFSET fix and cleanup [take #2] Franck Bui-Huu
2007-06-04 15:46 ` [PATCH 1/5] Allow generic spaces.h to be included by platform specific ones Franck Bui-Huu
2007-06-07 7:32 ` Ralf Baechle
2007-06-04 15:46 ` Franck Bui-Huu [this message]
2007-06-07 7:32 ` [PATCH 2/5] Clean up asm-mips/mach-generic/spaces.h Ralf Baechle
2007-06-04 15:46 ` [PATCH 3/5] Make PAGE_OFFSET aware of PHYS_OFFSET Franck Bui-Huu
2007-06-07 7:32 ` Ralf Baechle
2007-06-04 15:46 ` [PATCH 4/5] Move PHY_OFFSET definition in spaces.h Franck Bui-Huu
2007-06-07 7:32 ` Ralf Baechle
2007-06-04 15:46 ` [PATCH 5/5] Fix PHYS_OFFSET for 64-bits kernels with 32-bits symbols Franck Bui-Huu
2007-06-07 7:32 ` Ralf Baechle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=118097199516-git-send-email-fbuihuu@gmail.com \
--to=vagabon.xyz@gmail.com \
--cc=linux-mips@linux-mips.org \
--cc=ralf@linux-mips.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox