From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Fri, 20 Jan 2012 13:54:06 -0500 Subject: [U-Boot] [PATCH v2 1/8] sandbox: fdt: Add support for CONFIG_OF_CONTROL In-Reply-To: <1326242752-9981-1-git-send-email-sjg@chromium.org> References: <1326242752-9981-1-git-send-email-sjg@chromium.org> Message-ID: <201201201354.08053.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. URL: