On Thu, 2014-05-08 at 07:22 +0100, Marc Zyngier wrote: > Hi Ian, > > On 2014-05-07 20:24, Ian Campbell wrote: > > On Sat, 2014-02-15 at 13:36 +0000, Marc Zyngier wrote: > >> @@ -32,5 +33,11 @@ int arch_fixup_fdt(void *blob) > >> size[bank] = bd->bi_dram[bank].size; > >> } > >> > >> - return fdt_fixup_memory_banks(blob, start, size, > >> CONFIG_NR_DRAM_BANKS); > >> + ret = fdt_fixup_memory_banks(blob, start, size, > >> CONFIG_NR_DRAM_BANKS); > >> + if (ret) > >> + return ret; > >> + > >> +#if defined(CONFIG_ARMV7_NONSEC) || defined(CONFIG_ARMV7_VIRT) > >> + return armv7_update_dt(blob); > >> +#endif > > > > For platforms where neither of these CONFIG_* are set this results > > in: > > arch/arm/lib/bootm-fdt.c: In function ‘arch_fixup_fdt’: > > arch/arm/lib/bootm-fdt.c:43:1: warning: control reaches end of > > non-void function [-Wreturn-type] > > } > > I'm afraid this is old news. ;-) > > This bug has been fixed in v4: > http://lists.denx.de/pipermail/u-boot/2014-April/178280.html
AH, I thought I had the latest from your tree and I was only ccd on v3 so I missed it. Ian. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
