MPC834x BCSR_SIZE too small for use in a BAT
PATCH 1/1 MPC834x BCSR_SIZE too small for use in a BAT
Greetings,
The value of BCSR_SIZE (32K) in arch/ppc/platforms/83xx/mpc834x_sys.h
is too small for a BAT (min 128K). The size is used in an io_block_mapping call to map the Board Control and Status Registers. Since there are no current dependencies on the result of the io_block_mapping call, it was removed as a solution to the problem rather than increasing the value of BCSR_SIZE. If access to the BCSR block is needed, it can easily be mapped via ioremap at the point of use.
Randy Vinson
MPC834x BCSR_SIZE too small for use in a BAT.
The call to io_block_mapping was creating an invalid BAT entry because the value of BCSR_SIZE (32K) is too small to be used in a BAT (128K min). This change removes the io_block_mapping call since these registers can easily be mapped using ioremap at the point of use.
---
commit 269bcdbf0a9f000400052b2da38348733dbfe88f
tree d10f5c31c29228a00a1a7551e3e825b439bfc741
parent 1b981021e88c8c28dc6a22cb1a327d8dc9296473
author Randy Vinson <rvinson@linuxbox.(none)> Wed, 25 May 2005 16:14:40 -0700 committer Randy Vinson <rvinson@linuxbox.(none)> Wed, 25 May 2005 16:14:40 -0700
arch/ppc/platforms/83xx/mpc834x_sys.c | 1 - arch/ppc/platforms/83xx/mpc834x_sys.h | 1 - 2 files changed, 2 deletions(-)
arch/ppc/platforms/83xx/mpc834x_sys.c (mode:100644) +0 -1
@@ -127,7 +127,6 @@ { /* we steal the lowest ioremap addr for virt space */ io_block_mapping(VIRT_IMMRBAR, immrbar, 1024*1024, _PAGE_IO); - io_block_mapping(BCSR_VIRT_ADDR, BCSR_PHYS_ADDR, BCSR_SIZE, _PAGE_IO); } int
arch/ppc/platforms/83xx/mpc834x_sys.h (mode:100644) +0 -1
@@ -26,7 +26,6 @@ #define VIRT_IMMRBAR ((uint)0xfe000000) #define BCSR_PHYS_ADDR ((uint)0xf8000000) -#define BCSR_VIRT_ADDR ((uint)0xfe100000) #define BCSR_SIZE ((uint)(32 * 1024)) #ifdef CONFIG_PCI