Commit a18b6584 authored by Lennert Buytenhek's avatar Lennert Buytenhek Committed by Lennert Buytenhek

[ARM] Orion: make window setup a little more safe

Currently, Orion window setup uses hardcoded window indexes for each
of the boot/cs0/cs1/cs2/PCIe WA windows.  The static window allocation
used can clash if board support code will ever attempt to configure
both a dev2 and a PCIe WA window, as both of those use CPU mbus window
#7 at present.

This patch keeps track of the last used window, and opens subsequently
requested windows sequentially, starting from 4.  (Windows 0-3 are used
as MEM/IO windows for the PCI/PCIe buses.)
Signed-off-by: default avatarLennert Buytenhek <buytenh@marvell.com>
parent e7068ad3
...@@ -70,6 +70,7 @@ ...@@ -70,6 +70,7 @@
struct mbus_dram_target_info orion5x_mbus_dram_info; struct mbus_dram_target_info orion5x_mbus_dram_info;
static int __initdata win_alloc_count;
static int __init orion5x_cpu_win_can_remap(int win) static int __init orion5x_cpu_win_can_remap(int win)
{ {
...@@ -87,6 +88,12 @@ static int __init orion5x_cpu_win_can_remap(int win) ...@@ -87,6 +88,12 @@ static int __init orion5x_cpu_win_can_remap(int win)
static void __init setup_cpu_win(int win, u32 base, u32 size, static void __init setup_cpu_win(int win, u32 base, u32 size,
u8 target, u8 attr, int remap) u8 target, u8 attr, int remap)
{ {
if (win >= 8) {
printk(KERN_ERR "setup_cpu_win: trying to allocate "
"window %d\n", win);
return;
}
orion5x_write(CPU_WIN_BASE(win), base & 0xffff0000); orion5x_write(CPU_WIN_BASE(win), base & 0xffff0000);
orion5x_write(CPU_WIN_CTRL(win), orion5x_write(CPU_WIN_CTRL(win),
((size - 1) & 0xffff0000) | (attr << 8) | (target << 4) | 1); ((size - 1) & 0xffff0000) | (attr << 8) | (target << 4) | 1);
...@@ -128,6 +135,7 @@ void __init orion5x_setup_cpu_mbus_bridge(void) ...@@ -128,6 +135,7 @@ void __init orion5x_setup_cpu_mbus_bridge(void)
TARGET_PCIE, ATTR_PCIE_MEM, -1); TARGET_PCIE, ATTR_PCIE_MEM, -1);
setup_cpu_win(3, ORION5X_PCI_MEM_PHYS_BASE, ORION5X_PCI_MEM_SIZE, setup_cpu_win(3, ORION5X_PCI_MEM_PHYS_BASE, ORION5X_PCI_MEM_SIZE,
TARGET_PCI, ATTR_PCI_MEM, -1); TARGET_PCI, ATTR_PCI_MEM, -1);
win_alloc_count = 4;
/* /*
* Setup MBUS dram target info. * Setup MBUS dram target info.
...@@ -156,25 +164,30 @@ void __init orion5x_setup_cpu_mbus_bridge(void) ...@@ -156,25 +164,30 @@ void __init orion5x_setup_cpu_mbus_bridge(void)
void __init orion5x_setup_dev_boot_win(u32 base, u32 size) void __init orion5x_setup_dev_boot_win(u32 base, u32 size)
{ {
setup_cpu_win(4, base, size, TARGET_DEV_BUS, ATTR_DEV_BOOT, -1); setup_cpu_win(win_alloc_count++, base, size,
TARGET_DEV_BUS, ATTR_DEV_BOOT, -1);
} }
void __init orion5x_setup_dev0_win(u32 base, u32 size) void __init orion5x_setup_dev0_win(u32 base, u32 size)
{ {
setup_cpu_win(5, base, size, TARGET_DEV_BUS, ATTR_DEV_CS0, -1); setup_cpu_win(win_alloc_count++, base, size,
TARGET_DEV_BUS, ATTR_DEV_CS0, -1);
} }
void __init orion5x_setup_dev1_win(u32 base, u32 size) void __init orion5x_setup_dev1_win(u32 base, u32 size)
{ {
setup_cpu_win(6, base, size, TARGET_DEV_BUS, ATTR_DEV_CS1, -1); setup_cpu_win(win_alloc_count++, base, size,
TARGET_DEV_BUS, ATTR_DEV_CS1, -1);
} }
void __init orion5x_setup_dev2_win(u32 base, u32 size) void __init orion5x_setup_dev2_win(u32 base, u32 size)
{ {
setup_cpu_win(7, base, size, TARGET_DEV_BUS, ATTR_DEV_CS2, -1); setup_cpu_win(win_alloc_count++, base, size,
TARGET_DEV_BUS, ATTR_DEV_CS2, -1);
} }
void __init orion5x_setup_pcie_wa_win(u32 base, u32 size) void __init orion5x_setup_pcie_wa_win(u32 base, u32 size)
{ {
setup_cpu_win(7, base, size, TARGET_PCIE, ATTR_PCIE_WA, -1); setup_cpu_win(win_alloc_count++, base, size,
TARGET_PCIE, ATTR_PCIE_WA, -1);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment