On Tuesday 10 January 2012 19:45:45 Simon Glass wrote: > This adds support for a controlling fdt, mirroring the ARM implementation.
OK, but what's the point if we aren't doing FDT's ?
> +#ifdef CONFIG_OF_EMBED
> + /* Get a pointer to the FDT */
> + gd->fdt_blob = _binary_dt_dtb_start;
> +#elif defined CONFIG_OF_SEPARATE
> + /* FDT is at end of image */
> + gd->fdt_blob = (void *)(_end_ofs + _TEXT_BASE);
> +#endif
i'd be inclined to do:
+#if defined(CONFIG_OF_EMBED)
...
+#elif defined(CONFIG_OF_SEPARATE)
...
+#endif
-mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot

