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

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to