ARM: SPL: Clean up spl.c / spl_nand.c slightly
authorTom Rini <trini@ti.com>
Tue, 14 Aug 2012 19:06:43 +0000 (12:06 -0700)
committerTom Rini <trini@ti.com>
Thu, 27 Sep 2012 16:49:58 +0000 (09:49 -0700)
- Remove includes we don't need
- Switch some printf statements to puts
- Convert some printf statements to debug, introduce new puts statements
  - In most cases saying just "No mkimage signature, assuming
    u-boot.bin" or similar is sufficient.  This also means the non-DEBUG
    case doesn't need printf, in the core of SPL.
  - The other case here is that PLAIN_VERSION provided what we wanted
    already, so just use it.

Signed-off-by: Tom Rini <trini@ti.com>
arch/arm/cpu/armv7/omap-common/spl.c
arch/arm/cpu/armv7/omap-common/spl_nand.c

index 81991b2496ffddf7aa2255917a501a8fd1204d16..3a1029fa678a325fd2458d375f964671947a06d6 100644 (file)
 #include <asm/spl.h>
 #include <asm/u-boot.h>
 #include <asm/utils.h>
-#include <asm/arch/sys_proto.h>
 #include <nand.h>
-#include <mmc.h>
 #include <fat.h>
 #include <version.h>
-#include <asm/arch/mmc_host_def.h>
 #include <i2c.h>
 #include <image.h>
 #include <malloc.h>
@@ -78,8 +75,8 @@ void board_init_f(ulong dummy)
 #ifdef CONFIG_SPL_OS_BOOT
 __weak int spl_start_uboot(void)
 {
-       printf("SPL: Please implement spl_start_uboot() for your board\n");
-       printf("SPL: Direct Linux boot not active!\n");
+       puts("SPL: Please implement spl_start_uboot() for your board\n");
+       puts("SPL: Direct Linux boot not active!\n");
        return 1;
 }
 #endif
@@ -99,9 +96,9 @@ void spl_parse_image_header(const struct image_header *header)
                        spl_image.name, spl_image.load_addr, spl_image.size);
        } else {
                /* Signature not found - assume u-boot.bin */
-               printf("mkimage signature not found - ih_magic = %x\n",
+               puts("mkimage signature not found, assuming u-boot.bin ..\n");
+               debug("mkimage signature not found - ih_magic = %x\n",
                        header->ih_magic);
-               debug("Assuming u-boot.bin ..\n");
                /* Let's assume U-Boot will not be more than 200 KB */
                spl_image.size = 200 * 1024;
                spl_image.entry_point = CONFIG_SYS_TEXT_BASE;
@@ -179,7 +176,8 @@ void board_init_r(gd_t *id, ulong dummy)
                break;
 #endif
        default:
-               printf("SPL: Un-supported Boot Device - %d!!!\n", boot_device);
+               puts("SPL: Un-supported Boot Device\n");
+               debug("Found: %d\n", boot_device);
                hang();
                break;
        }
@@ -205,8 +203,6 @@ void board_init_r(gd_t *id, ulong dummy)
 /* This requires UART clocks to be enabled */
 void preloader_console_init(void)
 {
-       const char *u_boot_rev = U_BOOT_VERSION;
-
        gd = &gdata;
        gd->bd = &bdata;
        gd->flags |= GD_FLG_RELOC;
@@ -216,11 +212,8 @@ void preloader_console_init(void)
 
        gd->have_console = 1;
 
-       /* Avoid a second "U-Boot" coming from this string */
-       u_boot_rev = &u_boot_rev[7];
-
-       printf("\nU-Boot SPL %s (%s - %s)\n", u_boot_rev, U_BOOT_DATE,
-               U_BOOT_TIME);
+       puts("\nU-Boot SPL " PLAIN_VERSION " (" U_BOOT_DATE " - " \
+                       U_BOOT_TIME ")\n");
 #ifdef CONFIG_SPL_DISPLAY_PRINT
        spl_display_print();
 #endif
index 6552a5dfdec426da9e454bde39d4881b1fa2ec34..3e2f3590e69112484a41c235ecb946d2cda8eaa8 100644 (file)
@@ -24,7 +24,6 @@
 #include <asm/spl.h>
 #include <asm/u-boot.h>
 #include <asm/utils.h>
-#include <asm/arch/sys_proto.h>
 #include <asm/io.h>
 #include <nand.h>
 #include <version.h>