summary refs log tree commit diff
path: root/arch/arm/include
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2021-06-17 11:50:49 +0100
committerRussell King (Oracle) <rmk+kernel@armlinux.org.uk>2021-06-21 11:39:39 +0100
commite17362d683fb6bcda0e419ec0ad7cabb8252c509 (patch)
tree380bb31b60456ab791aafc27d657864322c3ab54 /arch/arm/include
parent33f087577ed3a048e65e7b50c92704e2f43bd1f7 (diff)
downloadlinux-e17362d683fb6bcda0e419ec0ad7cabb8252c509.tar.gz
ARM: 9097/1: mmu: Declare section start/end correctly
The kernel test robot reported an interesting bug:

A debug print was using %08x with kernel_sec_start and kernel_sec_end
being phys_addr_t which can be either u32 or u64 (possibly more).

Actually these should just be declared as u32 to begin with: they are
declared as such in the assembly in head.S and the kernel definitely
boots in a 32 bit physical address space. Redeclare the kernel_sec_start
and kernel_sec_end to rid the bug.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: 6e121df14ccd ("ARM: 9090/1: Map the lowmem and kernel separately")
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Diffstat (limited to 'arch/arm/include')
-rw-r--r--arch/arm/include/asm/memory.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/arm/include/asm/memory.h b/arch/arm/include/asm/memory.h
index ca213d7d095f..cfc9dfd70aad 100644
--- a/arch/arm/include/asm/memory.h
+++ b/arch/arm/include/asm/memory.h
@@ -162,8 +162,8 @@ extern unsigned long vectors_base;
  * Physical start and end address of the kernel sections. These addresses are
  * 2MB-aligned to match the section mappings placed over the kernel.
  */
-extern phys_addr_t kernel_sec_start;
-extern phys_addr_t kernel_sec_end;
+extern u32 kernel_sec_start;
+extern u32 kernel_sec_end;
 
 /*
  * Physical vs virtual RAM address space conversion.  These are