[PATCH] mach-ux500: add devicetree compat nodes

On Fri, Nov 11, 2011 at 4:17 PM, Rob Herring [off-list ref] wrote:

quoted

?MACHINE_START(U8500, "ST-Ericsson MOP500 platform")
? ? ? /* Maintainer: Srinidhi Kasagar [off-list ref] */
? ? ? .boot_params ? ?= 0x100,

@@ -651,6 +666,7 @@ MACHINE_START(U8500, "ST-Ericsson MOP500 platform")

? ? ? /* we re-use nomadik timer here */
? ? ? .timer ? ? ? ? ?= &ux500_timer,
? ? ? .init_machine ? = mop500_init_machine,
+ ? ? .dt_compat ? ? ?= mop500_dt_board_compat,
?MACHINE_END

?MACHINE_START(HREFV60, "ST-Ericsson U8500 Platform HREFv60+")

@@ -659,6 +675,7 @@ MACHINE_START(HREFV60, "ST-Ericsson U8500 Platform HREFv60+")

? ? ? .init_irq ? ? ? = ux500_init_irq,
? ? ? .timer ? ? ? ? ?= &ux500_timer,
? ? ? .init_machine ? = mop500_init_machine,
+ ? ? .dt_compat ? ? ?= hrefv60_dt_board_compat,
?MACHINE_END

?MACHINE_START(SNOWBALL, "Calao Systems Snowball platform")

@@ -668,4 +685,5 @@ MACHINE_START(SNOWBALL, "Calao Systems Snowball platform")

? ? ? /* we re-use nomadik timer here */
? ? ? .timer ? ? ? ? ?= &ux500_timer,
? ? ? .init_machine ? = mop500_init_machine,
+ ? ? .dt_compat ? ? ?= snowball_dt_board_compat,
?MACHINE_END

These all look very similar and should be combined. Why do you not have
1 machine for all DT platforms as other SOCs are doing?

This needs to be rebased for the latest kernel.
Which looks like this:

MACHINE_START(U8500, "ST-Ericsson MOP500 platform")
        /* Maintainer: Srinidhi Kasagar [off-list ref] */
        .atag_offset    = 0x100,
        .map_io         = u8500_map_io,
        .init_irq       = ux500_init_irq,
        /* we re-use nomadik timer here */
        .timer          = &ux500_timer,
        .init_machine   = mop500_init_machine,
MACHINE_END

MACHINE_START(HREFV60, "ST-Ericsson U8500 Platform HREFv60+")
        .atag_offset    = 0x100,
        .map_io         = u8500_map_io,
        .init_irq       = ux500_init_irq,
        .timer          = &ux500_timer,
        .init_machine   = hrefv60_init_machine,
MACHINE_END

MACHINE_START(SNOWBALL, "Calao Systems Snowball platform")
        .atag_offset    = 0x100,
        .map_io         = u8500_map_io,
        .init_irq       = ux500_init_irq,
        /* we re-use nomadik timer here */
        .timer          = &ux500_timer,
        .init_machine   = snowball_init_machine,
MACHINE_END

So - different .init_machine() calls, needed to simplify the board
files as of now. (Patch from Lee Jones.)

Yours,
Linus Walleij