On Tuesday 02 September 2014 14:02:34 Linus Walleij wrote:
> +static void __init realview_dt_init_machine(void)
> +{
> + int ret;
> +
> +#if IS_ENABLED(CONFIG_CACHE_L2X0)
> + l2x0_of_init(0, ~0);
> +#endif
> +
> + ret = of_platform_populate(NULL, of_default_bus_match_table,
> + NULL, NULL);
> + if (ret) {
> + pr_crit("could not populate device tree\n");
> + return;
> + }
> +}
> +
> +static const char *realview_dt_platform_compat[] __initconst = {
> + "arm,realview-eb",
> + "arm,realview-pb1176",
> + "arm,realview-pb11mp",
> + "arm,realview-pba8",
> + "arm,realview-pbx",
> + NULL,
> +};
> +
> +DT_MACHINE_START(REALVIEW_DT, "ARM RealView Machine (Device Tree Support)")
> + .init_machine = realview_dt_init_machine,
> +#ifdef CONFIG_ZONE_DMA
> + .dma_zone_size = SZ_256M,
> +#endif
> + .dt_compat = realview_dt_platform_compat,
> +MACHINE_END
>
This looks almost perfect to me, but there is one simplification you
can do now:
If you add the .l2c_aux_val/.l2c_aux_mask fields to the
machine descriptor, you can remove the rest of the
realview_dt_init_machine function, and have only the
dma_zone_size setting left here.
On a related note, I think we need to add the same setting to the
default machine descriptor in setup_machine_fdt() so it gets set
up correctly there too.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
