Re: [PATCH 5/9] ARM: kirkwood: use devicetree for orion-spi
From: Jason Cooper <hidden>
Date: 2012-06-10 17:06:20
Also in:
linux-arm-kernel, linux-i2c, linux-spi
Possibly related (same subject, not in this thread)
- 2012-06-14 · Re: [PATCH 5/9] ARM: kirkwood: use devicetree for orion-spi · Arnd Bergmann <hidden>
- 2012-06-14 · Re: [PATCH 5/9] ARM: kirkwood: use devicetree for orion-spi · Andrew Lunn <hidden>
- 2012-06-13 · Re: [PATCH 5/9] ARM: kirkwood: use devicetree for orion-spi · Arnd Bergmann <hidden>
- 2012-06-12 · Re: [PATCH 5/9] ARM: kirkwood: use devicetree for orion-spi · Jason Cooper <hidden>
- 2012-06-12 · Re: [PATCH 5/9] ARM: kirkwood: use devicetree for orion-spi · Andrew Lunn <hidden>
On Sun, Jun 10, 2012 at 12:31:57PM +0200, Andrew Lunn wrote:
quoted hunk
From: Michael Walle <redacted> Populate the devices with auxdata to set the device names which are used by clkdev to lookup the clocks. Signed-off-by: Michael Walle <redacted> Signed-off-by: Andrew Lunn <redacted> --- arch/arm/boot/dts/kirkwood.dtsi | 9 +++++++++ arch/arm/mach-kirkwood/board-dt.c | 8 +++++++- 2 files changed, 16 insertions(+), 1 deletion(-)diff --git a/arch/arm/boot/dts/kirkwood.dtsi b/arch/arm/boot/dts/kirkwood.dtsi index 8eab7c4..7d282ab 100644 --- a/arch/arm/boot/dts/kirkwood.dtsi +++ b/arch/arm/boot/dts/kirkwood.dtsi@@ -42,6 +42,15 @@ interrupts = <53>; }; + spi@10600 { + compatible = "marvell,orion-spi"; + #address-cells = <1>; + #size-cells = <0>; + cell-index = <0>; + reg = <0x10600 0x28>; + status = "disabled"; + }; + nand@3000000 { #address-cells = <1>; #size-cells = <1>;diff --git a/arch/arm/mach-kirkwood/board-dt.c b/arch/arm/mach-kirkwood/board-dt.c index fa51586..0942139 100644 --- a/arch/arm/mach-kirkwood/board-dt.c +++ b/arch/arm/mach-kirkwood/board-dt.c@@ -26,6 +26,11 @@ static struct of_device_id kirkwood_dt_match_table[] __initdata = { { } }; +struct of_dev_auxdata kirkwood_auxdata_lookup[] __initdata = { + OF_DEV_AUXDATA("marvell,orion-spi", 0xf1010600, "orion_spi.0", NULL),
Isn't this ---------------------------------^^^^^^^^^^ defined somewhere? This is done about 1/4 of the time (56 / 187) in the kernel. But honestly, I'd prefer it to be named. Other than that, Acked-by: Jason Cooper <redacted>
quoted hunk
+ {}, +}; + static void __init kirkwood_dt_init(void) { pr_info("Kirkwood: %s, TCLK=%d.\n", kirkwood_id(), kirkwood_tclk);@@ -69,7 +74,8 @@ static void __init kirkwood_dt_init(void) if (of_machine_is_compatible("raidsonic,ib-nas62x0")) ib62x0_init(); - of_platform_populate(NULL, kirkwood_dt_match_table, NULL, NULL); + of_platform_populate(NULL, kirkwood_dt_match_table, + kirkwood_auxdata_lookup, NULL); } static const char *kirkwood_dt_board_compat[] = {-- 1.7.10