From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 5 Jun 2014 15:15:40 -0400 Subject: [U-Boot] [U-Boot, v5, 3/4] lib, fdt: move fdtdec_get_int() out of lib/fdtdec.c In-Reply-To: <1401269616-10303-4-git-send-email-hs@denx.de> References: <1401269616-10303-4-git-send-email-hs@denx.de> Message-ID: <20140605191540.GA9129@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, May 28, 2014 at 11:33:35AM +0200, Heiko Schocher wrote: > move fdtdec_get_int() out of lib/fdtdec.c into lib/fdtdec_common.c > as this function is also used, if CONFIG_OF_CONTROL is not > used. Poped up on the ids8313 board using signed FIT images, > and activating CONFIG_SYS_GENERIC_BOARD. Without this patch > it shows on boot: > > No valid FDT found - please append one to U-Boot binary, use u-boot-dtb.bin or define CONFIG_OF_EMBED. For sandbox, use -d > > With this patch, it boots again with CONFIG_SYS_GENERIC_BOARD > enabled. > > Signed-off-by: Heiko Schocher > Acked-by: Simon Glass > Cc: Tom Rini The problem is that on architectures with old compilers (sparc, blackfin, nds32) this doesn't get discarded due to not being used but instead causes link errors. Can you figure out which option (CONFIG_FIT_SIGNATURE I suspect) drives this need and make sure we include fdtdec_common.o then? Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: