Hi Patrick, >Author: oxygene >Date: Tue Mar 1 08:30:14 2011 >New Revision: 6418 >URL: https://tracker.coreboot.org/trac/coreboot/changeset/6418 > >Log: >Mark non-returning function as noreturn to help some compiler versions > >Signed-off-by: Patrick Georgi <patrick.georgi at secunet.com> >Acked-by: Patrick Georgi <patrick.georgi at secunet.com> > >Modified: > trunk/src/northbridge/amd/lx/raminit.c > >Modified: trunk/src/northbridge/amd/lx/raminit.c >============================================================================== >--- trunk/src/northbridge/amd/lx/raminit.c Tue Mar 1 08:26:00 2011 > >(r6417) >+++ trunk/src/northbridge/amd/lx/raminit.c Tue Mar 1 08:30:14 2011 > >(r6418) >@@ -32,7 +32,7 @@ > printk(BIOS_DEBUG, " * %s\n", s); > } > >-static void hcf(void) >+static void __attribute__((noreturn)) hcf(void) > { > print_emerg("DIE\n"); > /* this guarantees we flush the UART fifos (if any) and also
Should this change also be applied for the almost same function in Geode GX2? Thanks,Nils. -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
