Commit 7f69f8a4 authored by Jason Gunthorpe's avatar Jason Gunthorpe Committed by Jason Cooper

ARM: kirkwood: Remove kirkwood_setup_wins and rely on the DT binding

kirkwood_setup_wins is the last manual caller of mbus in kirkwood, don't
call it for DT boards and rely on the DT having a mbus node with
a proper ranges property to setup these windows.

Move all the mbus ranges properties for all boards into kirkwood.dtsi,
since they are currently all the same.

This makes the DT self consistent, since the physical address of the
NAND and CRYPTO are both referenced internally. The arbitary Linux
constants KIRKWOOD_NAND_MEM_PHYS_BASE and KIRKWOOD_SRAM_PHYS_BASE
no longer have to match the DT values.
Signed-off-by: default avatarJason Gunthorpe <jgunthorpe@obsidianresearch.com>
Acked-by: default avatarEzequiel Garcia <ezequiel.garcia@free-electrons.com>
Tested-by: default avatarEzequiel Garcia <ezequiel.garcia@free-electrons.com>
Signed-off-by: default avatarJason Cooper <jason@lakedaemon.net>
parent 37078732
...@@ -19,7 +19,6 @@ / { ...@@ -19,7 +19,6 @@ / {
compatible = "marvell,db-88f6281-bp", "marvell,kirkwood-88f6281", "marvell,kirkwood"; compatible = "marvell,db-88f6281-bp", "marvell,kirkwood-88f6281", "marvell,kirkwood";
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -19,7 +19,6 @@ / { ...@@ -19,7 +19,6 @@ / {
compatible = "marvell,db-88f6282-bp", "marvell,kirkwood-88f6282", "marvell,kirkwood"; compatible = "marvell,db-88f6282-bp", "marvell,kirkwood-88f6282", "marvell,kirkwood";
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -19,7 +19,6 @@ chosen { ...@@ -19,7 +19,6 @@ chosen {
}; };
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -17,7 +17,6 @@ chosen { ...@@ -17,7 +17,6 @@ chosen {
}; };
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -17,7 +17,6 @@ chosen { ...@@ -17,7 +17,6 @@ chosen {
}; };
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -15,7 +15,6 @@ chosen { ...@@ -15,7 +15,6 @@ chosen {
}; };
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
/ { / {
mbus { mbus {
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000>;
pcie-controller { pcie-controller {
status = "okay"; status = "okay";
......
...@@ -27,6 +27,11 @@ mbus { ...@@ -27,6 +27,11 @@ mbus {
compatible = "marvell,kirkwood-mbus", "simple-bus"; compatible = "marvell,kirkwood-mbus", "simple-bus";
#address-cells = <2>; #address-cells = <2>;
#size-cells = <1>; #size-cells = <1>;
/* If a board file needs to change this ranges it must replace it completely */
ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000 /* internal-regs */
MBUS_ID(0x01, 0x2f) 0 0xf4000000 0x10000 /* nand flash */
MBUS_ID(0x03, 0x01) 0 0xf5000000 0x10000 /* crypto sram */
>;
controller = <&mbusc>; controller = <&mbusc>;
pcie-mem-aperture = <0xe0000000 0x10000000>; /* 256 MiB memory space */ pcie-mem-aperture = <0xe0000000 0x10000000>; /* 256 MiB memory space */
pcie-io-aperture = <0xf2000000 0x100000>; /* 1 MiB I/O space */ pcie-io-aperture = <0xf2000000 0x100000>; /* 1 MiB I/O space */
......
...@@ -92,7 +92,6 @@ static void __init kirkwood_dt_init(void) ...@@ -92,7 +92,6 @@ static void __init kirkwood_dt_init(void)
writel(readl(CPU_CONFIG) & ~CPU_CONFIG_ERROR_PROP, CPU_CONFIG); writel(readl(CPU_CONFIG) & ~CPU_CONFIG_ERROR_PROP, CPU_CONFIG);
BUG_ON(mvebu_mbus_dt_init()); BUG_ON(mvebu_mbus_dt_init());
kirkwood_setup_wins();
kirkwood_l2_init(); kirkwood_l2_init();
......
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