On Wed, Aug 24, 2011 at 03:09:08PM +0200, Benoit Cousson wrote: > Used the main OCP node to add bindings with the l3_noc driver. > Remove l3_noc static device creation if CONFIG_OF is defined. > > Signed-off-by: Benoit Cousson <b-cousson@xxxxxx> > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > Cc: Santosh Shilimkar <santosh.shilimkar@xxxxxx> > --- > arch/arm/boot/dts/omap4.dtsi | 3 ++- > arch/arm/mach-omap2/devices.c | 2 ++ > 2 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi > index 97a3ea7..928a74c 100644 > --- a/arch/arm/boot/dts/omap4.dtsi > +++ b/arch/arm/boot/dts/omap4.dtsi > @@ -64,10 +64,11 @@ > * hierarchy. > */ > ocp { > - compatible = "simple-bus"; > + compatible = "ti,l3-noc", "arteris,noc", "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > ranges; > + hwmods = "l3_main_1", "l3_main_2", "l3_main_3"; > > gic: interrupt-controller@48241000 { > compatible = "ti,omap4-gic", "arm,gic"; > diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c > index 2d4a199..5964650 100644 > --- a/arch/arm/mach-omap2/devices.c > +++ b/arch/arm/mach-omap2/devices.c > @@ -70,6 +70,7 @@ static int __init omap3_l3_init(void) > } > postcore_initcall(omap3_l3_init); > > +#ifndef CONFIG_OF > static int __init omap4_l3_init(void) > { > int l, i; > @@ -100,6 +101,7 @@ static int __init omap4_l3_init(void) > return IS_ERR(pdev) ? PTR_ERR(pdev) : 0; > } > postcore_initcall(omap4_l3_init); > +#endif Don't do this. Turning on CONFIG_OF must not break non-DT booting. Instead, check at runtime if a DT is available, and if it does then don't run the hook. g. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html