8 Jan
2012
8 Jan
'12
6:43 p.m.
Hi Mike,
On Sun, Jan 8, 2012 at 12:36 AM, Mike Frysinger vapier@gentoo.org wrote:
On Saturday 10 December 2011 16:08:07 Simon Glass wrote:
--- a/arch/arm/lib/bootm.c +++ b/arch/arm/lib/bootm.c
static void announce_and_cleanup(void) { printf("\nStarting kernel ...\n\n");
- bootstage_mark_name(BOOTSTAGE_ID_BOOTM_HANDOFF, "start_kernel");
+#ifdef CONFIG_BOOTSTAGE_REPORT
- bootstage_report();
+#endif
shouldn't the header make bootstage_report() into a stub when CONFIG_BOOTSTAGE_REPORT isn't defined ? then you won't need the ifdef protection here.
Well maybe. I am hoping for a link error if someone calls it without defining that CONFIG. Otherwise it could be very confusing for people that they can't get a report!
Regards Simon
-mike