Skip to contentSkip to conversation

MPC834x BCSR_SIZE too small for use in a BAT

PATCH New Randy Vinson 1 participant 1 message May 25, 2005

lore
Share & tools
lore
#0 lore raw

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
Index: arch/ppc/platforms/83xx/mpc834x_sys.c
===================================================================
@@ -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
Index: arch/ppc/platforms/83xx/mpc834x_sys.h
===================================================================
@@ -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