SPL: tiny-printf: avoid any BSS usage
authorAndre Przywara <andre.przywara@arm.com>
Fri, 8 Jul 2016 14:18:35 +0000 (15:18 +0100)
committerTom Rini <trini@konsulko.com>
Fri, 8 Jul 2016 16:50:34 +0000 (12:50 -0400)
As printf calls may be executed quite early, we should avoid using any
BSS stored variables, since some boards put BSS in DRAM, which may not
have been initialised yet.
Explicitly mark those "static global" variables as belonging to the
.data section, to keep tiny-printf clear of any BSS usage.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
lib/tiny-printf.c

index 451f4f7a673d74e8ae93cee83f0c7c5a690a193e..b334f053cc0ee2645aa3669701f0ec22419bb572 100644 (file)
 #include <stdarg.h>
 #include <serial.h>
 
-static char *bf;
-static char zs;
+/*
+ * This code in here may execute before the DRAM is initialised, so
+ * we should make sure that it doesn't touch BSS, which some boards
+ * put in DRAM.
+ */
+static char *bf __attribute__ ((section(".data")));
+static char zs __attribute__ ((section(".data")));
 
 /* Current position in sprintf() output string */
-static char *outstr;
+static char *outstr __attribute__ ((section(".data")));
 
 static void out(char c)
 {