From patchwork Tue Oct 11 16:15:49 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Warren X-Patchwork-Id: 119064 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 97046B6F6B for ; Wed, 12 Oct 2011 06:31:54 +1100 (EST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 9E9D4282E3; Tue, 11 Oct 2011 21:31:51 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6Y33RbMZbUJo; Tue, 11 Oct 2011 21:31:51 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 00BC528305; Tue, 11 Oct 2011 21:31:42 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 5732B287CE for ; Tue, 11 Oct 2011 18:16:16 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Ms7KK04yvkmp for ; Tue, 11 Oct 2011 18:16:06 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from avon.wwwdotorg.org (avon.wwwdotorg.org [70.85.31.133]) by theia.denx.de (Postfix) with ESMTPS id 3E3562879B for ; Tue, 11 Oct 2011 18:16:02 +0200 (CEST) Received: from severn.wwwdotorg.org (unknown [192.168.65.5]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by avon.wwwdotorg.org (Postfix) with ESMTPS id 6E706631B; Tue, 11 Oct 2011 10:16:33 -0600 (MDT) Received: from localhost.localdomain (searspoint.nvidia.com [216.228.112.21]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by severn.wwwdotorg.org (Postfix) with ESMTPSA id D7EABE40DC; Tue, 11 Oct 2011 10:15:57 -0600 (MDT) From: Stephen Warren To: Wolfgang Denk Date: Tue, 11 Oct 2011 10:15:49 -0600 Message-Id: <1318349751-23798-1-git-send-email-swarren@nvidia.com> X-Mailer: git-send-email 1.7.0.4 X-NVConfidentiality: public X-Virus-Scanned: clamav-milter 0.96.5 at avon.wwwdotorg.org X-Virus-Status: Clean X-Mailman-Approved-At: Tue, 11 Oct 2011 21:31:39 +0200 Cc: U-Boot Mailing List , Tom Warren Subject: [U-Boot] [PATCH 1/3] [COSMETIC] checkpatch whitespace cleanups X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.9 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: u-boot-bounces@lists.denx.de Errors-To: u-boot-bounces@lists.denx.de This avoids the following checkpatch warning in later patches: ERROR: "(foo*)" should be "(foo *)" ERROR: space required before the open brace '{' ERROR: space prohibited before that close parenthesis ')' ERROR: spaces required around that '||' (ctx:WxV) WARNING: space prohibited between function name and open parenthesis '(' WARNING: line over 80 characters This fixes all the white-space warnings/errors in my subsequent patch, and within this current patch. A number of other checkpatch warnings and errors are still present in this patch itself, but are beyond simple whitespace fixes, so are not solved by this patch. v2: New patch Signed-off-by: Stephen Warren --- common/cmd_bootm.c | 547 ++++++++++++----------- common/cmd_ximg.c | 72 ++-- common/image.c | 1193 ++++++++++++++++++++++++++----------------------- common/lynxkdi.c | 34 +- include/image.h | 230 +++++----- tools/default_image.c | 56 ++-- 6 files changed, 1105 insertions(+), 1027 deletions(-) diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c index c2e8038..5a03b1c 100644 --- a/common/cmd_bootm.c +++ b/common/cmd_bootm.c @@ -72,27 +72,28 @@ extern void bz_internal_error(int); #endif #if defined(CONFIG_CMD_IMI) -static int image_info (unsigned long addr); +static int image_info(unsigned long addr); #endif #if defined(CONFIG_CMD_IMLS) #include #include extern flash_info_t flash_info[]; /* info for FLASH chips */ -static int do_imls (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); +static int do_imls(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); #endif #ifdef CONFIG_SILENT_CONSOLE -static void fixup_silent_linux (void); +static void fixup_silent_linux(void); #endif -static image_header_t *image_get_kernel (ulong img_addr, int verify); +static image_header_t *image_get_kernel(ulong img_addr, int verify); #if defined(CONFIG_FIT) -static int fit_check_kernel (const void *fit, int os_noffset, int verify); +static int fit_check_kernel(const void *fit, int os_noffset, int verify); #endif -static void *boot_get_kernel (cmd_tbl_t *cmdtp, int flag,int argc, char * const argv[], - bootm_headers_t *images, ulong *os_data, ulong *os_len); +static void *boot_get_kernel(cmd_tbl_t *cmdtp, int flag, int argc, + char * const argv[], bootm_headers_t *images, + ulong *os_data, ulong *os_len); /* * Continue booting an OS image; caller already has: @@ -102,7 +103,7 @@ static void *boot_get_kernel (cmd_tbl_t *cmdtp, int flag,int argc, char * const * - loaded (first part of) image to header load address, * - disabled interrupts. */ -typedef int boot_os_fn (int flag, int argc, char * const argv[], +typedef int boot_os_fn(int flag, int argc, char * const argv[], bootm_headers_t *images); /* pointers to os/initrd/fdt */ #ifdef CONFIG_BOOTM_LINUX @@ -113,7 +114,7 @@ static boot_os_fn do_bootm_netbsd; #endif #if defined(CONFIG_LYNXKDI) static boot_os_fn do_bootm_lynxkdi; -extern void lynxkdi_boot (image_header_t *); +extern void lynxkdi_boot(image_header_t *); #endif #ifdef CONFIG_BOOTM_RTEMS static boot_os_fn do_bootm_rtems; @@ -124,8 +125,8 @@ static boot_os_fn do_bootm_ose; #if defined(CONFIG_CMD_ELF) static boot_os_fn do_bootm_vxworks; static boot_os_fn do_bootm_qnxelf; -int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); -int do_bootelf (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); +int do_bootvx(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); +int do_bootelf(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); #endif #if defined(CONFIG_INTEGRITY) static boot_os_fn do_bootm_integrity; @@ -192,81 +193,81 @@ static int bootm_start(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[] void *os_hdr; int ret; - memset ((void *)&images, 0, sizeof (images)); - images.verify = getenv_yesno ("verify"); + memset((void *)&images, 0, sizeof(images)); + images.verify = getenv_yesno("verify"); bootm_start_lmb(); /* get kernel image header, start address and length */ - os_hdr = boot_get_kernel (cmdtp, flag, argc, argv, + os_hdr = boot_get_kernel(cmdtp, flag, argc, argv, &images, &images.os.image_start, &images.os.image_len); if (images.os.image_len == 0) { - puts ("ERROR: can't get kernel image!\n"); + puts("ERROR: can't get kernel image!\n"); return 1; } /* get image parameters */ - switch (genimg_get_format (os_hdr)) { + switch (genimg_get_format(os_hdr)) { case IMAGE_FORMAT_LEGACY: - images.os.type = image_get_type (os_hdr); - images.os.comp = image_get_comp (os_hdr); - images.os.os = image_get_os (os_hdr); + images.os.type = image_get_type(os_hdr); + images.os.comp = image_get_comp(os_hdr); + images.os.os = image_get_os(os_hdr); - images.os.end = image_get_image_end (os_hdr); - images.os.load = image_get_load (os_hdr); + images.os.end = image_get_image_end(os_hdr); + images.os.load = image_get_load(os_hdr); break; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: - if (fit_image_get_type (images.fit_hdr_os, + if (fit_image_get_type(images.fit_hdr_os, images.fit_noffset_os, &images.os.type)) { - puts ("Can't get image type!\n"); - show_boot_progress (-109); + puts("Can't get image type!\n"); + show_boot_progress(-109); return 1; } - if (fit_image_get_comp (images.fit_hdr_os, + if (fit_image_get_comp(images.fit_hdr_os, images.fit_noffset_os, &images.os.comp)) { - puts ("Can't get image compression!\n"); - show_boot_progress (-110); + puts("Can't get image compression!\n"); + show_boot_progress(-110); return 1; } - if (fit_image_get_os (images.fit_hdr_os, + if (fit_image_get_os(images.fit_hdr_os, images.fit_noffset_os, &images.os.os)) { - puts ("Can't get image OS!\n"); - show_boot_progress (-111); + puts("Can't get image OS!\n"); + show_boot_progress(-111); return 1; } - images.os.end = fit_get_end (images.fit_hdr_os); + images.os.end = fit_get_end(images.fit_hdr_os); - if (fit_image_get_load (images.fit_hdr_os, images.fit_noffset_os, + if (fit_image_get_load(images.fit_hdr_os, images.fit_noffset_os, &images.os.load)) { - puts ("Can't get image load address!\n"); - show_boot_progress (-112); + puts("Can't get image load address!\n"); + show_boot_progress(-112); return 1; } break; #endif default: - puts ("ERROR: unknown image format type!\n"); + puts("ERROR: unknown image format type!\n"); return 1; } /* find kernel entry point */ if (images.legacy_hdr_valid) { - images.ep = image_get_ep (&images.legacy_hdr_os_copy); + images.ep = image_get_ep(&images.legacy_hdr_os_copy); #if defined(CONFIG_FIT) } else if (images.fit_uname_os) { - ret = fit_image_get_entry (images.fit_hdr_os, + ret = fit_image_get_entry(images.fit_hdr_os, images.fit_noffset_os, &images.ep); if (ret) { - puts ("Can't get entry point property!\n"); + puts("Can't get entry point property!\n"); return 1; } #endif } else { - puts ("Could not find kernel entry point!\n"); + puts("Could not find kernel entry point!\n"); return 1; } @@ -274,19 +275,19 @@ static int bootm_start(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[] (images.os.type == IH_TYPE_MULTI)) && (images.os.os == IH_OS_LINUX)) { /* find ramdisk */ - ret = boot_get_ramdisk (argc, argv, &images, IH_INITRD_ARCH, + ret = boot_get_ramdisk(argc, argv, &images, IH_INITRD_ARCH, &images.rd_start, &images.rd_end); if (ret) { - puts ("Ramdisk image is corrupt or invalid\n"); + puts("Ramdisk image is corrupt or invalid\n"); return 1; } #if defined(CONFIG_OF_LIBFDT) /* find flattened device tree */ - ret = boot_get_fdt (flag, argc, argv, &images, - &images.ft_addr, &images.ft_len); + ret = boot_get_fdt(flag, argc, argv, &images, + &images.ft_addr, &images.ft_len); if (ret) { - puts ("Could not find a valid device tree\n"); + puts("Could not find a valid device tree\n"); return 1; } @@ -316,15 +317,15 @@ static int bootm_load_os(image_info_t os, ulong *load_end, int boot_progress) int ret; #endif /* defined(CONFIG_LZMA) || defined(CONFIG_LZO) */ - const char *type_name = genimg_get_type_name (os.type); + const char *type_name = genimg_get_type_name(os.type); switch (comp) { case IH_COMP_NONE: if (load == blob_start || load == image_start) { - printf (" XIP %s ... ", type_name); + printf(" XIP %s ... ", type_name); } else { - printf (" Loading %s ... ", type_name); - memmove_wd ((void *)load, (void *)image_start, + printf(" Loading %s ... ", type_name); + memmove_wd((void *)load, (void *)image_start, image_len, CHUNKSZ); } *load_end = load + image_len; @@ -332,13 +333,13 @@ static int bootm_load_os(image_info_t os, ulong *load_end, int boot_progress) break; #ifdef CONFIG_GZIP case IH_COMP_GZIP: - printf (" Uncompressing %s ... ", type_name); - if (gunzip ((void *)load, unc_len, - (uchar *)image_start, &image_len) != 0) { - puts ("GUNZIP: uncompress, out-of-mem or overwrite error " - "- must RESET board to recover\n"); + printf(" Uncompressing %s ... ", type_name); + if (gunzip((void *)load, unc_len, + (uchar *)image_start, &image_len) != 0) { + puts("GUNZIP: uncompress, out-of-mem or overwrite " + "error - must RESET board to recover\n"); if (boot_progress) - show_boot_progress (-6); + show_boot_progress(-6); return BOOTM_ERR_RESET; } @@ -347,20 +348,20 @@ static int bootm_load_os(image_info_t os, ulong *load_end, int boot_progress) #endif /* CONFIG_GZIP */ #ifdef CONFIG_BZIP2 case IH_COMP_BZIP2: - printf (" Uncompressing %s ... ", type_name); + printf(" Uncompressing %s ... ", type_name); /* * If we've got less than 4 MB of malloc() space, * use slower decompression algorithm which requires * at most 2300 KB of memory. */ - int i = BZ2_bzBuffToBuffDecompress ((char*)load, + int i = BZ2_bzBuffToBuffDecompress((char *)load, &unc_len, (char *)image_start, image_len, CONFIG_SYS_MALLOC_LEN < (4096 * 1024), 0); if (i != BZ_OK) { - printf ("BUNZIP2: uncompress or overwrite error %d " + printf("BUNZIP2: uncompress or overwrite error %d " "- must RESET board to recover\n", i); if (boot_progress) - show_boot_progress (-6); + show_boot_progress(-6); return BOOTM_ERR_RESET; } @@ -370,16 +371,16 @@ static int bootm_load_os(image_info_t os, ulong *load_end, int boot_progress) #ifdef CONFIG_LZMA case IH_COMP_LZMA: { SizeT lzma_len = unc_len; - printf (" Uncompressing %s ... ", type_name); + printf(" Uncompressing %s ... ", type_name); ret = lzmaBuffToBuffDecompress( (unsigned char *)load, &lzma_len, (unsigned char *)image_start, image_len); unc_len = lzma_len; if (ret != SZ_OK) { - printf ("LZMA: uncompress or overwrite error %d " + printf("LZMA: uncompress or overwrite error %d " "- must RESET board to recover\n", ret); - show_boot_progress (-6); + show_boot_progress(-6); return BOOTM_ERR_RESET; } *load_end = load + unc_len; @@ -388,16 +389,16 @@ static int bootm_load_os(image_info_t os, ulong *load_end, int boot_progress) #endif /* CONFIG_LZMA */ #ifdef CONFIG_LZO case IH_COMP_LZO: - printf (" Uncompressing %s ... ", type_name); + printf(" Uncompressing %s ... ", type_name); ret = lzop_decompress((const unsigned char *)image_start, image_len, (unsigned char *)load, &unc_len); if (ret != LZO_E_OK) { - printf ("LZO: uncompress or overwrite error %d " + printf("LZO: uncompress or overwrite error %d " "- must RESET board to recover\n", ret); if (boot_progress) - show_boot_progress (-6); + show_boot_progress(-6); return BOOTM_ERR_RESET; } @@ -405,20 +406,22 @@ static int bootm_load_os(image_info_t os, ulong *load_end, int boot_progress) break; #endif /* CONFIG_LZO */ default: - printf ("Unimplemented compression type %d\n", comp); + printf("Unimplemented compression type %d\n", comp); return BOOTM_ERR_UNIMPLEMENTED; } flush_cache(load, (*load_end - load) * sizeof(ulong)); - puts ("OK\n"); - debug (" kernel loaded at 0x%08lx, end = 0x%08lx\n", load, *load_end); + puts("OK\n"); + debug(" kernel loaded at 0x%08lx, end = 0x%08lx\n", load, *load_end); if (boot_progress) - show_boot_progress (7); + show_boot_progress(7); if ((load < blob_end) && (*load_end > blob_start)) { - debug ("images.os.start = 0x%lX, images.os.end = 0x%lx\n", blob_start, blob_end); - debug ("images.os.load = 0x%lx, load_end = 0x%lx\n", load, *load_end); + debug("images.os.start = 0x%lX, images.os.end = 0x%lx\n", + blob_start, blob_end); + debug("images.os.load = 0x%lx, load_end = 0x%lx\n", load, + *load_end); return BOOTM_ERR_OVERLAP; } @@ -461,7 +464,8 @@ static cmd_tbl_t cmd_bootm_sub[] = { U_BOOT_CMD_MKENT(go, 0, 1, (void *)BOOTM_STATE_OS_GO, "", ""), }; -int do_bootm_subcommand (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_bootm_subcommand(cmd_tbl_t *cmdtp, int flag, int argc, + char * const argv[]) { int ret = 0; int state; @@ -485,7 +489,7 @@ int do_bootm_subcommand (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv } if (images.state >= state) { - printf ("Trying to execute a command out of order\n"); + printf("Trying to execute a command out of order\n"); return cmd_usage(cmdtp); } @@ -536,17 +540,17 @@ int do_bootm_subcommand (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv case BOOTM_STATE_OS_CMDLINE: ret = boot_fn(BOOTM_STATE_OS_CMDLINE, argc, argv, &images); if (ret) - printf ("cmdline subcommand not supported\n"); + printf("cmdline subcommand not supported\n"); break; case BOOTM_STATE_OS_BD_T: ret = boot_fn(BOOTM_STATE_OS_BD_T, argc, argv, &images); if (ret) - printf ("bdt subcommand not supported\n"); + printf("bdt subcommand not supported\n"); break; case BOOTM_STATE_OS_PREP: ret = boot_fn(BOOTM_STATE_OS_PREP, argc, argv, &images); if (ret) - printf ("prep subcommand not supported\n"); + printf("prep subcommand not supported\n"); break; case BOOTM_STATE_OS_GO: disable_interrupts(); @@ -562,7 +566,7 @@ int do_bootm_subcommand (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv /* bootm - boot application image from image in memory */ /*******************************************************************/ -int do_bootm (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_bootm(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) { ulong iflag; ulong load_end = 0; @@ -625,23 +629,26 @@ int do_bootm (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) if (ret < 0) { if (ret == BOOTM_ERR_RESET) - do_reset (cmdtp, flag, argc, argv); + do_reset(cmdtp, flag, argc, argv); if (ret == BOOTM_ERR_OVERLAP) { if (images.legacy_hdr_valid) { - if (image_get_type (&images.legacy_hdr_os_copy) == IH_TYPE_MULTI) - puts ("WARNING: legacy format multi component " - "image overwritten\n"); + image_header_t *hdr; + hdr = &images.legacy_hdr_os_copy; + if (image_get_type(hdr) == IH_TYPE_MULTI) + puts("WARNING: legacy format multi " + "component image " + "overwritten\n"); } else { - puts ("ERROR: new format image overwritten - " + puts("ERROR: new format image overwritten - " "must RESET the board to recover\n"); - show_boot_progress (-113); - do_reset (cmdtp, flag, argc, argv); + show_boot_progress(-113); + do_reset(cmdtp, flag, argc, argv); } } if (ret == BOOTM_ERR_UNIMPLEMENTED) { if (iflag) enable_interrupts(); - show_boot_progress (-7); + show_boot_progress(-7); return 1; } } @@ -656,7 +663,7 @@ int do_bootm (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) return 0; } - show_boot_progress (8); + show_boot_progress(8); #ifdef CONFIG_SILENT_CONSOLE if (images.os.os == IH_OS_LINUX) @@ -668,9 +675,9 @@ int do_bootm (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) if (boot_fn == NULL) { if (iflag) enable_interrupts(); - printf ("ERROR: booting os '%s' (%d) is not supported\n", + printf("ERROR: booting os '%s' (%d) is not supported\n", genimg_get_os_name(images.os.os), images.os.os); - show_boot_progress (-8); + show_boot_progress(-8); return 1; } @@ -678,11 +685,11 @@ int do_bootm (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) boot_fn(0, argc, argv, &images); - show_boot_progress (-9); + show_boot_progress(-9); #ifdef DEBUG - puts ("\n## Control returned to monitor - resetting...\n"); + puts("\n## Control returned to monitor - resetting...\n"); #endif - do_reset (cmdtp, flag, argc, argv); + do_reset(cmdtp, flag, argc, argv); return 1; } @@ -714,40 +721,40 @@ int bootm_maybe_autostart(cmd_tbl_t *cmdtp, const char *cmd) * pointer to a legacy image header if valid image was found * otherwise return NULL */ -static image_header_t *image_get_kernel (ulong img_addr, int verify) +static image_header_t *image_get_kernel(ulong img_addr, int verify) { image_header_t *hdr = (image_header_t *)img_addr; if (!image_check_magic(hdr)) { - puts ("Bad Magic Number\n"); - show_boot_progress (-1); + puts("Bad Magic Number\n"); + show_boot_progress(-1); return NULL; } - show_boot_progress (2); + show_boot_progress(2); - if (!image_check_hcrc (hdr)) { - puts ("Bad Header Checksum\n"); - show_boot_progress (-2); + if (!image_check_hcrc(hdr)) { + puts("Bad Header Checksum\n"); + show_boot_progress(-2); return NULL; } - show_boot_progress (3); - image_print_contents (hdr); + show_boot_progress(3); + image_print_contents(hdr); if (verify) { - puts (" Verifying Checksum ... "); - if (!image_check_dcrc (hdr)) { - printf ("Bad Data CRC\n"); - show_boot_progress (-3); + puts(" Verifying Checksum ... "); + if (!image_check_dcrc(hdr)) { + printf("Bad Data CRC\n"); + show_boot_progress(-3); return NULL; } - puts ("OK\n"); + puts("OK\n"); } - show_boot_progress (4); + show_boot_progress(4); - if (!image_check_target_arch (hdr)) { - printf ("Unsupported Architecture 0x%x\n", image_get_arch (hdr)); - show_boot_progress (-4); + if (!image_check_target_arch(hdr)) { + printf("Unsupported Architecture 0x%x\n", image_get_arch(hdr)); + show_boot_progress(-4); return NULL; } return hdr; @@ -766,36 +773,36 @@ static image_header_t *image_get_kernel (ulong img_addr, int verify) * 1, on success * 0, on failure */ -#if defined (CONFIG_FIT) -static int fit_check_kernel (const void *fit, int os_noffset, int verify) +#if defined(CONFIG_FIT) +static int fit_check_kernel(const void *fit, int os_noffset, int verify) { - fit_image_print (fit, os_noffset, " "); + fit_image_print(fit, os_noffset, " "); if (verify) { - puts (" Verifying Hash Integrity ... "); - if (!fit_image_check_hashes (fit, os_noffset)) { - puts ("Bad Data Hash\n"); - show_boot_progress (-104); + puts(" Verifying Hash Integrity ... "); + if (!fit_image_check_hashes(fit, os_noffset)) { + puts("Bad Data Hash\n"); + show_boot_progress(-104); return 0; } - puts ("OK\n"); + puts("OK\n"); } - show_boot_progress (105); + show_boot_progress(105); - if (!fit_image_check_target_arch (fit, os_noffset)) { - puts ("Unsupported Architecture\n"); - show_boot_progress (-105); + if (!fit_image_check_target_arch(fit, os_noffset)) { + puts("Unsupported Architecture\n"); + show_boot_progress(-105); return 0; } - show_boot_progress (106); - if (!fit_image_check_type (fit, os_noffset, IH_TYPE_KERNEL)) { - puts ("Not a kernel image\n"); - show_boot_progress (-106); + show_boot_progress(106); + if (!fit_image_check_type(fit, os_noffset, IH_TYPE_KERNEL)) { + puts("Not a kernel image\n"); + show_boot_progress(-106); return 0; } - show_boot_progress (107); + show_boot_progress(107); return 1; } #endif /* CONFIG_FIT */ @@ -812,8 +819,9 @@ static int fit_check_kernel (const void *fit, int os_noffset, int verify) * pointer to image header if valid image was found, plus kernel start * address and length, otherwise NULL */ -static void *boot_get_kernel (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[], - bootm_headers_t *images, ulong *os_data, ulong *os_len) +static void *boot_get_kernel(cmd_tbl_t *cmdtp, int flag, int argc, + char * const argv[], bootm_headers_t *images, ulong *os_data, + ulong *os_len) { image_header_t *hdr; ulong img_addr; @@ -830,127 +838,137 @@ static void *boot_get_kernel (cmd_tbl_t *cmdtp, int flag, int argc, char * const /* find out kernel image address */ if (argc < 2) { img_addr = load_addr; - debug ("* kernel: default image load address = 0x%08lx\n", + debug("* kernel: default image load address = 0x%08lx\n", load_addr); #if defined(CONFIG_FIT) - } else if (fit_parse_conf (argv[1], load_addr, &img_addr, + } else if (fit_parse_conf(argv[1], load_addr, &img_addr, &fit_uname_config)) { - debug ("* kernel: config '%s' from image at 0x%08lx\n", + debug("* kernel: config '%s' from image at 0x%08lx\n", fit_uname_config, img_addr); - } else if (fit_parse_subimage (argv[1], load_addr, &img_addr, + } else if (fit_parse_subimage(argv[1], load_addr, &img_addr, &fit_uname_kernel)) { - debug ("* kernel: subimage '%s' from image at 0x%08lx\n", + debug("* kernel: subimage '%s' from image at 0x%08lx\n", fit_uname_kernel, img_addr); #endif } else { img_addr = simple_strtoul(argv[1], NULL, 16); - debug ("* kernel: cmdline image address = 0x%08lx\n", img_addr); + debug("* kernel: cmdline image address = 0x%08lx\n", img_addr); } - show_boot_progress (1); + show_boot_progress(1); /* copy from dataflash if needed */ - img_addr = genimg_get_image (img_addr); + img_addr = genimg_get_image(img_addr); /* check image type, for FIT images get FIT kernel node */ *os_data = *os_len = 0; - switch (genimg_get_format ((void *)img_addr)) { + switch (genimg_get_format((void *)img_addr)) { case IMAGE_FORMAT_LEGACY: - printf ("## Booting kernel from Legacy Image at %08lx ...\n", + printf("## Booting kernel from Legacy Image at %08lx ...\n", img_addr); - hdr = image_get_kernel (img_addr, images->verify); + hdr = image_get_kernel(img_addr, images->verify); if (!hdr) return NULL; - show_boot_progress (5); + show_boot_progress(5); /* get os_data and os_len */ - switch (image_get_type (hdr)) { + switch (image_get_type(hdr)) { case IH_TYPE_KERNEL: - *os_data = image_get_data (hdr); - *os_len = image_get_data_size (hdr); + *os_data = image_get_data(hdr); + *os_len = image_get_data_size(hdr); break; case IH_TYPE_MULTI: - image_multi_getimg (hdr, 0, os_data, os_len); + image_multi_getimg(hdr, 0, os_data, os_len); break; case IH_TYPE_STANDALONE: - *os_data = image_get_data (hdr); - *os_len = image_get_data_size (hdr); + *os_data = image_get_data(hdr); + *os_len = image_get_data_size(hdr); break; default: - printf ("Wrong Image Type for %s command\n", cmdtp->name); - show_boot_progress (-5); + printf("Wrong Image Type for %s command\n", + cmdtp->name); + show_boot_progress(-5); return NULL; } /* - * copy image header to allow for image overwrites during kernel - * decompression. + * copy image header to allow for image overwrites during + * kernel decompression. */ - memmove (&images->legacy_hdr_os_copy, hdr, sizeof(image_header_t)); + memmove(&images->legacy_hdr_os_copy, hdr, + sizeof(image_header_t)); /* save pointer to image header */ images->legacy_hdr_os = hdr; images->legacy_hdr_valid = 1; - show_boot_progress (6); + show_boot_progress(6); break; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: fit_hdr = (void *)img_addr; - printf ("## Booting kernel from FIT Image at %08lx ...\n", + printf("## Booting kernel from FIT Image at %08lx ...\n", img_addr); - if (!fit_check_format (fit_hdr)) { - puts ("Bad FIT kernel image format!\n"); - show_boot_progress (-100); + if (!fit_check_format(fit_hdr)) { + puts("Bad FIT kernel image format!\n"); + show_boot_progress(-100); return NULL; } - show_boot_progress (100); + show_boot_progress(100); if (!fit_uname_kernel) { /* * no kernel image node unit name, try to get config * node first. If config unit node name is NULL - * fit_conf_get_node() will try to find default config node + * fit_conf_get_node() will try to find default config + * node */ - show_boot_progress (101); - cfg_noffset = fit_conf_get_node (fit_hdr, fit_uname_config); + show_boot_progress(101); + cfg_noffset = fit_conf_get_node(fit_hdr, + fit_uname_config); if (cfg_noffset < 0) { - show_boot_progress (-101); + show_boot_progress(-101); return NULL; } /* save configuration uname provided in the first * bootm argument */ - images->fit_uname_cfg = fdt_get_name (fit_hdr, cfg_noffset, NULL); - printf (" Using '%s' configuration\n", images->fit_uname_cfg); - show_boot_progress (103); - - os_noffset = fit_conf_get_kernel_node (fit_hdr, cfg_noffset); - fit_uname_kernel = fit_get_name (fit_hdr, os_noffset, NULL); + images->fit_uname_cfg = fdt_get_name(fit_hdr, + cfg_noffset, + NULL); + printf(" Using '%s' configuration\n", + images->fit_uname_cfg); + show_boot_progress(103); + + os_noffset = fit_conf_get_kernel_node(fit_hdr, + cfg_noffset); + fit_uname_kernel = fit_get_name(fit_hdr, os_noffset, + NULL); } else { /* get kernel component image node offset */ - show_boot_progress (102); - os_noffset = fit_image_get_node (fit_hdr, fit_uname_kernel); + show_boot_progress(102); + os_noffset = fit_image_get_node(fit_hdr, + fit_uname_kernel); } if (os_noffset < 0) { - show_boot_progress (-103); + show_boot_progress(-103); return NULL; } - printf (" Trying '%s' kernel subimage\n", fit_uname_kernel); + printf(" Trying '%s' kernel subimage\n", fit_uname_kernel); - show_boot_progress (104); - if (!fit_check_kernel (fit_hdr, os_noffset, images->verify)) + show_boot_progress(104); + if (!fit_check_kernel(fit_hdr, os_noffset, images->verify)) return NULL; /* get kernel image data address and length */ - if (fit_image_get_data (fit_hdr, os_noffset, &data, &len)) { - puts ("Could not find kernel subimage data!\n"); - show_boot_progress (-107); + if (fit_image_get_data(fit_hdr, os_noffset, &data, &len)) { + puts("Could not find kernel subimage data!\n"); + show_boot_progress(-107); return NULL; } - show_boot_progress (108); + show_boot_progress(108); *os_len = len; *os_data = (ulong)data; @@ -960,12 +978,12 @@ static void *boot_get_kernel (cmd_tbl_t *cmdtp, int flag, int argc, char * const break; #endif default: - printf ("Wrong Image Format for %s command\n", cmdtp->name); - show_boot_progress (-108); + printf("Wrong Image Format for %s command\n", cmdtp->name); + show_boot_progress(-108); return NULL; } - debug (" kernel data at 0x%08lx, len = 0x%08lx (%ld)\n", + debug(" kernel data at 0x%08lx, len = 0x%08lx (%ld)\n", *os_data, *os_len, *os_len); return (void *)img_addr; @@ -1013,15 +1031,15 @@ U_BOOT_CMD( /* bootd - boot default image */ /*******************************************************************/ #if defined(CONFIG_CMD_BOOTD) -int do_bootd (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_bootd(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) { int rcode = 0; #ifndef CONFIG_SYS_HUSH_PARSER - if (run_command (getenv ("bootcmd"), flag) < 0) + if (run_command(getenv("bootcmd"), flag) < 0) rcode = 1; #else - if (parse_string_outer (getenv ("bootcmd"), + if (parse_string_outer(getenv("bootcmd"), FLAG_PARSE_SEMICOLON | FLAG_EXIT_FROM_LOOP) != 0) rcode = 1; #endif @@ -1048,72 +1066,72 @@ U_BOOT_CMD( /* iminfo - print header info for a requested image */ /*******************************************************************/ #if defined(CONFIG_CMD_IMI) -int do_iminfo (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_iminfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) { int arg; ulong addr; int rcode = 0; if (argc < 2) { - return image_info (load_addr); + return image_info(load_addr); } for (arg = 1; arg < argc; ++arg) { - addr = simple_strtoul (argv[arg], NULL, 16); - if (image_info (addr) != 0) + addr = simple_strtoul(argv[arg], NULL, 16); + if (image_info(addr) != 0) rcode = 1; } return rcode; } -static int image_info (ulong addr) +static int image_info(ulong addr) { void *hdr = (void *)addr; - printf ("\n## Checking Image at %08lx ...\n", addr); + printf("\n## Checking Image at %08lx ...\n", addr); - switch (genimg_get_format (hdr)) { + switch (genimg_get_format(hdr)) { case IMAGE_FORMAT_LEGACY: - puts (" Legacy image found\n"); - if (!image_check_magic (hdr)) { - puts (" Bad Magic Number\n"); + puts(" Legacy image found\n"); + if (!image_check_magic(hdr)) { + puts(" Bad Magic Number\n"); return 1; } - if (!image_check_hcrc (hdr)) { - puts (" Bad Header Checksum\n"); + if (!image_check_hcrc(hdr)) { + puts(" Bad Header Checksum\n"); return 1; } - image_print_contents (hdr); + image_print_contents(hdr); - puts (" Verifying Checksum ... "); - if (!image_check_dcrc (hdr)) { - puts (" Bad Data CRC\n"); + puts(" Verifying Checksum ... "); + if (!image_check_dcrc(hdr)) { + puts(" Bad Data CRC\n"); return 1; } - puts ("OK\n"); + puts("OK\n"); return 0; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: - puts (" FIT image found\n"); + puts(" FIT image found\n"); - if (!fit_check_format (hdr)) { - puts ("Bad FIT image format!\n"); + if (!fit_check_format(hdr)) { + puts("Bad FIT image format!\n"); return 1; } - fit_print_contents (hdr); + fit_print_contents(hdr); - if (!fit_all_image_check_hashes (hdr)) { - puts ("Bad hash in FIT image!\n"); + if (!fit_all_image_check_hashes(hdr)) { + puts("Bad hash in FIT image!\n"); return 1; } return 0; #endif default: - puts ("Unknown image format!\n"); + puts("Unknown image format!\n"); break; } @@ -1135,7 +1153,7 @@ U_BOOT_CMD( /* imls - list all images found in flash */ /*******************************************************************/ #if defined(CONFIG_CMD_IMLS) -int do_imls (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_imls(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) { flash_info_t *info; int i, j; @@ -1152,28 +1170,28 @@ int do_imls (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) if (!hdr) goto next_sector; - switch (genimg_get_format (hdr)) { + switch (genimg_get_format(hdr)) { case IMAGE_FORMAT_LEGACY: - if (!image_check_hcrc (hdr)) + if (!image_check_hcrc(hdr)) goto next_sector; - printf ("Legacy Image at %08lX:\n", (ulong)hdr); - image_print_contents (hdr); + printf("Legacy Image at %08lX:\n", (ulong)hdr); + image_print_contents(hdr); - puts (" Verifying Checksum ... "); - if (!image_check_dcrc (hdr)) { - puts ("Bad Data CRC\n"); + puts(" Verifying Checksum ... "); + if (!image_check_dcrc(hdr)) { + puts("Bad Data CRC\n"); } else { - puts ("OK\n"); + puts("OK\n"); } break; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: - if (!fit_check_format (hdr)) + if (!fit_check_format(hdr)) goto next_sector; - printf ("FIT Image at %08lX:\n", (ulong)hdr); - fit_print_contents (hdr); + printf("FIT Image at %08lX:\n", (ulong)hdr); + fit_print_contents(hdr); break; #endif default: @@ -1201,34 +1219,34 @@ U_BOOT_CMD( /* helper routines */ /*******************************************************************/ #ifdef CONFIG_SILENT_CONSOLE -static void fixup_silent_linux () +static void fixup_silent_linux() { char buf[256], *start, *end; - char *cmdline = getenv ("bootargs"); + char *cmdline = getenv("bootargs"); /* Only fix cmdline when requested */ if (!(gd->flags & GD_FLG_SILENT)) return; - debug ("before silent fix-up: %s\n", cmdline); + debug("before silent fix-up: %s\n", cmdline); if (cmdline) { - if ((start = strstr (cmdline, "console=")) != NULL) { - end = strchr (start, ' '); - strncpy (buf, cmdline, (start - cmdline + 8)); + if ((start = strstr(cmdline, "console=")) != NULL) { + end = strchr(start, ' '); + strncpy(buf, cmdline, (start - cmdline + 8)); if (end) - strcpy (buf + (start - cmdline + 8), end); + strcpy(buf + (start - cmdline + 8), end); else buf[start - cmdline + 8] = '\0'; } else { - strcpy (buf, cmdline); - strcat (buf, " console="); + strcpy(buf, cmdline); + strcat(buf, " console="); } } else { - strcpy (buf, "console="); + strcpy(buf, "console="); } - setenv ("bootargs", buf); - debug ("after silent fix-up: %s\n", buf); + setenv("bootargs", buf); + debug("after silent fix-up: %s\n", buf); } #endif /* CONFIG_SILENT_CONSOLE */ @@ -1238,7 +1256,7 @@ static void fixup_silent_linux () /*******************************************************************/ #ifdef CONFIG_BOOTM_NETBSD -static int do_bootm_netbsd (int flag, int argc, char * const argv[], +static int do_bootm_netbsd(int flag, int argc, char * const argv[], bootm_headers_t *images) { void (*loader)(bd_t *, image_header_t *, char *, char *); @@ -1252,7 +1270,7 @@ static int do_bootm_netbsd (int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("NetBSD"); + fit_unsupported_reset("NetBSD"); return 1; } #endif @@ -1270,20 +1288,20 @@ static int do_bootm_netbsd (int flag, int argc, char * const argv[], * address of the original image header. */ os_hdr = NULL; - if (image_check_type (&images->legacy_hdr_os_copy, IH_TYPE_MULTI)) { - image_multi_getimg (hdr, 1, &kernel_data, &kernel_len); + if (image_check_type(&images->legacy_hdr_os_copy, IH_TYPE_MULTI)) { + image_multi_getimg(hdr, 1, &kernel_data, &kernel_len); if (kernel_len) os_hdr = hdr; } consdev = ""; -#if defined (CONFIG_8xx_CONS_SMC1) +#if defined(CONFIG_8xx_CONS_SMC1) consdev = "smc1"; -#elif defined (CONFIG_8xx_CONS_SMC2) +#elif defined(CONFIG_8xx_CONS_SMC2) consdev = "smc2"; -#elif defined (CONFIG_8xx_CONS_SCC2) +#elif defined(CONFIG_8xx_CONS_SCC2) consdev = "scc2"; -#elif defined (CONFIG_8xx_CONS_SCC3) +#elif defined(CONFIG_8xx_CONS_SCC3) consdev = "scc3"; #endif @@ -1292,25 +1310,26 @@ static int do_bootm_netbsd (int flag, int argc, char * const argv[], int i; for (i = 2, len = 0; i < argc; i += 1) - len += strlen (argv[i]) + 1; - cmdline = malloc (len); + len += strlen(argv[i]) + 1; + cmdline = malloc(len); for (i = 2, len = 0; i < argc; i += 1) { if (i > 2) cmdline[len++] = ' '; - strcpy (&cmdline[len], argv[i]); - len += strlen (argv[i]); + strcpy(&cmdline[len], argv[i]); + len += strlen(argv[i]); } - } else if ((cmdline = getenv ("bootargs")) == NULL) { + } else if ((cmdline = getenv("bootargs")) == NULL) { cmdline = ""; } loader = (void (*)(bd_t *, image_header_t *, char *, char *))images->ep; - printf ("## Transferring control to NetBSD stage-2 loader (at address %08lx) ...\n", + printf("## Transferring control to NetBSD stage-2 loader " + "(at address %08lx) ...\n", (ulong)loader); - show_boot_progress (15); + show_boot_progress(15); /* * NetBSD Stage-2 Loader Parameters: @@ -1319,14 +1338,14 @@ static int do_bootm_netbsd (int flag, int argc, char * const argv[], * r5: console device * r6: boot args string */ - (*loader) (gd->bd, os_hdr, consdev, cmdline); + (*loader)(gd->bd, os_hdr, consdev, cmdline); return 1; } #endif /* CONFIG_BOOTM_NETBSD*/ #ifdef CONFIG_LYNXKDI -static int do_bootm_lynxkdi (int flag, int argc, char * const argv[], +static int do_bootm_lynxkdi(int flag, int argc, char * const argv[], bootm_headers_t *images) { image_header_t *hdr = &images->legacy_hdr_os_copy; @@ -1336,19 +1355,19 @@ static int do_bootm_lynxkdi (int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("Lynx"); + fit_unsupported_reset("Lynx"); return 1; } #endif - lynxkdi_boot ((image_header_t *)hdr); + lynxkdi_boot((image_header_t *)hdr); return 1; } #endif /* CONFIG_LYNXKDI */ #ifdef CONFIG_BOOTM_RTEMS -static int do_bootm_rtems (int flag, int argc, char * const argv[], +static int do_bootm_rtems(int flag, int argc, char * const argv[], bootm_headers_t *images) { void (*entry_point)(bd_t *); @@ -1358,17 +1377,17 @@ static int do_bootm_rtems (int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("RTEMS"); + fit_unsupported_reset("RTEMS"); return 1; } #endif entry_point = (void (*)(bd_t *))images->ep; - printf ("## Transferring control to RTEMS (at address %08lx) ...\n", + printf("## Transferring control to RTEMS (at address %08lx) ...\n", (ulong)entry_point); - show_boot_progress (15); + show_boot_progress(15); /* * RTEMS Parameters: @@ -1381,7 +1400,7 @@ static int do_bootm_rtems (int flag, int argc, char * const argv[], #endif /* CONFIG_BOOTM_RTEMS */ #if defined(CONFIG_BOOTM_OSE) -static int do_bootm_ose (int flag, int argc, char * const argv[], +static int do_bootm_ose(int flag, int argc, char * const argv[], bootm_headers_t *images) { void (*entry_point)(void); @@ -1391,17 +1410,17 @@ static int do_bootm_ose (int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("OSE"); + fit_unsupported_reset("OSE"); return 1; } #endif entry_point = (void (*)(void))images->ep; - printf ("## Transferring control to OSE (at address %08lx) ...\n", + printf("## Transferring control to OSE (at address %08lx) ...\n", (ulong)entry_point); - show_boot_progress (15); + show_boot_progress(15); /* * OSE Parameters: @@ -1414,7 +1433,7 @@ static int do_bootm_ose (int flag, int argc, char * const argv[], #endif /* CONFIG_BOOTM_OSE */ #if defined(CONFIG_CMD_ELF) -static int do_bootm_vxworks (int flag, int argc, char * const argv[], +static int do_bootm_vxworks(int flag, int argc, char * const argv[], bootm_headers_t *images) { char str[80]; @@ -1424,7 +1443,7 @@ static int do_bootm_vxworks (int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("VxWorks"); + fit_unsupported_reset("VxWorks"); return 1; } #endif @@ -1447,7 +1466,7 @@ static int do_bootm_qnxelf(int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("QNX"); + fit_unsupported_reset("QNX"); return 1; } #endif @@ -1462,7 +1481,7 @@ static int do_bootm_qnxelf(int flag, int argc, char * const argv[], #endif #ifdef CONFIG_INTEGRITY -static int do_bootm_integrity (int flag, int argc, char * const argv[], +static int do_bootm_integrity(int flag, int argc, char * const argv[], bootm_headers_t *images) { void (*entry_point)(void); @@ -1472,17 +1491,17 @@ static int do_bootm_integrity (int flag, int argc, char * const argv[], #if defined(CONFIG_FIT) if (!images->legacy_hdr_valid) { - fit_unsupported_reset ("INTEGRITY"); + fit_unsupported_reset("INTEGRITY"); return 1; } #endif entry_point = (void (*)(void))images->ep; - printf ("## Transferring control to INTEGRITY (at address %08lx) ...\n", + printf("## Transferring control to INTEGRITY (at address %08lx) ...\n", (ulong)entry_point); - show_boot_progress (15); + show_boot_progress(15); /* * INTEGRITY Parameters: diff --git a/common/cmd_ximg.c b/common/cmd_ximg.c index 850188e..0414589 100644 --- a/common/cmd_ximg.c +++ b/common/cmd_ximg.c @@ -62,7 +62,7 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) uint unc_len = CONFIG_SYS_XIMG_LEN; uint8_t comp; - verify = getenv_yesno ("verify"); + verify = getenv_yesno("verify"); if (argc > 1) { addr = simple_strtoul(argv[1], NULL, 16); @@ -77,33 +77,33 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) dest = simple_strtoul(argv[3], NULL, 16); } - switch (genimg_get_format ((void *)addr)) { + switch (genimg_get_format((void *)addr)) { case IMAGE_FORMAT_LEGACY: printf("## Copying part %d from legacy image " "at %08lx ...\n", part, addr); hdr = (image_header_t *)addr; - if (!image_check_magic (hdr)) { + if (!image_check_magic(hdr)) { printf("Bad Magic Number\n"); return 1; } - if (!image_check_hcrc (hdr)) { + if (!image_check_hcrc(hdr)) { printf("Bad Header Checksum\n"); return 1; } #ifdef DEBUG - image_print_contents (hdr); + image_print_contents(hdr); #endif - if (!image_check_type (hdr, IH_TYPE_MULTI)) { + if (!image_check_type(hdr, IH_TYPE_MULTI)) { printf("Wrong Image Type for %s command\n", cmdtp->name); return 1; } - comp = image_get_comp (hdr); + comp = image_get_comp(hdr); if ((comp != IH_COMP_NONE) && (argc < 4)) { printf("Must specify load address for %s command " "with compressed image\n", @@ -113,25 +113,25 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) if (verify) { printf(" Verifying Checksum ... "); - if (!image_check_dcrc (hdr)) { + if (!image_check_dcrc(hdr)) { printf("Bad Data CRC\n"); return 1; } printf("OK\n"); } - count = image_multi_count (hdr); + count = image_multi_count(hdr); if (part >= count) { printf("Bad Image Part\n"); return 1; } - image_multi_getimg (hdr, part, &data, &len); + image_multi_getimg(hdr, part, &data, &len); break; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: if (uname == NULL) { - puts ("No FIT subimage unit name\n"); + puts("No FIT subimage unit name\n"); return 1; } @@ -139,19 +139,19 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) "at %08lx ...\n", uname, addr); fit_hdr = (const void *)addr; - if (!fit_check_format (fit_hdr)) { - puts ("Bad FIT image format\n"); + if (!fit_check_format(fit_hdr)) { + puts("Bad FIT image format\n"); return 1; } /* get subimage node offset */ - noffset = fit_image_get_node (fit_hdr, uname); + noffset = fit_image_get_node(fit_hdr, uname); if (noffset < 0) { - printf ("Can't find '%s' FIT subimage\n", uname); + printf("Can't find '%s' FIT subimage\n", uname); return 1; } - if (fit_image_check_comp (fit_hdr, noffset, IH_COMP_NONE) + if (fit_image_check_comp(fit_hdr, noffset, IH_COMP_NONE) && (argc < 4)) { printf("Must specify load address for %s command " "with compressed image\n", @@ -161,21 +161,21 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) /* verify integrity */ if (verify) { - if (!fit_image_check_hashes (fit_hdr, noffset)) { - puts ("Bad Data Hash\n"); + if (!fit_image_check_hashes(fit_hdr, noffset)) { + puts("Bad Data Hash\n"); return 1; } } /* get subimage data address and length */ - if (fit_image_get_data (fit_hdr, noffset, + if (fit_image_get_data(fit_hdr, noffset, &fit_data, &fit_len)) { - puts ("Could not find script subimage data\n"); + puts("Could not find script subimage data\n"); return 1; } - if (fit_image_get_comp (fit_hdr, noffset, &comp)) { - puts ("Could not find script subimage " + if (fit_image_get_comp(fit_hdr, noffset, &comp)) { + puts("Could not find script subimage " "compression type\n"); return 1; } @@ -185,7 +185,7 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) break; #endif default: - puts ("Invalid image type for imxtract\n"); + puts("Invalid image type for imxtract\n"); return 1; } @@ -199,28 +199,28 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) void *to = (void *) dest; void *from = (void *)data; - printf (" Loading part %d ... ", part); + printf(" Loading part %d ... ", part); while (l > 0) { tail = (l > CHUNKSZ) ? CHUNKSZ : l; WATCHDOG_RESET(); - memmove (to, from, tail); + memmove(to, from, tail); to += tail; from += tail; l -= tail; } } #else /* !(CONFIG_HW_WATCHDOG || CONFIG_WATCHDOG) */ - printf (" Loading part %d ... ", part); - memmove ((char *) dest, (char *)data, len); + printf(" Loading part %d ... ", part); + memmove((char *) dest, (char *)data, len); #endif /* CONFIG_HW_WATCHDOG || CONFIG_WATCHDOG */ break; #ifdef CONFIG_GZIP case IH_COMP_GZIP: - printf (" Uncompressing part %d ... ", part); - if (gunzip ((void *) dest, unc_len, - (uchar *) data, &len) != 0) { - puts ("GUNZIP ERROR - image not loaded\n"); + printf(" Uncompressing part %d ... ", part); + if (gunzip((void *) dest, unc_len, + (uchar *) data, &len) != 0) { + puts("GUNZIP ERROR - image not loaded\n"); return 1; } break; @@ -230,19 +230,19 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) { int i; - printf (" Uncompressing part %d ... ", part); + printf(" Uncompressing part %d ... ", part); /* * If we've got less than 4 MB of malloc() * space, use slower decompression algorithm * which requires at most 2300 KB of memory. */ i = BZ2_bzBuffToBuffDecompress( - (char*)ntohl(hdr->ih_load), + (char *)ntohl(hdr->ih_load), &unc_len, (char *)data, len, CONFIG_SYS_MALLOC_LEN < (4096 * 1024), 0); if (i != BZ_OK) { - printf ("BUNZIP2 ERROR %d - " + printf("BUNZIP2 ERROR %d - " "image not loaded\n", i); return 1; } @@ -250,10 +250,10 @@ do_imgextract(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) break; #endif /* CONFIG_BZIP2 */ default: - printf ("Unimplemented compression type %d\n", comp); + printf("Unimplemented compression type %d\n", comp); return 1; } - puts ("OK\n"); + puts("OK\n"); } sprintf(pbuf, "%8lx", data); diff --git a/common/image.c b/common/image.c index ed6b26b..9b4d4ad 100644 --- a/common/image.c +++ b/common/image.c @@ -45,7 +45,7 @@ #include -#if defined(CONFIG_FIT) || defined (CONFIG_OF_LIBFDT) +#if defined(CONFIG_FIT) || defined(CONFIG_OF_LIBFDT) #include #include #include @@ -55,7 +55,7 @@ #include #include -static int fit_check_ramdisk (const void *fit, int os_noffset, +static int fit_check_ramdisk(const void *fit, int os_noffset, uint8_t arch, int verify); #endif @@ -65,7 +65,7 @@ extern int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); DECLARE_GLOBAL_DATA_PTR; -static const image_header_t* image_get_ramdisk (ulong rd_addr, uint8_t arch, +static const image_header_t *image_get_ramdisk(ulong rd_addr, uint8_t arch, int verify); #else #include "mkimage.h" @@ -155,37 +155,37 @@ static const table_entry_t uimage_comp[] = { { -1, "", "", }, }; -uint32_t crc32 (uint32_t, const unsigned char *, uint); -uint32_t crc32_wd (uint32_t, const unsigned char *, uint, uint); +uint32_t crc32(uint32_t, const unsigned char *, uint); +uint32_t crc32_wd(uint32_t, const unsigned char *, uint, uint); #if defined(CONFIG_TIMESTAMP) || defined(CONFIG_CMD_DATE) || defined(USE_HOSTCC) -static void genimg_print_time (time_t timestamp); +static void genimg_print_time(time_t timestamp); #endif /*****************************************************************************/ /* Legacy format routines */ /*****************************************************************************/ -int image_check_hcrc (const image_header_t *hdr) +int image_check_hcrc(const image_header_t *hdr) { ulong hcrc; - ulong len = image_get_header_size (); + ulong len = image_get_header_size(); image_header_t header; /* Copy header so we can blank CRC field for re-calculation */ - memmove (&header, (char *)hdr, image_get_header_size ()); - image_set_hcrc (&header, 0); + memmove(&header, (char *)hdr, image_get_header_size()); + image_set_hcrc(&header, 0); - hcrc = crc32 (0, (unsigned char *)&header, len); + hcrc = crc32(0, (unsigned char *)&header, len); - return (hcrc == image_get_hcrc (hdr)); + return (hcrc == image_get_hcrc(hdr)); } -int image_check_dcrc (const image_header_t *hdr) +int image_check_dcrc(const image_header_t *hdr) { - ulong data = image_get_data (hdr); - ulong len = image_get_data_size (hdr); - ulong dcrc = crc32_wd (0, (unsigned char *)data, len, CHUNKSZ_CRC32); + ulong data = image_get_data(hdr); + ulong len = image_get_data_size(hdr); + ulong dcrc = crc32_wd(0, (unsigned char *)data, len, CHUNKSZ_CRC32); - return (dcrc == image_get_dcrc (hdr)); + return (dcrc == image_get_dcrc(hdr)); } /** @@ -201,14 +201,14 @@ int image_check_dcrc (const image_header_t *hdr) * returns: * number of components */ -ulong image_multi_count (const image_header_t *hdr) +ulong image_multi_count(const image_header_t *hdr) { ulong i, count = 0; uint32_t *size; /* get start of the image payload, which in case of multi * component images that points to a table of component sizes */ - size = (uint32_t *)image_get_data (hdr); + size = (uint32_t *)image_get_data(hdr); /* count non empty slots */ for (i = 0; size[i]; ++i) @@ -234,7 +234,7 @@ ulong image_multi_count (const image_header_t *hdr) * data address and size of the component, if idx is valid * 0 in data and len, if idx is out of range */ -void image_multi_getimg (const image_header_t *hdr, ulong idx, +void image_multi_getimg(const image_header_t *hdr, ulong idx, ulong *data, ulong *len) { int i; @@ -242,24 +242,24 @@ void image_multi_getimg (const image_header_t *hdr, ulong idx, ulong offset, count, img_data; /* get number of component */ - count = image_multi_count (hdr); + count = image_multi_count(hdr); /* get start of the image payload, which in case of multi * component images that points to a table of component sizes */ - size = (uint32_t *)image_get_data (hdr); + size = (uint32_t *)image_get_data(hdr); /* get address of the proper component data start, which means * skipping sizes table (add 1 for last, null entry) */ - img_data = image_get_data (hdr) + (count + 1) * sizeof (uint32_t); + img_data = image_get_data(hdr) + (count + 1) * sizeof(uint32_t); if (idx < count) { - *len = uimage_to_cpu (size[idx]); + *len = uimage_to_cpu(size[idx]); offset = 0; /* go over all indices preceding requested component idx */ for (i = 0; i < idx; i++) { /* add up i-th component size, rounding up to 4 bytes */ - offset += (uimage_to_cpu (size[i]) + 3) & ~3 ; + offset += (uimage_to_cpu(size[i]) + 3) & ~3 ; } /* calculate idx-th component data address */ @@ -270,16 +270,16 @@ void image_multi_getimg (const image_header_t *hdr, ulong idx, } } -static void image_print_type (const image_header_t *hdr) +static void image_print_type(const image_header_t *hdr) { const char *os, *arch, *type, *comp; - os = genimg_get_os_name (image_get_os (hdr)); - arch = genimg_get_arch_name (image_get_arch (hdr)); - type = genimg_get_type_name (image_get_type (hdr)); - comp = genimg_get_comp_name (image_get_comp (hdr)); + os = genimg_get_os_name(image_get_os(hdr)); + arch = genimg_get_arch_name(image_get_arch(hdr)); + type = genimg_get_type_name(image_get_type(hdr)); + comp = genimg_get_comp_name(image_get_comp(hdr)); - printf ("%s %s %s (%s)\n", arch, os, type, comp); + printf("%s %s %s (%s)\n", arch, os, type, comp); } /** @@ -294,7 +294,7 @@ static void image_print_type (const image_header_t *hdr) * returns: * no returned results */ -void image_print_contents (const void *ptr) +void image_print_contents(const void *ptr) { const image_header_t *hdr = (const image_header_t *)ptr; const char *p; @@ -305,38 +305,38 @@ void image_print_contents (const void *ptr) p = " "; #endif - printf ("%sImage Name: %.*s\n", p, IH_NMLEN, image_get_name (hdr)); + printf("%sImage Name: %.*s\n", p, IH_NMLEN, image_get_name(hdr)); #if defined(CONFIG_TIMESTAMP) || defined(CONFIG_CMD_DATE) || defined(USE_HOSTCC) - printf ("%sCreated: ", p); - genimg_print_time ((time_t)image_get_time (hdr)); + printf("%sCreated: ", p); + genimg_print_time((time_t)image_get_time(hdr)); #endif - printf ("%sImage Type: ", p); - image_print_type (hdr); - printf ("%sData Size: ", p); - genimg_print_size (image_get_data_size (hdr)); - printf ("%sLoad Address: %08x\n", p, image_get_load (hdr)); - printf ("%sEntry Point: %08x\n", p, image_get_ep (hdr)); - - if (image_check_type (hdr, IH_TYPE_MULTI) || - image_check_type (hdr, IH_TYPE_SCRIPT)) { + printf("%sImage Type: ", p); + image_print_type(hdr); + printf("%sData Size: ", p); + genimg_print_size(image_get_data_size(hdr)); + printf("%sLoad Address: %08x\n", p, image_get_load(hdr)); + printf("%sEntry Point: %08x\n", p, image_get_ep(hdr)); + + if (image_check_type(hdr, IH_TYPE_MULTI) || + image_check_type(hdr, IH_TYPE_SCRIPT)) { int i; ulong data, len; - ulong count = image_multi_count (hdr); + ulong count = image_multi_count(hdr); - printf ("%sContents:\n", p); + printf("%sContents:\n", p); for (i = 0; i < count; i++) { - image_multi_getimg (hdr, i, &data, &len); + image_multi_getimg(hdr, i, &data, &len); - printf ("%s Image %d: ", p, i); - genimg_print_size (len); + printf("%s Image %d: ", p, i); + genimg_print_size(len); - if (image_check_type (hdr, IH_TYPE_SCRIPT) && i > 0) { + if (image_check_type(hdr, IH_TYPE_SCRIPT) && i > 0) { /* * the user may need to know offsets * if planning to do something with * multiple files */ - printf ("%s Offset = 0x%08lx\n", p, data); + printf("%s Offset = 0x%08lx\n", p, data); } } } @@ -362,44 +362,44 @@ void image_print_contents (const void *ptr) * pointer to a ramdisk image header, if image was found and valid * otherwise, return NULL */ -static const image_header_t *image_get_ramdisk (ulong rd_addr, uint8_t arch, +static const image_header_t *image_get_ramdisk(ulong rd_addr, uint8_t arch, int verify) { const image_header_t *rd_hdr = (const image_header_t *)rd_addr; - if (!image_check_magic (rd_hdr)) { - puts ("Bad Magic Number\n"); - show_boot_progress (-10); + if (!image_check_magic(rd_hdr)) { + puts("Bad Magic Number\n"); + show_boot_progress(-10); return NULL; } - if (!image_check_hcrc (rd_hdr)) { - puts ("Bad Header Checksum\n"); - show_boot_progress (-11); + if (!image_check_hcrc(rd_hdr)) { + puts("Bad Header Checksum\n"); + show_boot_progress(-11); return NULL; } - show_boot_progress (10); - image_print_contents (rd_hdr); + show_boot_progress(10); + image_print_contents(rd_hdr); if (verify) { puts(" Verifying Checksum ... "); - if (!image_check_dcrc (rd_hdr)) { - puts ("Bad Data CRC\n"); - show_boot_progress (-12); + if (!image_check_dcrc(rd_hdr)) { + puts("Bad Data CRC\n"); + show_boot_progress(-12); return NULL; } puts("OK\n"); } - show_boot_progress (11); + show_boot_progress(11); - if (!image_check_os (rd_hdr, IH_OS_LINUX) || - !image_check_arch (rd_hdr, arch) || - !image_check_type (rd_hdr, IH_TYPE_RAMDISK)) { - printf ("No Linux %s Ramdisk Image\n", + if (!image_check_os(rd_hdr, IH_OS_LINUX) || + !image_check_arch(rd_hdr, arch) || + !image_check_type(rd_hdr, IH_TYPE_RAMDISK)) { + printf("No Linux %s Ramdisk Image\n", genimg_get_arch_name(arch)); - show_boot_progress (-13); + show_boot_progress(-13); return NULL; } @@ -411,17 +411,17 @@ static const image_header_t *image_get_ramdisk (ulong rd_addr, uint8_t arch, /* Shared dual-format routines */ /*****************************************************************************/ #ifndef USE_HOSTCC -int getenv_yesno (char *var) +int getenv_yesno(char *var) { - char *s = getenv (var); + char *s = getenv(var); return (s && (*s == 'n')) ? 0 : 1; } ulong getenv_bootm_low(void) { - char *s = getenv ("bootm_low"); + char *s = getenv("bootm_low"); if (s) { - ulong tmp = simple_strtoul (s, NULL, 16); + ulong tmp = simple_strtoul(s, NULL, 16); return tmp; } @@ -437,14 +437,14 @@ ulong getenv_bootm_low(void) phys_size_t getenv_bootm_size(void) { phys_size_t tmp; - char *s = getenv ("bootm_size"); + char *s = getenv("bootm_size"); if (s) { - tmp = (phys_size_t)simple_strtoull (s, NULL, 16); + tmp = (phys_size_t)simple_strtoull(s, NULL, 16); return tmp; } s = getenv("bootm_low"); if (s) - tmp = (phys_size_t)simple_strtoull (s, NULL, 16); + tmp = (phys_size_t)simple_strtoull(s, NULL, 16); else tmp = 0; @@ -459,9 +459,9 @@ phys_size_t getenv_bootm_size(void) phys_size_t getenv_bootm_mapsize(void) { phys_size_t tmp; - char *s = getenv ("bootm_mapsize"); + char *s = getenv("bootm_mapsize"); if (s) { - tmp = (phys_size_t)simple_strtoull (s, NULL, 16); + tmp = (phys_size_t)simple_strtoull(s, NULL, 16); return tmp; } @@ -472,7 +472,7 @@ phys_size_t getenv_bootm_mapsize(void) #endif } -void memmove_wd (void *to, void *from, size_t len, ulong chunksz) +void memmove_wd(void *to, void *from, size_t len, ulong chunksz) { if (to == from) return; @@ -480,42 +480,42 @@ void memmove_wd (void *to, void *from, size_t len, ulong chunksz) #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG) while (len > 0) { size_t tail = (len > chunksz) ? chunksz : len; - WATCHDOG_RESET (); - memmove (to, from, tail); + WATCHDOG_RESET(); + memmove(to, from, tail); to += tail; from += tail; len -= tail; } #else /* !(CONFIG_HW_WATCHDOG || CONFIG_WATCHDOG) */ - memmove (to, from, len); + memmove(to, from, len); #endif /* CONFIG_HW_WATCHDOG || CONFIG_WATCHDOG */ } #endif /* !USE_HOSTCC */ -void genimg_print_size (uint32_t size) +void genimg_print_size(uint32_t size) { #ifndef USE_HOSTCC - printf ("%d Bytes = ", size); - print_size (size, "\n"); + printf("%d Bytes = ", size); + print_size(size, "\n"); #else - printf ("%d Bytes = %.2f kB = %.2f MB\n", + printf("%d Bytes = %.2f kB = %.2f MB\n", size, (double)size / 1.024e3, (double)size / 1.048576e6); #endif } #if defined(CONFIG_TIMESTAMP) || defined(CONFIG_CMD_DATE) || defined(USE_HOSTCC) -static void genimg_print_time (time_t timestamp) +static void genimg_print_time(time_t timestamp) { #ifndef USE_HOSTCC struct rtc_time tm; - to_tm (timestamp, &tm); - printf ("%4d-%02d-%02d %2d:%02d:%02d UTC\n", + to_tm(timestamp, &tm); + printf("%4d-%02d-%02d %2d:%02d:%02d UTC\n", tm.tm_year, tm.tm_mon, tm.tm_mday, tm.tm_hour, tm.tm_min, tm.tm_sec); #else - printf ("%s", ctime(×tamp)); + printf("%s", ctime(×tamp)); #endif } #endif /* CONFIG_TIMESTAMP || CONFIG_CMD_DATE || USE_HOSTCC */ @@ -547,24 +547,26 @@ char *get_table_entry_name(const table_entry_t *table, char *msg, int id) return (msg); } -const char *genimg_get_os_name (uint8_t os) +const char *genimg_get_os_name(uint8_t os) { - return (get_table_entry_name (uimage_os, "Unknown OS", os)); + return (get_table_entry_name(uimage_os, "Unknown OS", os)); } -const char *genimg_get_arch_name (uint8_t arch) +const char *genimg_get_arch_name(uint8_t arch) { - return (get_table_entry_name (uimage_arch, "Unknown Architecture", arch)); + return (get_table_entry_name(uimage_arch, "Unknown Architecture", + arch)); } -const char *genimg_get_type_name (uint8_t type) +const char *genimg_get_type_name(uint8_t type) { - return (get_table_entry_name (uimage_type, "Unknown Image", type)); + return (get_table_entry_name(uimage_type, "Unknown Image", type)); } -const char *genimg_get_comp_name (uint8_t comp) +const char *genimg_get_comp_name(uint8_t comp) { - return (get_table_entry_name (uimage_comp, "Unknown Compression", comp)); + return (get_table_entry_name(uimage_comp, "Unknown Compression", + comp)); } /** @@ -590,17 +592,17 @@ int get_table_entry_id(const table_entry_t *table, for (t = table; t->id >= 0; ++t) { if (t->sname && strcasecmp(t->sname, name) == 0) - return (t->id); + return(t->id); } - fprintf (stderr, "\nInvalid %s Type - valid names are", table_name); + fprintf(stderr, "\nInvalid %s Type - valid names are", table_name); for (t = table; t->id >= 0; ++t) { if (t->sname == NULL) continue; - fprintf (stderr, "%c %s", (first) ? ':' : ',', t->sname); + fprintf(stderr, "%c %s", (first) ? ':' : ',', t->sname); first = 0; } - fprintf (stderr, "\n"); + fprintf(stderr, "\n"); #else for (t = table; t->id >= 0; ++t) { #ifdef CONFIG_NEEDS_MANUAL_RELOC @@ -610,29 +612,29 @@ int get_table_entry_id(const table_entry_t *table, #endif return (t->id); } - debug ("Invalid %s Type: %s\n", table_name, name); + debug("Invalid %s Type: %s\n", table_name, name); #endif /* USE_HOSTCC */ return (-1); } -int genimg_get_os_id (const char *name) +int genimg_get_os_id(const char *name) { - return (get_table_entry_id (uimage_os, "OS", name)); + return (get_table_entry_id(uimage_os, "OS", name)); } -int genimg_get_arch_id (const char *name) +int genimg_get_arch_id(const char *name) { - return (get_table_entry_id (uimage_arch, "CPU", name)); + return (get_table_entry_id(uimage_arch, "CPU", name)); } -int genimg_get_type_id (const char *name) +int genimg_get_type_id(const char *name) { - return (get_table_entry_id (uimage_type, "Image", name)); + return (get_table_entry_id(uimage_type, "Image", name)); } -int genimg_get_comp_id (const char *name) +int genimg_get_comp_id(const char *name) { - return (get_table_entry_id (uimage_comp, "Compression", name)); + return (get_table_entry_id(uimage_comp, "Compression", name)); } #ifndef USE_HOSTCC @@ -650,7 +652,7 @@ int genimg_get_comp_id (const char *name) * returns: * image format type or IMAGE_FORMAT_INVALID if no image is present */ -int genimg_get_format (void *img_addr) +int genimg_get_format(void *img_addr) { ulong format = IMAGE_FORMAT_INVALID; const image_header_t *hdr; @@ -664,7 +666,7 @@ int genimg_get_format (void *img_addr) #if defined(CONFIG_FIT) || defined(CONFIG_OF_LIBFDT) else { fit_hdr = (char *)img_addr; - if (fdt_check_header (fit_hdr) == 0) + if (fdt_check_header(fit_hdr) == 0) format = IMAGE_FORMAT_FIT; } #endif @@ -682,55 +684,59 @@ int genimg_get_format (void *img_addr) * returns: * image start address after possible relocation from special storage */ -ulong genimg_get_image (ulong img_addr) +ulong genimg_get_image(ulong img_addr) { ulong ram_addr = img_addr; #ifdef CONFIG_HAS_DATAFLASH ulong h_size, d_size; - if (addr_dataflash (img_addr)){ + if (addr_dataflash(img_addr)) { /* ger RAM address */ ram_addr = CONFIG_SYS_LOAD_ADDR; /* get header size */ - h_size = image_get_header_size (); + h_size = image_get_header_size(); #if defined(CONFIG_FIT) if (sizeof(struct fdt_header) > h_size) h_size = sizeof(struct fdt_header); #endif /* read in header */ - debug (" Reading image header from dataflash address " + debug(" Reading image header from dataflash address " "%08lx to RAM address %08lx\n", img_addr, ram_addr); - read_dataflash (img_addr, h_size, (char *)ram_addr); + read_dataflash(img_addr, h_size, (char *)ram_addr); /* get data size */ - switch (genimg_get_format ((void *)ram_addr)) { + switch (genimg_get_format((void *)ram_addr)) { case IMAGE_FORMAT_LEGACY: - d_size = image_get_data_size ((const image_header_t *)ram_addr); - debug (" Legacy format image found at 0x%08lx, size 0x%08lx\n", + d_size = image_get_data_size( + (const image_header_t *)ram_addr); + debug(" Legacy format image found at 0x%08lx, " + "size 0x%08lx\n", ram_addr, d_size); break; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: - d_size = fit_get_size ((const void *)ram_addr) - h_size; - debug (" FIT/FDT format image found at 0x%08lx, size 0x%08lx\n", + d_size = fit_get_size((const void *)ram_addr) - h_size; + debug(" FIT/FDT format image found at 0x%08lx, " + "size 0x%08lx\n", ram_addr, d_size); break; #endif default: - printf (" No valid image found at 0x%08lx\n", img_addr); + printf(" No valid image found at 0x%08lx\n", + img_addr); return ram_addr; } /* read in image data */ - debug (" Reading image remaining data from dataflash address " + debug(" Reading image remaining data from dataflash address " "%08lx to RAM address %08lx\n", img_addr + h_size, ram_addr + h_size); - read_dataflash (img_addr + h_size, d_size, + read_dataflash(img_addr + h_size, d_size, (char *)(ram_addr + h_size)); } @@ -750,7 +756,7 @@ ulong genimg_get_image (ulong img_addr) * 0, no FIT support or no configuration found * 1, configuration found */ -int genimg_has_config (bootm_headers_t *images) +int genimg_has_config(bootm_headers_t *images) { #if defined(CONFIG_FIT) if (images->fit_uname_cfg) @@ -781,7 +787,7 @@ int genimg_has_config (bootm_headers_t *images) * 1, if ramdisk image is found but corrupted, or invalid * rd_start and rd_end are set to 0 if no ramdisk exists */ -int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, +int boot_get_ramdisk(int argc, char * const argv[], bootm_headers_t *images, uint8_t arch, ulong *rd_start, ulong *rd_end) { ulong rd_addr, rd_load; @@ -806,9 +812,9 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, * ramdisk argument */ if ((argc >= 3) && (strcmp(argv[2], "-") == 0)) { - debug ("## Skipping init Ramdisk\n"); + debug("## Skipping init Ramdisk\n"); rd_len = rd_data = 0; - } else if (argc >= 3 || genimg_has_config (images)) { + } else if (argc >= 3 || genimg_has_config(images)) { #if defined(CONFIG_FIT) if (argc >= 3) { /* @@ -822,19 +828,22 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, else default_addr = load_addr; - if (fit_parse_conf (argv[2], default_addr, + if (fit_parse_conf(argv[2], default_addr, &rd_addr, &fit_uname_config)) { - debug ("* ramdisk: config '%s' from image at 0x%08lx\n", + debug("* ramdisk: config '%s' from image at " + "0x%08lx\n", fit_uname_config, rd_addr); - } else if (fit_parse_subimage (argv[2], default_addr, + } else if (fit_parse_subimage(argv[2], default_addr, &rd_addr, &fit_uname_ramdisk)) { - debug ("* ramdisk: subimage '%s' from image at 0x%08lx\n", + debug("* ramdisk: subimage '%s' from image at " + "0x%08lx\n", fit_uname_ramdisk, rd_addr); } else #endif { rd_addr = simple_strtoul(argv[2], NULL, 16); - debug ("* ramdisk: cmdline image address = 0x%08lx\n", + debug("* ramdisk: cmdline image address = " + "0x%08lx\n", rd_addr); } #if defined(CONFIG_FIT) @@ -844,7 +853,8 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, */ rd_addr = (ulong)images->fit_hdr_os; fit_uname_config = images->fit_uname_cfg; - debug ("* ramdisk: using config '%s' from image at 0x%08lx\n", + debug("* ramdisk: using config '%s' from image " + "at 0x%08lx\n", fit_uname_config, rd_addr); /* @@ -852,57 +862,59 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, * if not, don't try to use it, quit silently. */ fit_hdr = (void *)rd_addr; - cfg_noffset = fit_conf_get_node (fit_hdr, fit_uname_config); + cfg_noffset = fit_conf_get_node(fit_hdr, + fit_uname_config); if (cfg_noffset < 0) { - debug ("* ramdisk: no such config\n"); + debug("* ramdisk: no such config\n"); return 1; } - rd_noffset = fit_conf_get_ramdisk_node (fit_hdr, cfg_noffset); + rd_noffset = fit_conf_get_ramdisk_node(fit_hdr, + cfg_noffset); if (rd_noffset < 0) { - debug ("* ramdisk: no ramdisk in config\n"); + debug("* ramdisk: no ramdisk in config\n"); return 0; } } #endif /* copy from dataflash if needed */ - rd_addr = genimg_get_image (rd_addr); + rd_addr = genimg_get_image(rd_addr); /* * Check if there is an initrd image at the * address provided in the second bootm argument * check image type, for FIT images get FIT node. */ - switch (genimg_get_format ((void *)rd_addr)) { + switch (genimg_get_format((void *)rd_addr)) { case IMAGE_FORMAT_LEGACY: - printf ("## Loading init Ramdisk from Legacy " + printf("## Loading init Ramdisk from Legacy " "Image at %08lx ...\n", rd_addr); - show_boot_progress (9); - rd_hdr = image_get_ramdisk (rd_addr, arch, + show_boot_progress(9); + rd_hdr = image_get_ramdisk(rd_addr, arch, images->verify); if (rd_hdr == NULL) return 1; - rd_data = image_get_data (rd_hdr); - rd_len = image_get_data_size (rd_hdr); - rd_load = image_get_load (rd_hdr); + rd_data = image_get_data(rd_hdr); + rd_len = image_get_data_size(rd_hdr); + rd_load = image_get_load(rd_hdr); break; #if defined(CONFIG_FIT) case IMAGE_FORMAT_FIT: fit_hdr = (void *)rd_addr; - printf ("## Loading init Ramdisk from FIT " + printf("## Loading init Ramdisk from FIT " "Image at %08lx ...\n", rd_addr); - show_boot_progress (120); - if (!fit_check_format (fit_hdr)) { - puts ("Bad FIT ramdisk image format!\n"); - show_boot_progress (-120); + show_boot_progress(120); + if (!fit_check_format(fit_hdr)) { + puts("Bad FIT ramdisk image format!\n"); + show_boot_progress(-120); return 1; } - show_boot_progress (121); + show_boot_progress(121); if (!fit_uname_ramdisk) { /* @@ -910,52 +922,63 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, * node first. If config unit node name is NULL * fit_conf_get_node() will try to find default config node */ - show_boot_progress (122); - cfg_noffset = fit_conf_get_node (fit_hdr, fit_uname_config); + show_boot_progress(122); + cfg_noffset = fit_conf_get_node(fit_hdr, + fit_uname_config); if (cfg_noffset < 0) { - puts ("Could not find configuration node\n"); - show_boot_progress (-122); + puts("Could not find configuration " + "node\n"); + show_boot_progress(-122); return 1; } - fit_uname_config = fdt_get_name (fit_hdr, cfg_noffset, NULL); - printf (" Using '%s' configuration\n", fit_uname_config); + fit_uname_config = fdt_get_name(fit_hdr, + cfg_noffset, NULL); + printf(" Using '%s' configuration\n", + fit_uname_config); - rd_noffset = fit_conf_get_ramdisk_node (fit_hdr, cfg_noffset); - fit_uname_ramdisk = fit_get_name (fit_hdr, rd_noffset, NULL); + rd_noffset = fit_conf_get_ramdisk_node(fit_hdr, + cfg_noffset); + fit_uname_ramdisk = fit_get_name(fit_hdr, + rd_noffset, NULL); } else { /* get ramdisk component image node offset */ - show_boot_progress (123); - rd_noffset = fit_image_get_node (fit_hdr, fit_uname_ramdisk); + show_boot_progress(123); + rd_noffset = fit_image_get_node(fit_hdr, + fit_uname_ramdisk); } if (rd_noffset < 0) { - puts ("Could not find subimage node\n"); - show_boot_progress (-124); + puts("Could not find subimage node\n"); + show_boot_progress(-124); return 1; } - printf (" Trying '%s' ramdisk subimage\n", fit_uname_ramdisk); + printf(" Trying '%s' ramdisk subimage\n", + fit_uname_ramdisk); - show_boot_progress (125); - if (!fit_check_ramdisk (fit_hdr, rd_noffset, arch, images->verify)) + show_boot_progress(125); + if (!fit_check_ramdisk(fit_hdr, rd_noffset, arch, + images->verify)) return 1; /* get ramdisk image data address and length */ - if (fit_image_get_data (fit_hdr, rd_noffset, &data, &size)) { - puts ("Could not find ramdisk subimage data!\n"); - show_boot_progress (-127); + if (fit_image_get_data(fit_hdr, rd_noffset, &data, + &size)) { + puts("Could not find ramdisk subimage data!\n"); + show_boot_progress(-127); return 1; } - show_boot_progress (128); + show_boot_progress(128); rd_data = (ulong)data; rd_len = size; - if (fit_image_get_load (fit_hdr, rd_noffset, &rd_load)) { - puts ("Can't get ramdisk subimage load address!\n"); - show_boot_progress (-129); + if (fit_image_get_load(fit_hdr, rd_noffset, &rd_load)) { + puts("Can't get ramdisk subimage load " + "address!\n"); + show_boot_progress(-129); return 1; } - show_boot_progress (129); + show_boot_progress(129); images->fit_hdr_rd = fit_hdr; images->fit_uname_rd = fit_uname_ramdisk; @@ -963,37 +986,39 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, break; #endif default: - puts ("Wrong Ramdisk Image Format\n"); + puts("Wrong Ramdisk Image Format\n"); rd_data = rd_len = rd_load = 0; return 1; } } else if (images->legacy_hdr_valid && - image_check_type (&images->legacy_hdr_os_copy, IH_TYPE_MULTI)) { + image_check_type(&images->legacy_hdr_os_copy, + IH_TYPE_MULTI)) { + /* * Now check if we have a legacy mult-component image, * get second entry data start address and len. */ - show_boot_progress (13); - printf ("## Loading init Ramdisk from multi component " + show_boot_progress(13); + printf("## Loading init Ramdisk from multi component " "Legacy Image at %08lx ...\n", (ulong)images->legacy_hdr_os); - image_multi_getimg (images->legacy_hdr_os, 1, &rd_data, &rd_len); + image_multi_getimg(images->legacy_hdr_os, 1, &rd_data, &rd_len); } else { /* * no initrd image */ - show_boot_progress (14); + show_boot_progress(14); rd_len = rd_data = 0; } if (!rd_data) { - debug ("## No init Ramdisk\n"); + debug("## No init Ramdisk\n"); } else { *rd_start = rd_data; *rd_end = rd_data + rd_len; } - debug (" ramdisk start = 0x%08lx, ramdisk end = 0x%08lx\n", + debug(" ramdisk start = 0x%08lx, ramdisk end = 0x%08lx\n", *rd_start, *rd_end); return 0; @@ -1021,18 +1046,18 @@ int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, * 0 - success * -1 - failure */ -int boot_ramdisk_high (struct lmb *lmb, ulong rd_data, ulong rd_len, +int boot_ramdisk_high(struct lmb *lmb, ulong rd_data, ulong rd_len, ulong *initrd_start, ulong *initrd_end) { char *s; ulong initrd_high; int initrd_copy_to_ram = 1; - if ((s = getenv ("initrd_high")) != NULL) { + if ((s = getenv("initrd_high")) != NULL) { /* a value of "no" or a similar string will act like 0, * turning the "load high" feature off. This is intentional. */ - initrd_high = simple_strtoul (s, NULL, 16); + initrd_high = simple_strtoul(s, NULL, 16); if (initrd_high == ~0) initrd_copy_to_ram = 0; } else { @@ -1046,41 +1071,43 @@ int boot_ramdisk_high (struct lmb *lmb, ulong rd_data, ulong rd_len, lmb_reserve(lmb, logbuffer_base() - LOGBUFF_OVERHEAD, LOGBUFF_RESERVE); #endif - debug ("## initrd_high = 0x%08lx, copy_to_ram = %d\n", + debug("## initrd_high = 0x%08lx, copy_to_ram = %d\n", initrd_high, initrd_copy_to_ram); if (rd_data) { if (!initrd_copy_to_ram) { /* zero-copy ramdisk support */ - debug (" in-place initrd\n"); + debug(" in-place initrd\n"); *initrd_start = rd_data; *initrd_end = rd_data + rd_len; lmb_reserve(lmb, rd_data, rd_len); } else { if (initrd_high) - *initrd_start = (ulong)lmb_alloc_base (lmb, rd_len, 0x1000, initrd_high); + *initrd_start = (ulong)lmb_alloc_base(lmb, + rd_len, 0x1000, initrd_high); else - *initrd_start = (ulong)lmb_alloc (lmb, rd_len, 0x1000); + *initrd_start = (ulong)lmb_alloc(lmb, rd_len, + 0x1000); if (*initrd_start == 0) { - puts ("ramdisk - allocation error\n"); + puts("ramdisk - allocation error\n"); goto error; } - show_boot_progress (12); + show_boot_progress(12); *initrd_end = *initrd_start + rd_len; - printf (" Loading Ramdisk to %08lx, end %08lx ... ", + printf(" Loading Ramdisk to %08lx, end %08lx ... ", *initrd_start, *initrd_end); - memmove_wd ((void *)*initrd_start, + memmove_wd((void *)*initrd_start, (void *)rd_data, rd_len, CHUNKSZ); - puts ("OK\n"); + puts("OK\n"); } } else { *initrd_start = 0; *initrd_end = 0; } - debug (" ramdisk load start = 0x%08lx, ramdisk load end = 0x%08lx\n", + debug(" ramdisk load start = 0x%08lx, ramdisk load end = 0x%08lx\n", *initrd_start, *initrd_end); return 0; @@ -1091,41 +1118,41 @@ error: #endif /* CONFIG_SYS_BOOT_RAMDISK_HIGH */ #ifdef CONFIG_OF_LIBFDT -static void fdt_error (const char *msg) +static void fdt_error(const char *msg) { - puts ("ERROR: "); - puts (msg); - puts (" - must RESET the board to recover.\n"); + puts("ERROR: "); + puts(msg); + puts(" - must RESET the board to recover.\n"); } -static const image_header_t *image_get_fdt (ulong fdt_addr) +static const image_header_t *image_get_fdt(ulong fdt_addr) { const image_header_t *fdt_hdr = (const image_header_t *)fdt_addr; - image_print_contents (fdt_hdr); + image_print_contents(fdt_hdr); - puts (" Verifying Checksum ... "); - if (!image_check_hcrc (fdt_hdr)) { - fdt_error ("fdt header checksum invalid"); + puts(" Verifying Checksum ... "); + if (!image_check_hcrc(fdt_hdr)) { + fdt_error("fdt header checksum invalid"); return NULL; } - if (!image_check_dcrc (fdt_hdr)) { - fdt_error ("fdt checksum invalid"); + if (!image_check_dcrc(fdt_hdr)) { + fdt_error("fdt checksum invalid"); return NULL; } - puts ("OK\n"); + puts("OK\n"); - if (!image_check_type (fdt_hdr, IH_TYPE_FLATDT)) { - fdt_error ("uImage is not a fdt"); + if (!image_check_type(fdt_hdr, IH_TYPE_FLATDT)) { + fdt_error("uImage is not a fdt"); return NULL; } - if (image_get_comp (fdt_hdr) != IH_COMP_NONE) { - fdt_error ("uImage is compressed"); + if (image_get_comp(fdt_hdr) != IH_COMP_NONE) { + fdt_error("uImage is compressed"); return NULL; } - if (fdt_check_header ((char *)image_get_data (fdt_hdr)) != 0) { - fdt_error ("uImage data is not a fdt"); + if (fdt_check_header((char *)image_get_data(fdt_hdr)) != 0) { + fdt_error("uImage data is not a fdt"); return NULL; } return fdt_hdr; @@ -1145,26 +1172,26 @@ static const image_header_t *image_get_fdt (ulong fdt_addr) * 0, on failure */ #if defined(CONFIG_FIT) -static int fit_check_fdt (const void *fit, int fdt_noffset, int verify) +static int fit_check_fdt(const void *fit, int fdt_noffset, int verify) { - fit_image_print (fit, fdt_noffset, " "); + fit_image_print(fit, fdt_noffset, " "); if (verify) { - puts (" Verifying Hash Integrity ... "); - if (!fit_image_check_hashes (fit, fdt_noffset)) { - fdt_error ("Bad Data Hash"); + puts(" Verifying Hash Integrity ... "); + if (!fit_image_check_hashes(fit, fdt_noffset)) { + fdt_error("Bad Data Hash"); return 0; } - puts ("OK\n"); + puts("OK\n"); } - if (!fit_image_check_type (fit, fdt_noffset, IH_TYPE_FLATDT)) { - fdt_error ("Not a FDT image"); + if (!fit_image_check_type(fit, fdt_noffset, IH_TYPE_FLATDT)) { + fdt_error("Not a FDT image"); return 0; } - if (!fit_image_check_comp (fit, fdt_noffset, IH_COMP_NONE)) { - fdt_error ("FDT image is compressed"); + if (!fit_image_check_comp(fit, fdt_noffset, IH_COMP_NONE)) { + fdt_error("FDT image is compressed"); return 0; } @@ -1191,7 +1218,7 @@ void boot_fdt_add_mem_rsv_regions(struct lmb *lmb, void *fdt_blob) uint64_t addr, size; int i, total; - if (fdt_check_header (fdt_blob) != 0) + if (fdt_check_header(fdt_blob) != 0) return; total = fdt_num_mem_rsv(fdt_blob); @@ -1221,7 +1248,7 @@ void boot_fdt_add_mem_rsv_regions(struct lmb *lmb, void *fdt_blob) * 0 - success * 1 - failure */ -int boot_relocate_fdt (struct lmb *lmb, char **of_flat_tree, ulong *of_size) +int boot_relocate_fdt(struct lmb *lmb, char **of_flat_tree, ulong *of_size) { void *fdt_blob = *of_flat_tree; void *of_start = 0; @@ -1234,8 +1261,8 @@ int boot_relocate_fdt (struct lmb *lmb, char **of_flat_tree, ulong *of_size) if (*of_size == 0) return 0; - if (fdt_check_header (fdt_blob) != 0) { - fdt_error ("image is not a fdt"); + if (fdt_check_header(fdt_blob) != 0) { + fdt_error("image is not a fdt"); goto error; } @@ -1285,18 +1312,18 @@ int boot_relocate_fdt (struct lmb *lmb, char **of_flat_tree, ulong *of_size) printf(" Using Device Tree in place at %p, end %p\n", of_start, of_start + of_len - 1); } else { - debug ("## device tree at %p ... %p (len=%ld [0x%lX])\n", + debug("## device tree at %p ... %p (len=%ld [0x%lX])\n", fdt_blob, fdt_blob + *of_size - 1, of_len, of_len); - printf (" Loading Device Tree to %p, end %p ... ", + printf(" Loading Device Tree to %p, end %p ... ", of_start, of_start + of_len - 1); - err = fdt_open_into (fdt_blob, of_start, of_len); + err = fdt_open_into(fdt_blob, of_start, of_len); if (err != 0) { - fdt_error ("fdt move failed"); + fdt_error("fdt move failed"); goto error; } - puts ("OK\n"); + puts("OK\n"); } *of_flat_tree = of_start; @@ -1331,8 +1358,8 @@ error: * 1, if fdt image is found but corrupted * of_flat_tree and of_size are set to 0 if no fdt exists */ -int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *images, - char **of_flat_tree, ulong *of_size) +int boot_get_fdt(int flag, int argc, char * const argv[], + bootm_headers_t *images, char **of_flat_tree, ulong *of_size) { const image_header_t *fdt_hdr; ulong fdt_addr; @@ -1353,7 +1380,7 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag *of_flat_tree = NULL; *of_size = 0; - if (argc > 3 || genimg_has_config (images)) { + if (argc > 3 || genimg_has_config(images)) { #if defined(CONFIG_FIT) if (argc > 3) { /* @@ -1369,19 +1396,22 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag else default_addr = load_addr; - if (fit_parse_conf (argv[3], default_addr, + if (fit_parse_conf(argv[3], default_addr, &fdt_addr, &fit_uname_config)) { - debug ("* fdt: config '%s' from image at 0x%08lx\n", + debug("* fdt: config '%s' from image at " + "0x%08lx\n", fit_uname_config, fdt_addr); - } else if (fit_parse_subimage (argv[3], default_addr, + } else if (fit_parse_subimage(argv[3], default_addr, &fdt_addr, &fit_uname_fdt)) { - debug ("* fdt: subimage '%s' from image at 0x%08lx\n", + debug("* fdt: subimage '%s' from image at " + "0x%08lx\n", fit_uname_fdt, fdt_addr); } else #endif { fdt_addr = simple_strtoul(argv[3], NULL, 16); - debug ("* fdt: cmdline image address = 0x%08lx\n", + debug("* fdt: cmdline image address = " + "0x%08lx\n", fdt_addr); } #if defined(CONFIG_FIT) @@ -1391,7 +1421,8 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag */ fdt_addr = (ulong)images->fit_hdr_os; fit_uname_config = images->fit_uname_cfg; - debug ("* fdt: using config '%s' from image at 0x%08lx\n", + debug("* fdt: using config '%s' from image " + "at 0x%08lx\n", fit_uname_config, fdt_addr); /* @@ -1399,39 +1430,40 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag * if not quit silently. */ fit_hdr = (void *)fdt_addr; - cfg_noffset = fit_conf_get_node (fit_hdr, + cfg_noffset = fit_conf_get_node(fit_hdr, fit_uname_config); if (cfg_noffset < 0) { - debug ("* fdt: no such config\n"); + debug("* fdt: no such config\n"); return 0; } - fdt_noffset = fit_conf_get_fdt_node (fit_hdr, + fdt_noffset = fit_conf_get_fdt_node(fit_hdr, cfg_noffset); if (fdt_noffset < 0) { - debug ("* fdt: no fdt in config\n"); + debug("* fdt: no fdt in config\n"); return 0; } } #endif - debug ("## Checking for 'FDT'/'FDT Image' at %08lx\n", + debug("## Checking for 'FDT'/'FDT Image' at %08lx\n", fdt_addr); /* copy from dataflash if needed */ - fdt_addr = genimg_get_image (fdt_addr); + fdt_addr = genimg_get_image(fdt_addr); /* * Check if there is an FDT image at the * address provided in the second bootm argument * check image type, for FIT images get a FIT node. */ - switch (genimg_get_format ((void *)fdt_addr)) { + switch (genimg_get_format((void *)fdt_addr)) { case IMAGE_FORMAT_LEGACY: /* verify fdt_addr points to a valid image header */ - printf ("## Flattened Device Tree from Legacy Image at %08lx\n", + printf("## Flattened Device Tree from Legacy Image " + "at %08lx\n", fdt_addr); - fdt_hdr = image_get_fdt (fdt_addr); + fdt_hdr = image_get_fdt(fdt_addr); if (!fdt_hdr) goto error; @@ -1440,22 +1472,22 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag * make sure we don't overwrite initial image */ image_start = (ulong)fdt_hdr; - image_end = image_get_image_end (fdt_hdr); + image_end = image_get_image_end(fdt_hdr); - load_start = image_get_load (fdt_hdr); - load_end = load_start + image_get_data_size (fdt_hdr); + load_start = image_get_load(fdt_hdr); + load_end = load_start + image_get_data_size(fdt_hdr); if ((load_start < image_end) && (load_end > image_start)) { - fdt_error ("fdt overwritten"); + fdt_error("fdt overwritten"); goto error; } - debug (" Loading FDT from 0x%08lx to 0x%08lx\n", - image_get_data (fdt_hdr), load_start); + debug(" Loading FDT from 0x%08lx to 0x%08lx\n", + image_get_data(fdt_hdr), load_start); - memmove ((void *)load_start, - (void *)image_get_data (fdt_hdr), - image_get_data_size (fdt_hdr)); + memmove((void *)load_start, + (void *)image_get_data(fdt_hdr), + image_get_data_size(fdt_hdr)); fdt_blob = (char *)load_start; break; @@ -1467,12 +1499,13 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag */ #if defined(CONFIG_FIT) /* check FDT blob vs FIT blob */ - if (fit_check_format ((const void *)fdt_addr)) { + if (fit_check_format((const void *)fdt_addr)) { /* * FIT image */ fit_hdr = (void *)fdt_addr; - printf ("## Flattened Device Tree from FIT Image at %08lx\n", + printf("## Flattened Device Tree from FIT " + "Image at %08lx\n", fdt_addr); if (!fit_uname_fdt) { @@ -1483,50 +1516,56 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag * fit_conf_get_node() will try to * find default config node */ - cfg_noffset = fit_conf_get_node (fit_hdr, + cfg_noffset = fit_conf_get_node(fit_hdr, fit_uname_config); if (cfg_noffset < 0) { - fdt_error ("Could not find configuration node\n"); + fdt_error("Could not find " + "configuration " + "node\n"); goto error; } - fit_uname_config = fdt_get_name (fit_hdr, + fit_uname_config = fdt_get_name(fit_hdr, cfg_noffset, NULL); - printf (" Using '%s' configuration\n", + printf(" Using '%s' configuration\n", fit_uname_config); - fdt_noffset = fit_conf_get_fdt_node (fit_hdr, + fdt_noffset = fit_conf_get_fdt_node( + fit_hdr, cfg_noffset); - fit_uname_fdt = fit_get_name (fit_hdr, + fit_uname_fdt = fit_get_name(fit_hdr, fdt_noffset, NULL); } else { /* get FDT component image node offset */ - fdt_noffset = fit_image_get_node (fit_hdr, - fit_uname_fdt); + fdt_noffset = fit_image_get_node( + fit_hdr, + fit_uname_fdt); } if (fdt_noffset < 0) { - fdt_error ("Could not find subimage node\n"); + fdt_error("Could not find subimage " + "node\n"); goto error; } - printf (" Trying '%s' FDT blob subimage\n", + printf(" Trying '%s' FDT blob subimage\n", fit_uname_fdt); - if (!fit_check_fdt (fit_hdr, fdt_noffset, + if (!fit_check_fdt(fit_hdr, fdt_noffset, images->verify)) goto error; /* get ramdisk image data address and length */ - if (fit_image_get_data (fit_hdr, fdt_noffset, + if (fit_image_get_data(fit_hdr, fdt_noffset, &data, &size)) { - fdt_error ("Could not find FDT subimage data"); + fdt_error("Could not find FDT " + "subimage data"); goto error; } /* verift that image data is a proper FDT blob */ - if (fdt_check_header ((char *)data) != 0) { - fdt_error ("Subimage data is not a FTD"); + if (fdt_check_header((char *)data) != 0) { + fdt_error("Subimage data is not a FTD"); goto error; } @@ -1535,22 +1574,24 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag * make sure we don't overwrite initial image */ image_start = (ulong)fit_hdr; - image_end = fit_get_end (fit_hdr); + image_end = fit_get_end(fit_hdr); - if (fit_image_get_load (fit_hdr, fdt_noffset, + if (fit_image_get_load(fit_hdr, fdt_noffset, &load_start) == 0) { load_end = load_start + size; if ((load_start < image_end) && (load_end > image_start)) { - fdt_error ("FDT overwritten"); + fdt_error("FDT overwritten"); goto error; } - printf (" Loading FDT from 0x%08lx to 0x%08lx\n", - (ulong)data, load_start); + printf(" Loading FDT from 0x%08lx " + "to 0x%08lx\n", + (ulong)data, + load_start); - memmove ((void *)load_start, + memmove((void *)load_start, (void *)data, size); fdt_blob = (char *)load_start; @@ -1569,19 +1610,23 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag * FDT blob */ fdt_blob = (char *)fdt_addr; - debug ("* fdt: raw FDT blob\n"); - printf ("## Flattened Device Tree blob at %08lx\n", (long)fdt_blob); + debug("* fdt: raw FDT blob\n"); + printf("## Flattened Device Tree blob at " + "%08lx\n", (long)fdt_blob); } break; default: - puts ("ERROR: Did not find a cmdline Flattened Device Tree\n"); + puts("ERROR: Did not find a cmdline Flattened Device " + "Tree\n"); goto error; } - printf (" Booting using the fdt blob at 0x%x\n", (int)fdt_blob); + printf(" Booting using the fdt blob at 0x%x\n", + (int)fdt_blob); } else if (images->legacy_hdr_valid && - image_check_type (&images->legacy_hdr_os_copy, IH_TYPE_MULTI)) { + image_check_type(&images->legacy_hdr_os_copy, + IH_TYPE_MULTI)) { ulong fdt_data, fdt_len; @@ -1589,37 +1634,39 @@ int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *imag * Now check if we have a legacy multi-component image, * get second entry data start address and len. */ - printf ("## Flattened Device Tree from multi " + printf("## Flattened Device Tree from multi " "component Image at %08lX\n", (ulong)images->legacy_hdr_os); - image_multi_getimg (images->legacy_hdr_os, 2, &fdt_data, &fdt_len); + image_multi_getimg(images->legacy_hdr_os, 2, &fdt_data, + &fdt_len); if (fdt_len) { fdt_blob = (char *)fdt_data; - printf (" Booting using the fdt at 0x%x\n", (int)fdt_blob); + printf(" Booting using the fdt at 0x%x\n", + (int)fdt_blob); - if (fdt_check_header (fdt_blob) != 0) { - fdt_error ("image is not a fdt"); + if (fdt_check_header(fdt_blob) != 0) { + fdt_error("image is not a fdt"); goto error; } if (fdt_totalsize(fdt_blob) != fdt_len) { - fdt_error ("fdt size != image size"); + fdt_error("fdt size != image size"); goto error; } } else { - debug ("## No Flattened Device Tree\n"); + debug("## No Flattened Device Tree\n"); return 0; } } else { - debug ("## No Flattened Device Tree\n"); + debug("## No Flattened Device Tree\n"); return 0; } *of_flat_tree = fdt_blob; *of_size = fdt_totalsize(fdt_blob); - debug (" of_flat_tree at 0x%08lx size 0x%08lx\n", + debug(" of_flat_tree at 0x%08lx size 0x%08lx\n", (ulong)*of_flat_tree, *of_size); return 0; @@ -1647,7 +1694,7 @@ error: * 0 - success * -1 - failure */ -int boot_get_cmdline (struct lmb *lmb, ulong *cmd_start, ulong *cmd_end) +int boot_get_cmdline(struct lmb *lmb, ulong *cmd_start, ulong *cmd_end) { char *cmdline; char *s; @@ -1666,7 +1713,7 @@ int boot_get_cmdline (struct lmb *lmb, ulong *cmd_start, ulong *cmd_end) *cmd_start = (ulong) & cmdline[0]; *cmd_end = *cmd_start + strlen(cmdline); - debug ("## cmdline at 0x%08lx ... 0x%08lx\n", *cmd_start, *cmd_end); + debug("## cmdline at 0x%08lx ... 0x%08lx\n", *cmd_start, *cmd_end); return 0; } @@ -1686,7 +1733,7 @@ int boot_get_cmdline (struct lmb *lmb, ulong *cmd_start, ulong *cmd_end) * 0 - success * -1 - failure */ -int boot_get_kbd (struct lmb *lmb, bd_t **kbd) +int boot_get_kbd(struct lmb *lmb, bd_t **kbd) { *kbd = (bd_t *)(ulong)lmb_alloc_base(lmb, sizeof(bd_t), 0xf, getenv_bootm_mapsize() + getenv_bootm_low()); @@ -1695,7 +1742,7 @@ int boot_get_kbd (struct lmb *lmb, bd_t **kbd) **kbd = *(gd->bd); - debug ("## kernel board info at 0x%08lx\n", (ulong)*kbd); + debug("## kernel board info at 0x%08lx\n", (ulong)*kbd); #if defined(DEBUG) && defined(CONFIG_CMD_BDI) do_bdinfo(NULL, 0, 0, NULL); @@ -1711,7 +1758,7 @@ int boot_get_kbd (struct lmb *lmb, bd_t **kbd) /* New uImage format routines */ /*****************************************************************************/ #ifndef USE_HOSTCC -static int fit_parse_spec (const char *spec, char sepc, ulong addr_curr, +static int fit_parse_spec(const char *spec, char sepc, ulong addr_curr, ulong *addr, const char **name) { const char *sep; @@ -1719,10 +1766,10 @@ static int fit_parse_spec (const char *spec, char sepc, ulong addr_curr, *addr = addr_curr; *name = NULL; - sep = strchr (spec, sepc); + sep = strchr(spec, sepc); if (sep) { if (sep - spec > 0) - *addr = simple_strtoul (spec, NULL, 16); + *addr = simple_strtoul(spec, NULL, 16); *name = sep + 1; return 1; @@ -1752,10 +1799,10 @@ static int fit_parse_spec (const char *spec, char sepc, ulong addr_curr, * addr and conf_name are set accordingly * 0 otherwise */ -inline int fit_parse_conf (const char *spec, ulong addr_curr, +inline int fit_parse_conf(const char *spec, ulong addr_curr, ulong *addr, const char **conf_name) { - return fit_parse_spec (spec, '#', addr_curr, addr, conf_name); + return fit_parse_spec(spec, '#', addr_curr, addr, conf_name); } /** @@ -1778,21 +1825,21 @@ inline int fit_parse_conf (const char *spec, ulong addr_curr, * addr and image_name are set accordingly * 0 otherwise */ -inline int fit_parse_subimage (const char *spec, ulong addr_curr, +inline int fit_parse_subimage(const char *spec, ulong addr_curr, ulong *addr, const char **image_name) { - return fit_parse_spec (spec, ':', addr_curr, addr, image_name); + return fit_parse_spec(spec, ':', addr_curr, addr, image_name); } #endif /* !USE_HOSTCC */ -static void fit_get_debug (const void *fit, int noffset, +static void fit_get_debug(const void *fit, int noffset, char *prop_name, int err) { - debug ("Can't get '%s' property from FIT 0x%08lx, " + debug("Can't get '%s' property from FIT 0x%08lx, " "node: offset %d, name %s (%s)\n", prop_name, (ulong)fit, noffset, - fit_get_name (fit, noffset, NULL), - fdt_strerror (err)); + fit_get_name(fit, noffset, NULL), + fdt_strerror(err)); } /** @@ -1807,7 +1854,7 @@ static void fit_get_debug (const void *fit, int noffset, * returns: * no returned results */ -void fit_print_contents (const void *fit) +void fit_print_contents(const void *fit) { char *desc; char *uname; @@ -1829,72 +1876,74 @@ void fit_print_contents (const void *fit) #endif /* Root node properties */ - ret = fit_get_desc (fit, 0, &desc); - printf ("%sFIT description: ", p); + ret = fit_get_desc(fit, 0, &desc); + printf("%sFIT description: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("%s\n", desc); + printf("%s\n", desc); #if defined(CONFIG_TIMESTAMP) || defined(CONFIG_CMD_DATE) || defined(USE_HOSTCC) - ret = fit_get_timestamp (fit, 0, ×tamp); - printf ("%sCreated: ", p); + ret = fit_get_timestamp(fit, 0, ×tamp); + printf("%sCreated: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - genimg_print_time (timestamp); + genimg_print_time(timestamp); #endif /* Find images parent node offset */ - images_noffset = fdt_path_offset (fit, FIT_IMAGES_PATH); + images_noffset = fdt_path_offset(fit, FIT_IMAGES_PATH); if (images_noffset < 0) { - printf ("Can't find images parent node '%s' (%s)\n", - FIT_IMAGES_PATH, fdt_strerror (images_noffset)); + printf("Can't find images parent node '%s' (%s)\n", + FIT_IMAGES_PATH, fdt_strerror(images_noffset)); return; } /* Process its subnodes, print out component images details */ - for (ndepth = 0, count = 0, noffset = fdt_next_node (fit, images_noffset, &ndepth); + for (ndepth = 0, count = 0, + noffset = fdt_next_node(fit, images_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* * Direct child node of the images parent node, * i.e. component image node. */ - printf ("%s Image %u (%s)\n", p, count++, + printf("%s Image %u (%s)\n", p, count++, fit_get_name(fit, noffset, NULL)); - fit_image_print (fit, noffset, p); + fit_image_print(fit, noffset, p); } } /* Find configurations parent node offset */ - confs_noffset = fdt_path_offset (fit, FIT_CONFS_PATH); + confs_noffset = fdt_path_offset(fit, FIT_CONFS_PATH); if (confs_noffset < 0) { - debug ("Can't get configurations parent node '%s' (%s)\n", - FIT_CONFS_PATH, fdt_strerror (confs_noffset)); + debug("Can't get configurations parent node '%s' (%s)\n", + FIT_CONFS_PATH, fdt_strerror(confs_noffset)); return; } /* get default configuration unit name from default property */ - uname = (char *)fdt_getprop (fit, noffset, FIT_DEFAULT_PROP, NULL); + uname = (char *)fdt_getprop(fit, noffset, FIT_DEFAULT_PROP, NULL); if (uname) - printf ("%s Default Configuration: '%s'\n", p, uname); + printf("%s Default Configuration: '%s'\n", p, uname); /* Process its subnodes, print out configurations details */ - for (ndepth = 0, count = 0, noffset = fdt_next_node (fit, confs_noffset, &ndepth); + for (ndepth = 0, count = 0, + noffset = fdt_next_node(fit, confs_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* * Direct child node of the configurations parent node, * i.e. configuration node. */ - printf ("%s Configuration %u (%s)\n", p, count++, + printf("%s Configuration %u (%s)\n", p, count++, fit_get_name(fit, noffset, NULL)); - fit_conf_print (fit, noffset, p); + fit_conf_print(fit, noffset, p); } } } @@ -1914,7 +1963,7 @@ void fit_print_contents (const void *fit) * returns: * no returned results */ -void fit_image_print (const void *fit, int image_noffset, const char *p) +void fit_image_print(const void *fit, int image_noffset, const char *p) { char *desc; uint8_t type, arch, os, comp; @@ -1926,74 +1975,74 @@ void fit_image_print (const void *fit, int image_noffset, const char *p) int ret; /* Mandatory properties */ - ret = fit_get_desc (fit, image_noffset, &desc); - printf ("%s Description: ", p); + ret = fit_get_desc(fit, image_noffset, &desc); + printf("%s Description: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("%s\n", desc); + printf("%s\n", desc); - fit_image_get_type (fit, image_noffset, &type); - printf ("%s Type: %s\n", p, genimg_get_type_name (type)); + fit_image_get_type(fit, image_noffset, &type); + printf("%s Type: %s\n", p, genimg_get_type_name(type)); - fit_image_get_comp (fit, image_noffset, &comp); - printf ("%s Compression: %s\n", p, genimg_get_comp_name (comp)); + fit_image_get_comp(fit, image_noffset, &comp); + printf("%s Compression: %s\n", p, genimg_get_comp_name(comp)); - ret = fit_image_get_data (fit, image_noffset, &data, &size); + ret = fit_image_get_data(fit, image_noffset, &data, &size); #ifndef USE_HOSTCC - printf ("%s Data Start: ", p); + printf("%s Data Start: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("0x%08lx\n", (ulong)data); + printf("0x%08lx\n", (ulong)data); #endif - printf ("%s Data Size: ", p); + printf("%s Data Size: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - genimg_print_size (size); + genimg_print_size(size); /* Remaining, type dependent properties */ if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE) || (type == IH_TYPE_RAMDISK) || (type == IH_TYPE_FIRMWARE) || (type == IH_TYPE_FLATDT)) { - fit_image_get_arch (fit, image_noffset, &arch); - printf ("%s Architecture: %s\n", p, genimg_get_arch_name (arch)); + fit_image_get_arch(fit, image_noffset, &arch); + printf("%s Architecture: %s\n", p, genimg_get_arch_name(arch)); } if (type == IH_TYPE_KERNEL) { - fit_image_get_os (fit, image_noffset, &os); - printf ("%s OS: %s\n", p, genimg_get_os_name (os)); + fit_image_get_os(fit, image_noffset, &os); + printf("%s OS: %s\n", p, genimg_get_os_name(os)); } if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE) || (type == IH_TYPE_FIRMWARE)) { - ret = fit_image_get_load (fit, image_noffset, &load); - printf ("%s Load Address: ", p); + ret = fit_image_get_load(fit, image_noffset, &load); + printf("%s Load Address: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("0x%08lx\n", load); + printf("0x%08lx\n", load); } if ((type == IH_TYPE_KERNEL) || (type == IH_TYPE_STANDALONE)) { - fit_image_get_entry (fit, image_noffset, &entry); - printf ("%s Entry Point: ", p); + fit_image_get_entry(fit, image_noffset, &entry); + printf("%s Entry Point: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("0x%08lx\n", entry); + printf("0x%08lx\n", entry); } /* Process all hash subnodes of the component image node */ - for (ndepth = 0, noffset = fdt_next_node (fit, image_noffset, &ndepth); + for (ndepth = 0, noffset = fdt_next_node(fit, image_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* Direct child node of the component image node */ - fit_image_print_hash (fit, noffset, p); + fit_image_print_hash(fit, noffset, p); } } } @@ -2009,7 +2058,7 @@ void fit_image_print (const void *fit, int image_noffset, const char *p) * returns: * no returned results */ -void fit_image_print_hash (const void *fit, int noffset, const char *p) +void fit_image_print_hash(const void *fit, int noffset, const char *p) { char *algo; uint8_t *value; @@ -2021,33 +2070,33 @@ void fit_image_print_hash (const void *fit, int noffset, const char *p) * Multiple hash nodes require unique unit node * names, e.g. hash@1, hash@2, etc. */ - if (strncmp (fit_get_name(fit, noffset, NULL), + if (strncmp(fit_get_name(fit, noffset, NULL), FIT_HASH_NODENAME, strlen(FIT_HASH_NODENAME)) != 0) return; - debug ("%s Hash node: '%s'\n", p, - fit_get_name (fit, noffset, NULL)); + debug("%s Hash node: '%s'\n", p, + fit_get_name(fit, noffset, NULL)); - printf ("%s Hash algo: ", p); - if (fit_image_hash_get_algo (fit, noffset, &algo)) { - printf ("invalid/unsupported\n"); + printf("%s Hash algo: ", p); + if (fit_image_hash_get_algo(fit, noffset, &algo)) { + printf("invalid/unsupported\n"); return; } - printf ("%s\n", algo); + printf("%s\n", algo); - ret = fit_image_hash_get_value (fit, noffset, &value, + ret = fit_image_hash_get_value(fit, noffset, &value, &value_len); - printf ("%s Hash value: ", p); + printf("%s Hash value: ", p); if (ret) { - printf ("unavailable\n"); + printf("unavailable\n"); } else { for (i = 0; i < value_len; i++) - printf ("%02x", value[i]); - printf ("\n"); + printf("%02x", value[i]); + printf("\n"); } - debug ("%s Hash len: %d\n", p, value_len); + debug("%s Hash len: %d\n", p, value_len); } /** @@ -2063,13 +2112,13 @@ void fit_image_print_hash (const void *fit, int noffset, const char *p) * 0, on success * -1, on failure */ -int fit_get_desc (const void *fit, int noffset, char **desc) +int fit_get_desc(const void *fit, int noffset, char **desc) { int len; - *desc = (char *)fdt_getprop (fit, noffset, FIT_DESC_PROP, &len); + *desc = (char *)fdt_getprop(fit, noffset, FIT_DESC_PROP, &len); if (*desc == NULL) { - fit_get_debug (fit, noffset, FIT_DESC_PROP, len); + fit_get_debug(fit, noffset, FIT_DESC_PROP, len); return -1; } @@ -2091,22 +2140,22 @@ int fit_get_desc (const void *fit, int noffset, char **desc) * -1, on property read failure * -2, on wrong timestamp size */ -int fit_get_timestamp (const void *fit, int noffset, time_t *timestamp) +int fit_get_timestamp(const void *fit, int noffset, time_t *timestamp) { int len; const void *data; - data = fdt_getprop (fit, noffset, FIT_TIMESTAMP_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_TIMESTAMP_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_TIMESTAMP_PROP, len); + fit_get_debug(fit, noffset, FIT_TIMESTAMP_PROP, len); return -1; } - if (len != sizeof (uint32_t)) { - debug ("FIT timestamp with incorrect size of (%u)\n", len); + if (len != sizeof(uint32_t)) { + debug("FIT timestamp with incorrect size of (%u)\n", len); return -2; } - *timestamp = uimage_to_cpu (*((uint32_t *)data)); + *timestamp = uimage_to_cpu(*((uint32_t *)data)); return 0; } @@ -2123,21 +2172,21 @@ int fit_get_timestamp (const void *fit, int noffset, time_t *timestamp) * image node offset when found (>=0) * negative number on failure (FDT_ERR_* code) */ -int fit_image_get_node (const void *fit, const char *image_uname) +int fit_image_get_node(const void *fit, const char *image_uname) { int noffset, images_noffset; - images_noffset = fdt_path_offset (fit, FIT_IMAGES_PATH); + images_noffset = fdt_path_offset(fit, FIT_IMAGES_PATH); if (images_noffset < 0) { - debug ("Can't find images parent node '%s' (%s)\n", - FIT_IMAGES_PATH, fdt_strerror (images_noffset)); + debug("Can't find images parent node '%s' (%s)\n", + FIT_IMAGES_PATH, fdt_strerror(images_noffset)); return images_noffset; } - noffset = fdt_subnode_offset (fit, images_noffset, image_uname); + noffset = fdt_subnode_offset(fit, images_noffset, image_uname); if (noffset < 0) { - debug ("Can't get node offset for image unit name: '%s' (%s)\n", - image_uname, fdt_strerror (noffset)); + debug("Can't get node offset for image unit name: '%s' (%s)\n", + image_uname, fdt_strerror(noffset)); } return noffset; @@ -2157,21 +2206,21 @@ int fit_image_get_node (const void *fit, const char *image_uname) * 0, on success * -1, on failure */ -int fit_image_get_os (const void *fit, int noffset, uint8_t *os) +int fit_image_get_os(const void *fit, int noffset, uint8_t *os) { int len; const void *data; /* Get OS name from property data */ - data = fdt_getprop (fit, noffset, FIT_OS_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_OS_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_OS_PROP, len); + fit_get_debug(fit, noffset, FIT_OS_PROP, len); *os = -1; return -1; } /* Translate OS name to id */ - *os = genimg_get_os_id (data); + *os = genimg_get_os_id(data); return 0; } @@ -2189,21 +2238,21 @@ int fit_image_get_os (const void *fit, int noffset, uint8_t *os) * 0, on success * -1, on failure */ -int fit_image_get_arch (const void *fit, int noffset, uint8_t *arch) +int fit_image_get_arch(const void *fit, int noffset, uint8_t *arch) { int len; const void *data; /* Get architecture name from property data */ - data = fdt_getprop (fit, noffset, FIT_ARCH_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_ARCH_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_ARCH_PROP, len); + fit_get_debug(fit, noffset, FIT_ARCH_PROP, len); *arch = -1; return -1; } /* Translate architecture name to id */ - *arch = genimg_get_arch_id (data); + *arch = genimg_get_arch_id(data); return 0; } @@ -2221,21 +2270,21 @@ int fit_image_get_arch (const void *fit, int noffset, uint8_t *arch) * 0, on success * -1, on failure */ -int fit_image_get_type (const void *fit, int noffset, uint8_t *type) +int fit_image_get_type(const void *fit, int noffset, uint8_t *type) { int len; const void *data; /* Get image type name from property data */ - data = fdt_getprop (fit, noffset, FIT_TYPE_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_TYPE_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_TYPE_PROP, len); + fit_get_debug(fit, noffset, FIT_TYPE_PROP, len); *type = -1; return -1; } /* Translate image type name to id */ - *type = genimg_get_type_id (data); + *type = genimg_get_type_id(data); return 0; } @@ -2253,21 +2302,21 @@ int fit_image_get_type (const void *fit, int noffset, uint8_t *type) * 0, on success * -1, on failure */ -int fit_image_get_comp (const void *fit, int noffset, uint8_t *comp) +int fit_image_get_comp(const void *fit, int noffset, uint8_t *comp) { int len; const void *data; /* Get compression name from property data */ - data = fdt_getprop (fit, noffset, FIT_COMP_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_COMP_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_COMP_PROP, len); + fit_get_debug(fit, noffset, FIT_COMP_PROP, len); *comp = -1; return -1; } /* Translate compression name to id */ - *comp = genimg_get_comp_id (data); + *comp = genimg_get_comp_id(data); return 0; } @@ -2284,18 +2333,18 @@ int fit_image_get_comp (const void *fit, int noffset, uint8_t *comp) * 0, on success * -1, on failure */ -int fit_image_get_load (const void *fit, int noffset, ulong *load) +int fit_image_get_load(const void *fit, int noffset, ulong *load) { int len; const uint32_t *data; - data = fdt_getprop (fit, noffset, FIT_LOAD_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_LOAD_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_LOAD_PROP, len); + fit_get_debug(fit, noffset, FIT_LOAD_PROP, len); return -1; } - *load = uimage_to_cpu (*data); + *load = uimage_to_cpu(*data); return 0; } @@ -2312,18 +2361,18 @@ int fit_image_get_load (const void *fit, int noffset, ulong *load) * 0, on success * -1, on failure */ -int fit_image_get_entry (const void *fit, int noffset, ulong *entry) +int fit_image_get_entry(const void *fit, int noffset, ulong *entry) { int len; const uint32_t *data; - data = fdt_getprop (fit, noffset, FIT_ENTRY_PROP, &len); + data = fdt_getprop(fit, noffset, FIT_ENTRY_PROP, &len); if (data == NULL) { - fit_get_debug (fit, noffset, FIT_ENTRY_PROP, len); + fit_get_debug(fit, noffset, FIT_ENTRY_PROP, len); return -1; } - *entry = uimage_to_cpu (*data); + *entry = uimage_to_cpu(*data); return 0; } @@ -2342,14 +2391,14 @@ int fit_image_get_entry (const void *fit, int noffset, ulong *entry) * 0, on success * -1, on failure */ -int fit_image_get_data (const void *fit, int noffset, +int fit_image_get_data(const void *fit, int noffset, const void **data, size_t *size) { int len; - *data = fdt_getprop (fit, noffset, FIT_DATA_PROP, &len); + *data = fdt_getprop(fit, noffset, FIT_DATA_PROP, &len); if (*data == NULL) { - fit_get_debug (fit, noffset, FIT_DATA_PROP, len); + fit_get_debug(fit, noffset, FIT_DATA_PROP, len); *size = 0; return -1; } @@ -2371,13 +2420,13 @@ int fit_image_get_data (const void *fit, int noffset, * 0, on success * -1, on failure */ -int fit_image_hash_get_algo (const void *fit, int noffset, char **algo) +int fit_image_hash_get_algo(const void *fit, int noffset, char **algo) { int len; - *algo = (char *)fdt_getprop (fit, noffset, FIT_ALGO_PROP, &len); + *algo = (char *)fdt_getprop(fit, noffset, FIT_ALGO_PROP, &len); if (*algo == NULL) { - fit_get_debug (fit, noffset, FIT_ALGO_PROP, len); + fit_get_debug(fit, noffset, FIT_ALGO_PROP, len); return -1; } @@ -2399,14 +2448,14 @@ int fit_image_hash_get_algo (const void *fit, int noffset, char **algo) * 0, on success * -1, on failure */ -int fit_image_hash_get_value (const void *fit, int noffset, uint8_t **value, +int fit_image_hash_get_value(const void *fit, int noffset, uint8_t **value, int *value_len) { int len; - *value = (uint8_t *)fdt_getprop (fit, noffset, FIT_VALUE_PROP, &len); + *value = (uint8_t *)fdt_getprop(fit, noffset, FIT_VALUE_PROP, &len); if (*value == NULL) { - fit_get_debug (fit, noffset, FIT_VALUE_PROP, len); + fit_get_debug(fit, noffset, FIT_VALUE_PROP, len); *value_len = 0; return -1; } @@ -2428,18 +2477,18 @@ int fit_image_hash_get_value (const void *fit, int noffset, uint8_t **value, * 0, on success * -1, on property read failure */ -int fit_set_timestamp (void *fit, int noffset, time_t timestamp) +int fit_set_timestamp(void *fit, int noffset, time_t timestamp) { uint32_t t; int ret; - t = cpu_to_uimage (timestamp); - ret = fdt_setprop (fit, noffset, FIT_TIMESTAMP_PROP, &t, - sizeof (uint32_t)); + t = cpu_to_uimage(timestamp); + ret = fdt_setprop(fit, noffset, FIT_TIMESTAMP_PROP, &t, + sizeof(uint32_t)); if (ret) { - printf ("Can't set '%s' property for '%s' node (%s)\n", - FIT_TIMESTAMP_PROP, fit_get_name (fit, noffset, NULL), - fdt_strerror (ret)); + printf("Can't set '%s' property for '%s' node (%s)\n", + FIT_TIMESTAMP_PROP, fit_get_name(fit, noffset, NULL), + fdt_strerror(ret)); return -1; } @@ -2463,23 +2512,23 @@ int fit_set_timestamp (void *fit, int noffset, time_t timestamp) * 0, on success * -1, when algo is unsupported */ -static int calculate_hash (const void *data, int data_len, const char *algo, +static int calculate_hash(const void *data, int data_len, const char *algo, uint8_t *value, int *value_len) { - if (strcmp (algo, "crc32") == 0 ) { - *((uint32_t *)value) = crc32_wd (0, data, data_len, + if (strcmp(algo, "crc32") == 0) { + *((uint32_t *)value) = crc32_wd(0, data, data_len, CHUNKSZ_CRC32); - *((uint32_t *)value) = cpu_to_uimage (*((uint32_t *)value)); + *((uint32_t *)value) = cpu_to_uimage(*((uint32_t *)value)); *value_len = 4; - } else if (strcmp (algo, "sha1") == 0 ) { - sha1_csum_wd ((unsigned char *) data, data_len, + } else if (strcmp(algo, "sha1") == 0) { + sha1_csum_wd((unsigned char *) data, data_len, (unsigned char *) value, CHUNKSZ_SHA1); *value_len = 20; - } else if (strcmp (algo, "md5") == 0 ) { - md5_wd ((unsigned char *)data, data_len, value, CHUNKSZ_MD5); + } else if (strcmp(algo, "md5") == 0) { + md5_wd((unsigned char *)data, data_len, value, CHUNKSZ_MD5); *value_len = 16; } else { - debug ("Unsupported hash alogrithm\n"); + debug("Unsupported hash alogrithm\n"); return -1; } return 0; @@ -2498,7 +2547,7 @@ static int calculate_hash (const void *data, int data_len, const char *algo, * 0, on success * libfdt error code, on failure */ -int fit_set_hashes (void *fit) +int fit_set_hashes(void *fit) { int images_noffset; int noffset; @@ -2506,23 +2555,23 @@ int fit_set_hashes (void *fit) int ret; /* Find images parent node offset */ - images_noffset = fdt_path_offset (fit, FIT_IMAGES_PATH); + images_noffset = fdt_path_offset(fit, FIT_IMAGES_PATH); if (images_noffset < 0) { - printf ("Can't find images parent node '%s' (%s)\n", - FIT_IMAGES_PATH, fdt_strerror (images_noffset)); + printf("Can't find images parent node '%s' (%s)\n", + FIT_IMAGES_PATH, fdt_strerror(images_noffset)); return images_noffset; } /* Process its subnodes, print out component images details */ - for (ndepth = 0, noffset = fdt_next_node (fit, images_noffset, &ndepth); + for (ndepth = 0, noffset = fdt_next_node(fit, images_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* * Direct child node of the images parent node, * i.e. component image node. */ - ret = fit_image_set_hashes (fit, noffset); + ret = fit_image_set_hashes(fit, noffset); if (ret) return ret; } @@ -2560,7 +2609,7 @@ int fit_set_hashes (void *fit) * 0 on sucess * <0 on failure */ -int fit_image_set_hashes (void *fit, int image_noffset) +int fit_image_set_hashes(void *fit, int image_noffset) { const void *data; size_t size; @@ -2571,15 +2620,15 @@ int fit_image_set_hashes (void *fit, int image_noffset) int ndepth; /* Get image data and data length */ - if (fit_image_get_data (fit, image_noffset, &data, &size)) { - printf ("Can't get image data/size\n"); + if (fit_image_get_data(fit, image_noffset, &data, &size)) { + printf("Can't get image data/size\n"); return -1; } /* Process all hash subnodes of the component image node */ - for (ndepth = 0, noffset = fdt_next_node (fit, image_noffset, &ndepth); + for (ndepth = 0, noffset = fdt_next_node(fit, image_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* Direct child node of the component image node */ @@ -2588,35 +2637,37 @@ int fit_image_set_hashes (void *fit, int image_noffset) * Multiple hash nodes require unique unit node * names, e.g. hash@1, hash@2, etc. */ - if (strncmp (fit_get_name(fit, noffset, NULL), + if (strncmp(fit_get_name(fit, noffset, NULL), FIT_HASH_NODENAME, strlen(FIT_HASH_NODENAME)) != 0) { /* Not a hash subnode, skip it */ continue; } - if (fit_image_hash_get_algo (fit, noffset, &algo)) { - printf ("Can't get hash algo property for " + if (fit_image_hash_get_algo(fit, noffset, &algo)) { + printf("Can't get hash algo property for " "'%s' hash node in '%s' image node\n", - fit_get_name (fit, noffset, NULL), - fit_get_name (fit, image_noffset, NULL)); + fit_get_name(fit, noffset, NULL), + fit_get_name(fit, image_noffset, NULL)); return -1; } - if (calculate_hash (data, size, algo, value, &value_len)) { - printf ("Unsupported hash algorithm (%s) for " + if (calculate_hash(data, size, algo, value, + &value_len)) { + printf("Unsupported hash algorithm (%s) for " "'%s' hash node in '%s' image node\n", - algo, fit_get_name (fit, noffset, NULL), - fit_get_name (fit, image_noffset, NULL)); + algo, fit_get_name(fit, noffset, NULL), + fit_get_name(fit, image_noffset, + NULL)); return -1; } - if (fit_image_hash_set_value (fit, noffset, value, + if (fit_image_hash_set_value(fit, noffset, value, value_len)) { - printf ("Can't set hash value for " + printf("Can't set hash value for " "'%s' hash node in '%s' image node\n", - fit_get_name (fit, noffset, NULL), - fit_get_name (fit, image_noffset, NULL)); + fit_get_name(fit, noffset, NULL), + fit_get_name(fit, image_noffset, NULL)); return -1; } } @@ -2639,16 +2690,16 @@ int fit_image_set_hashes (void *fit, int image_noffset) * 0, on success * -1, on failure */ -int fit_image_hash_set_value (void *fit, int noffset, uint8_t *value, +int fit_image_hash_set_value(void *fit, int noffset, uint8_t *value, int value_len) { int ret; - ret = fdt_setprop (fit, noffset, FIT_VALUE_PROP, value, value_len); + ret = fdt_setprop(fit, noffset, FIT_VALUE_PROP, value, value_len); if (ret) { - printf ("Can't set hash '%s' property for '%s' node (%s)\n", - FIT_VALUE_PROP, fit_get_name (fit, noffset, NULL), - fdt_strerror (ret)); + printf("Can't set hash '%s' property for '%s' node(%s)\n", + FIT_VALUE_PROP, fit_get_name(fit, noffset, NULL), + fdt_strerror(ret)); return -1; } @@ -2669,7 +2720,7 @@ int fit_image_hash_set_value (void *fit, int noffset, uint8_t *value, * 1, if all hashes are valid * 0, otherwise (or on error) */ -int fit_image_check_hashes (const void *fit, int image_noffset) +int fit_image_check_hashes(const void *fit, int image_noffset) { const void *data; size_t size; @@ -2683,15 +2734,15 @@ int fit_image_check_hashes (const void *fit, int image_noffset) char *err_msg = ""; /* Get image data and data length */ - if (fit_image_get_data (fit, image_noffset, &data, &size)) { - printf ("Can't get image data/size\n"); + if (fit_image_get_data(fit, image_noffset, &data, &size)) { + printf("Can't get image data/size\n"); return 0; } /* Process all hash subnodes of the component image node */ - for (ndepth = 0, noffset = fdt_next_node (fit, image_noffset, &ndepth); + for (ndepth = 0, noffset = fdt_next_node(fit, image_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* Direct child node of the component image node */ @@ -2700,47 +2751,49 @@ int fit_image_check_hashes (const void *fit, int image_noffset) * Multiple hash nodes require unique unit node * names, e.g. hash@1, hash@2, etc. */ - if (strncmp (fit_get_name(fit, noffset, NULL), + if (strncmp(fit_get_name(fit, noffset, NULL), FIT_HASH_NODENAME, strlen(FIT_HASH_NODENAME)) != 0) continue; - if (fit_image_hash_get_algo (fit, noffset, &algo)) { + if (fit_image_hash_get_algo(fit, noffset, &algo)) { err_msg = " error!\nCan't get hash algo " "property"; goto error; } - printf ("%s", algo); + printf("%s", algo); - if (fit_image_hash_get_value (fit, noffset, &fit_value, + if (fit_image_hash_get_value(fit, noffset, &fit_value, &fit_value_len)) { err_msg = " error!\nCan't get hash value " "property"; goto error; } - if (calculate_hash (data, size, algo, value, &value_len)) { - err_msg = " error!\nUnsupported hash algorithm"; + if (calculate_hash(data, size, algo, value, + &value_len)) { + err_msg = " error!\n" + "Unsupported hash algorithm"; goto error; } if (value_len != fit_value_len) { err_msg = " error !\nBad hash value len"; goto error; - } else if (memcmp (value, fit_value, value_len) != 0) { + } else if (memcmp(value, fit_value, value_len) != 0) { err_msg = " error!\nBad hash value"; goto error; } - printf ("+ "); + printf("+ "); } } return 1; error: - printf ("%s for '%s' hash node in '%s' image node\n", - err_msg, fit_get_name (fit, noffset, NULL), - fit_get_name (fit, image_noffset, NULL)); + printf("%s for '%s' hash node in '%s' image node\n", + err_msg, fit_get_name(fit, noffset, NULL), + fit_get_name(fit, image_noffset, NULL)); return 0; } @@ -2755,7 +2808,7 @@ error: * 1, if all hashes of all images are valid * 0, otherwise (or on error) */ -int fit_all_image_check_hashes (const void *fit) +int fit_all_image_check_hashes(const void *fit) { int images_noffset; int noffset; @@ -2763,31 +2816,31 @@ int fit_all_image_check_hashes (const void *fit) int count; /* Find images parent node offset */ - images_noffset = fdt_path_offset (fit, FIT_IMAGES_PATH); + images_noffset = fdt_path_offset(fit, FIT_IMAGES_PATH); if (images_noffset < 0) { - printf ("Can't find images parent node '%s' (%s)\n", - FIT_IMAGES_PATH, fdt_strerror (images_noffset)); + printf("Can't find images parent node '%s' (%s)\n", + FIT_IMAGES_PATH, fdt_strerror(images_noffset)); return 0; } /* Process all image subnodes, check hashes for each */ - printf ("## Checking hash(es) for FIT Image at %08lx ...\n", + printf("## Checking hash(es) for FIT Image at %08lx ...\n", (ulong)fit); for (ndepth = 0, count = 0, - noffset = fdt_next_node (fit, images_noffset, &ndepth); + noffset = fdt_next_node(fit, images_noffset, &ndepth); (noffset >= 0) && (ndepth > 0); - noffset = fdt_next_node (fit, noffset, &ndepth)) { + noffset = fdt_next_node(fit, noffset, &ndepth)) { if (ndepth == 1) { /* * Direct child node of the images parent node, * i.e. component image node. */ - printf (" Hash(es) for Image %u (%s): ", count++, - fit_get_name (fit, noffset, NULL)); + printf(" Hash(es) for Image %u (%s): ", count++, + fit_get_name(fit, noffset, NULL)); - if (!fit_image_check_hashes (fit, noffset)) + if (!fit_image_check_hashes(fit, noffset)) return 0; - printf ("\n"); + printf("\n"); } } return 1; @@ -2806,11 +2859,11 @@ int fit_all_image_check_hashes (const void *fit) * 1 if image is of given os type * 0 otherwise (or on error) */ -int fit_image_check_os (const void *fit, int noffset, uint8_t os) +int fit_image_check_os(const void *fit, int noffset, uint8_t os) { uint8_t image_os; - if (fit_image_get_os (fit, noffset, &image_os)) + if (fit_image_get_os(fit, noffset, &image_os)) return 0; return (os == image_os); } @@ -2828,11 +2881,11 @@ int fit_image_check_os (const void *fit, int noffset, uint8_t os) * 1 if image is of given arch * 0 otherwise (or on error) */ -int fit_image_check_arch (const void *fit, int noffset, uint8_t arch) +int fit_image_check_arch(const void *fit, int noffset, uint8_t arch) { uint8_t image_arch; - if (fit_image_get_arch (fit, noffset, &image_arch)) + if (fit_image_get_arch(fit, noffset, &image_arch)) return 0; return (arch == image_arch); } @@ -2850,11 +2903,11 @@ int fit_image_check_arch (const void *fit, int noffset, uint8_t arch) * 1 if image is of given type * 0 otherwise (or on error) */ -int fit_image_check_type (const void *fit, int noffset, uint8_t type) +int fit_image_check_type(const void *fit, int noffset, uint8_t type) { uint8_t image_type; - if (fit_image_get_type (fit, noffset, &image_type)) + if (fit_image_get_type(fit, noffset, &image_type)) return 0; return (type == image_type); } @@ -2873,11 +2926,11 @@ int fit_image_check_type (const void *fit, int noffset, uint8_t type) * 1 if image uses requested compression * 0 otherwise (or on error) */ -int fit_image_check_comp (const void *fit, int noffset, uint8_t comp) +int fit_image_check_comp(const void *fit, int noffset, uint8_t comp) { uint8_t image_comp; - if (fit_image_get_comp (fit, noffset, &image_comp)) + if (fit_image_get_comp(fit, noffset, &image_comp)) return 0; return (comp == image_comp); } @@ -2893,25 +2946,25 @@ int fit_image_check_comp (const void *fit, int noffset, uint8_t comp) * 1, on success * 0, on failure */ -int fit_check_format (const void *fit) +int fit_check_format(const void *fit) { /* mandatory / node 'description' property */ - if (fdt_getprop (fit, 0, FIT_DESC_PROP, NULL) == NULL) { - debug ("Wrong FIT format: no description\n"); + if (fdt_getprop(fit, 0, FIT_DESC_PROP, NULL) == NULL) { + debug("Wrong FIT format: no description\n"); return 0; } #if defined(CONFIG_TIMESTAMP) || defined(CONFIG_CMD_DATE) || defined(USE_HOSTCC) /* mandatory / node 'timestamp' property */ - if (fdt_getprop (fit, 0, FIT_TIMESTAMP_PROP, NULL) == NULL) { - debug ("Wrong FIT format: no timestamp\n"); + if (fdt_getprop(fit, 0, FIT_TIMESTAMP_PROP, NULL) == NULL) { + debug("Wrong FIT format: no timestamp\n"); return 0; } #endif /* mandatory subimages parent '/images' node */ - if (fdt_path_offset (fit, FIT_IMAGES_PATH) < 0) { - debug ("Wrong FIT format: no images parent node\n"); + if (fdt_path_offset(fit, FIT_IMAGES_PATH) < 0) { + debug("Wrong FIT format: no images parent node\n"); return 0; } @@ -2935,50 +2988,53 @@ int fit_check_format (const void *fit) * configuration node offset when found (>=0) * negative number on failure (FDT_ERR_* code) */ -int fit_conf_get_node (const void *fit, const char *conf_uname) +int fit_conf_get_node(const void *fit, const char *conf_uname) { int noffset, confs_noffset; int len; - confs_noffset = fdt_path_offset (fit, FIT_CONFS_PATH); + confs_noffset = fdt_path_offset(fit, FIT_CONFS_PATH); if (confs_noffset < 0) { - debug ("Can't find configurations parent node '%s' (%s)\n", - FIT_CONFS_PATH, fdt_strerror (confs_noffset)); + debug("Can't find configurations parent node '%s' (%s)\n", + FIT_CONFS_PATH, fdt_strerror(confs_noffset)); return confs_noffset; } if (conf_uname == NULL) { /* get configuration unit name from the default property */ - debug ("No configuration specified, trying default...\n"); - conf_uname = (char *)fdt_getprop (fit, confs_noffset, FIT_DEFAULT_PROP, &len); + debug("No configuration specified, trying default...\n"); + conf_uname = (char *)fdt_getprop(fit, confs_noffset, + FIT_DEFAULT_PROP, &len); if (conf_uname == NULL) { - fit_get_debug (fit, confs_noffset, FIT_DEFAULT_PROP, len); + fit_get_debug(fit, confs_noffset, FIT_DEFAULT_PROP, + len); return len; } - debug ("Found default configuration: '%s'\n", conf_uname); + debug("Found default configuration: '%s'\n", conf_uname); } - noffset = fdt_subnode_offset (fit, confs_noffset, conf_uname); + noffset = fdt_subnode_offset(fit, confs_noffset, conf_uname); if (noffset < 0) { - debug ("Can't get node offset for configuration unit name: '%s' (%s)\n", - conf_uname, fdt_strerror (noffset)); + debug("Can't get node offset for configuration unit name: " + "'%s' (%s)\n", + conf_uname, fdt_strerror(noffset)); } return noffset; } -static int __fit_conf_get_prop_node (const void *fit, int noffset, +static int __fit_conf_get_prop_node(const void *fit, int noffset, const char *prop_name) { char *uname; int len; /* get kernel image unit name from configuration kernel property */ - uname = (char *)fdt_getprop (fit, noffset, prop_name, &len); + uname = (char *)fdt_getprop(fit, noffset, prop_name, &len); if (uname == NULL) return len; - return fit_image_get_node (fit, uname); + return fit_image_get_node(fit, uname); } /** @@ -2995,9 +3051,9 @@ static int __fit_conf_get_prop_node (const void *fit, int noffset, * image node offset when found (>=0) * negative number on failure (FDT_ERR_* code) */ -int fit_conf_get_kernel_node (const void *fit, int noffset) +int fit_conf_get_kernel_node(const void *fit, int noffset) { - return __fit_conf_get_prop_node (fit, noffset, FIT_KERNEL_PROP); + return __fit_conf_get_prop_node(fit, noffset, FIT_KERNEL_PROP); } /** @@ -3014,9 +3070,9 @@ int fit_conf_get_kernel_node (const void *fit, int noffset) * image node offset when found (>=0) * negative number on failure (FDT_ERR_* code) */ -int fit_conf_get_ramdisk_node (const void *fit, int noffset) +int fit_conf_get_ramdisk_node(const void *fit, int noffset) { - return __fit_conf_get_prop_node (fit, noffset, FIT_RAMDISK_PROP); + return __fit_conf_get_prop_node(fit, noffset, FIT_RAMDISK_PROP); } /** @@ -3033,9 +3089,9 @@ int fit_conf_get_ramdisk_node (const void *fit, int noffset) * image node offset when found (>=0) * negative number on failure (FDT_ERR_* code) */ -int fit_conf_get_fdt_node (const void *fit, int noffset) +int fit_conf_get_fdt_node(const void *fit, int noffset) { - return __fit_conf_get_prop_node (fit, noffset, FIT_FDT_PROP); + return __fit_conf_get_prop_node(fit, noffset, FIT_FDT_PROP); } /** @@ -3050,35 +3106,35 @@ int fit_conf_get_fdt_node (const void *fit, int noffset) * returns: * no returned results */ -void fit_conf_print (const void *fit, int noffset, const char *p) +void fit_conf_print(const void *fit, int noffset, const char *p) { char *desc; char *uname; int ret; /* Mandatory properties */ - ret = fit_get_desc (fit, noffset, &desc); - printf ("%s Description: ", p); + ret = fit_get_desc(fit, noffset, &desc); + printf("%s Description: ", p); if (ret) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("%s\n", desc); + printf("%s\n", desc); - uname = (char *)fdt_getprop (fit, noffset, FIT_KERNEL_PROP, NULL); - printf ("%s Kernel: ", p); + uname = (char *)fdt_getprop(fit, noffset, FIT_KERNEL_PROP, NULL); + printf("%s Kernel: ", p); if (uname == NULL) - printf ("unavailable\n"); + printf("unavailable\n"); else - printf ("%s\n", uname); + printf("%s\n", uname); /* Optional properties */ - uname = (char *)fdt_getprop (fit, noffset, FIT_RAMDISK_PROP, NULL); + uname = (char *)fdt_getprop(fit, noffset, FIT_RAMDISK_PROP, NULL); if (uname) - printf ("%s Init Ramdisk: %s\n", p, uname); + printf("%s Init Ramdisk: %s\n", p, uname); - uname = (char *)fdt_getprop (fit, noffset, FIT_FDT_PROP, NULL); + uname = (char *)fdt_getprop(fit, noffset, FIT_FDT_PROP, NULL); if (uname) - printf ("%s FDT: %s\n", p, uname); + printf("%s FDT: %s\n", p, uname); } /** @@ -3096,31 +3152,32 @@ void fit_conf_print (const void *fit, int noffset, const char *p) * 0, on failure */ #ifndef USE_HOSTCC -static int fit_check_ramdisk (const void *fit, int rd_noffset, uint8_t arch, int verify) +static int fit_check_ramdisk(const void *fit, int rd_noffset, uint8_t arch, + int verify) { - fit_image_print (fit, rd_noffset, " "); + fit_image_print(fit, rd_noffset, " "); if (verify) { - puts (" Verifying Hash Integrity ... "); - if (!fit_image_check_hashes (fit, rd_noffset)) { - puts ("Bad Data Hash\n"); - show_boot_progress (-125); + puts(" Verifying Hash Integrity ... "); + if (!fit_image_check_hashes(fit, rd_noffset)) { + puts("Bad Data Hash\n"); + show_boot_progress(-125); return 0; } - puts ("OK\n"); + puts("OK\n"); } - show_boot_progress (126); - if (!fit_image_check_os (fit, rd_noffset, IH_OS_LINUX) || - !fit_image_check_arch (fit, rd_noffset, arch) || - !fit_image_check_type (fit, rd_noffset, IH_TYPE_RAMDISK)) { - printf ("No Linux %s Ramdisk Image\n", + show_boot_progress(126); + if (!fit_image_check_os(fit, rd_noffset, IH_OS_LINUX) || + !fit_image_check_arch(fit, rd_noffset, arch) || + !fit_image_check_type(fit, rd_noffset, IH_TYPE_RAMDISK)) { + printf("No Linux %s Ramdisk Image\n", genimg_get_arch_name(arch)); - show_boot_progress (-126); + show_boot_progress(-126); return 0; } - show_boot_progress (127); + show_boot_progress(127); return 1; } #endif /* USE_HOSTCC */ diff --git a/common/lynxkdi.c b/common/lynxkdi.c index b23135b..22ad384 100644 --- a/common/lynxkdi.c +++ b/common/lynxkdi.c @@ -23,45 +23,47 @@ DECLARE_GLOBAL_DATA_PTR; #if defined(CONFIG_MPC8260) || defined(CONFIG_440EP) || defined(CONFIG_440GR) -void lynxkdi_boot (image_header_t *hdr) +void lynxkdi_boot(image_header_t *hdr) { - void (*lynxkdi)(void) = (void(*)(void))image_get_ep (hdr); + void (*lynxkdi)(void) = (void(*)(void))image_get_ep(hdr); lynxos_bootparms_t *parms = (lynxos_bootparms_t *)0x0020; bd_t *kbd; - u32 *psz = (u32 *)(image_get_load (hdr) + 0x0204); + u32 *psz = (u32 *)(image_get_load(hdr) + 0x0204); - memset (parms, 0, sizeof(*parms)); + memset(parms, 0, sizeof(*parms)); kbd = gd->bd; parms->clock_ref = kbd->bi_busfreq; parms->dramsz = kbd->bi_memsize; eth_getenv_enetaddr("ethaddr", parms->ethaddr); - mtspr (SPRN_SPRG2, 0x0020); + mtspr(SPRN_SPRG2, 0x0020); /* Do a simple check for Bluecat so we can pass the * kernel command line parameters. */ - if (le32_to_cpu (*psz) == image_get_data_size (hdr)) { /* FIXME: NOT SURE HERE ! */ + /* FIXME: NOT SURE HERE ! */ + if (le32_to_cpu(*psz) == image_get_data_size(hdr)) { char *args; - char *cmdline = (char *)(image_get_load (hdr) + 0x020c); + char *cmdline = (char *)(image_get_load(hdr) + 0x020c); int len; - printf ("Booting Bluecat KDI ...\n"); - udelay (200*1000); /* Allow serial port to flush */ - if ((args = getenv ("bootargs")) == NULL) + printf("Booting Bluecat KDI ...\n"); + udelay(200*1000); /* Allow serial port to flush */ + if ((args = getenv("bootargs")) == NULL) args = ""; /* Prepend the cmdline */ - len = strlen (args); - if (len && (len + strlen (cmdline) + 2 < (0x0400 - 0x020c))) { - memmove (cmdline + strlen (args) + 1, cmdline, strlen (cmdline)); - strcpy (cmdline, args); + len = strlen(args); + if (len && (len + strlen(cmdline) + 2 < (0x0400 - 0x020c))) { + memmove(cmdline + strlen(args) + 1, cmdline, + strlen(cmdline)); + strcpy(cmdline, args); cmdline[len] = ' '; } } else { - printf ("Booting LynxOS KDI ...\n"); + printf("Booting LynxOS KDI ...\n"); } - lynxkdi (); + lynxkdi(); } #else #error "Lynx KDI support not implemented for configured CPU" diff --git a/include/image.h b/include/image.h index cca1cc5..0d683ad 100644 --- a/include/image.h +++ b/include/image.h @@ -311,15 +311,15 @@ int get_table_entry_id(const table_entry_t *table, */ char *get_table_entry_name(const table_entry_t *table, char *msg, int id); -const char *genimg_get_os_name (uint8_t os); -const char *genimg_get_arch_name (uint8_t arch); -const char *genimg_get_type_name (uint8_t type); -const char *genimg_get_comp_name (uint8_t comp); -int genimg_get_os_id (const char *name); -int genimg_get_arch_id (const char *name); -int genimg_get_type_id (const char *name); -int genimg_get_comp_id (const char *name); -void genimg_print_size (uint32_t size); +const char *genimg_get_os_name(uint8_t os); +const char *genimg_get_arch_name(uint8_t arch); +const char *genimg_get_type_name(uint8_t type); +const char *genimg_get_comp_name(uint8_t comp); +int genimg_get_os_id(const char *name); +int genimg_get_arch_id(const char *name); +int genimg_get_type_id(const char *name); +int genimg_get_comp_id(const char *name); +void genimg_print_size(uint32_t size); #ifndef USE_HOSTCC /* Image format types, returned by _get_format() routine */ @@ -327,72 +327,72 @@ void genimg_print_size (uint32_t size); #define IMAGE_FORMAT_LEGACY 0x01 /* legacy image_header based format */ #define IMAGE_FORMAT_FIT 0x02 /* new, libfdt based format */ -int genimg_get_format (void *img_addr); -int genimg_has_config (bootm_headers_t *images); -ulong genimg_get_image (ulong img_addr); +int genimg_get_format(void *img_addr); +int genimg_has_config(bootm_headers_t *images); +ulong genimg_get_image(ulong img_addr); -int boot_get_ramdisk (int argc, char * const argv[], bootm_headers_t *images, +int boot_get_ramdisk(int argc, char * const argv[], bootm_headers_t *images, uint8_t arch, ulong *rd_start, ulong *rd_end); #ifdef CONFIG_OF_LIBFDT -int boot_get_fdt (int flag, int argc, char * const argv[], bootm_headers_t *images, - char **of_flat_tree, ulong *of_size); +int boot_get_fdt(int flag, int argc, char * const argv[], + bootm_headers_t *images, char **of_flat_tree, ulong *of_size); void boot_fdt_add_mem_rsv_regions(struct lmb *lmb, void *fdt_blob); -int boot_relocate_fdt (struct lmb *lmb, char **of_flat_tree, ulong *of_size); +int boot_relocate_fdt(struct lmb *lmb, char **of_flat_tree, ulong *of_size); #endif #ifdef CONFIG_SYS_BOOT_RAMDISK_HIGH -int boot_ramdisk_high (struct lmb *lmb, ulong rd_data, ulong rd_len, +int boot_ramdisk_high(struct lmb *lmb, ulong rd_data, ulong rd_len, ulong *initrd_start, ulong *initrd_end); #endif /* CONFIG_SYS_BOOT_RAMDISK_HIGH */ #ifdef CONFIG_SYS_BOOT_GET_CMDLINE -int boot_get_cmdline (struct lmb *lmb, ulong *cmd_start, ulong *cmd_end); +int boot_get_cmdline(struct lmb *lmb, ulong *cmd_start, ulong *cmd_end); #endif /* CONFIG_SYS_BOOT_GET_CMDLINE */ #ifdef CONFIG_SYS_BOOT_GET_KBD -int boot_get_kbd (struct lmb *lmb, bd_t **kbd); +int boot_get_kbd(struct lmb *lmb, bd_t **kbd); #endif /* CONFIG_SYS_BOOT_GET_KBD */ #endif /* !USE_HOSTCC */ /*******************************************************************/ /* Legacy format specific code (prefixed with image_) */ /*******************************************************************/ -static inline uint32_t image_get_header_size (void) +static inline uint32_t image_get_header_size(void) { - return (sizeof (image_header_t)); + return (sizeof(image_header_t)); } #define image_get_hdr_l(f) \ static inline uint32_t image_get_##f(const image_header_t *hdr) \ { \ - return uimage_to_cpu (hdr->ih_##f); \ + return uimage_to_cpu(hdr->ih_##f); \ } -image_get_hdr_l (magic) /* image_get_magic */ -image_get_hdr_l (hcrc) /* image_get_hcrc */ -image_get_hdr_l (time) /* image_get_time */ -image_get_hdr_l (size) /* image_get_size */ -image_get_hdr_l (load) /* image_get_load */ -image_get_hdr_l (ep) /* image_get_ep */ -image_get_hdr_l (dcrc) /* image_get_dcrc */ +image_get_hdr_l(magic) /* image_get_magic */ +image_get_hdr_l(hcrc) /* image_get_hcrc */ +image_get_hdr_l(time) /* image_get_time */ +image_get_hdr_l(size) /* image_get_size */ +image_get_hdr_l(load) /* image_get_load */ +image_get_hdr_l(ep) /* image_get_ep */ +image_get_hdr_l(dcrc) /* image_get_dcrc */ #define image_get_hdr_b(f) \ static inline uint8_t image_get_##f(const image_header_t *hdr) \ { \ return hdr->ih_##f; \ } -image_get_hdr_b (os) /* image_get_os */ -image_get_hdr_b (arch) /* image_get_arch */ -image_get_hdr_b (type) /* image_get_type */ -image_get_hdr_b (comp) /* image_get_comp */ +image_get_hdr_b(os) /* image_get_os */ +image_get_hdr_b(arch) /* image_get_arch */ +image_get_hdr_b(type) /* image_get_type */ +image_get_hdr_b(comp) /* image_get_comp */ -static inline char *image_get_name (const image_header_t *hdr) +static inline char *image_get_name(const image_header_t *hdr) { return (char *)hdr->ih_name; } -static inline uint32_t image_get_data_size (const image_header_t *hdr) +static inline uint32_t image_get_data_size(const image_header_t *hdr) { - return image_get_size (hdr); + return image_get_size(hdr); } /** @@ -406,83 +406,83 @@ static inline uint32_t image_get_data_size (const image_header_t *hdr) * returns: * image payload data start address */ -static inline ulong image_get_data (const image_header_t *hdr) +static inline ulong image_get_data(const image_header_t *hdr) { - return ((ulong)hdr + image_get_header_size ()); + return ((ulong)hdr + image_get_header_size()); } -static inline uint32_t image_get_image_size (const image_header_t *hdr) +static inline uint32_t image_get_image_size(const image_header_t *hdr) { - return (image_get_size (hdr) + image_get_header_size ()); + return (image_get_size(hdr) + image_get_header_size()); } -static inline ulong image_get_image_end (const image_header_t *hdr) +static inline ulong image_get_image_end(const image_header_t *hdr) { - return ((ulong)hdr + image_get_image_size (hdr)); + return ((ulong)hdr + image_get_image_size(hdr)); } #define image_set_hdr_l(f) \ static inline void image_set_##f(image_header_t *hdr, uint32_t val) \ { \ - hdr->ih_##f = cpu_to_uimage (val); \ + hdr->ih_##f = cpu_to_uimage(val); \ } -image_set_hdr_l (magic) /* image_set_magic */ -image_set_hdr_l (hcrc) /* image_set_hcrc */ -image_set_hdr_l (time) /* image_set_time */ -image_set_hdr_l (size) /* image_set_size */ -image_set_hdr_l (load) /* image_set_load */ -image_set_hdr_l (ep) /* image_set_ep */ -image_set_hdr_l (dcrc) /* image_set_dcrc */ +image_set_hdr_l(magic) /* image_set_magic */ +image_set_hdr_l(hcrc) /* image_set_hcrc */ +image_set_hdr_l(time) /* image_set_time */ +image_set_hdr_l(size) /* image_set_size */ +image_set_hdr_l(load) /* image_set_load */ +image_set_hdr_l(ep) /* image_set_ep */ +image_set_hdr_l(dcrc) /* image_set_dcrc */ #define image_set_hdr_b(f) \ static inline void image_set_##f(image_header_t *hdr, uint8_t val) \ { \ hdr->ih_##f = val; \ } -image_set_hdr_b (os) /* image_set_os */ -image_set_hdr_b (arch) /* image_set_arch */ -image_set_hdr_b (type) /* image_set_type */ -image_set_hdr_b (comp) /* image_set_comp */ +image_set_hdr_b(os) /* image_set_os */ +image_set_hdr_b(arch) /* image_set_arch */ +image_set_hdr_b(type) /* image_set_type */ +image_set_hdr_b(comp) /* image_set_comp */ -static inline void image_set_name (image_header_t *hdr, const char *name) +static inline void image_set_name(image_header_t *hdr, const char *name) { - strncpy (image_get_name (hdr), name, IH_NMLEN); + strncpy(image_get_name(hdr), name, IH_NMLEN); } -int image_check_hcrc (const image_header_t *hdr); -int image_check_dcrc (const image_header_t *hdr); +int image_check_hcrc(const image_header_t *hdr); +int image_check_dcrc(const image_header_t *hdr); #ifndef USE_HOSTCC -int getenv_yesno (char *var); +int getenv_yesno(char *var); ulong getenv_bootm_low(void); phys_size_t getenv_bootm_size(void); phys_size_t getenv_bootm_mapsize(void); -void memmove_wd (void *to, void *from, size_t len, ulong chunksz); +void memmove_wd(void *to, void *from, size_t len, ulong chunksz); #endif -static inline int image_check_magic (const image_header_t *hdr) +static inline int image_check_magic(const image_header_t *hdr) { - return (image_get_magic (hdr) == IH_MAGIC); + return (image_get_magic(hdr) == IH_MAGIC); } -static inline int image_check_type (const image_header_t *hdr, uint8_t type) +static inline int image_check_type(const image_header_t *hdr, uint8_t type) { - return (image_get_type (hdr) == type); + return (image_get_type(hdr) == type); } -static inline int image_check_arch (const image_header_t *hdr, uint8_t arch) +static inline int image_check_arch(const image_header_t *hdr, uint8_t arch) { - return (image_get_arch (hdr) == arch); + return (image_get_arch(hdr) == arch); } -static inline int image_check_os (const image_header_t *hdr, uint8_t os) +static inline int image_check_os(const image_header_t *hdr, uint8_t os) { - return (image_get_os (hdr) == os); + return (image_get_os(hdr) == os); } -ulong image_multi_count (const image_header_t *hdr); -void image_multi_getimg (const image_header_t *hdr, ulong idx, +ulong image_multi_count(const image_header_t *hdr); +void image_multi_getimg(const image_header_t *hdr, ulong idx, ulong *data, ulong *len); -void image_print_contents (const void *hdr); +void image_print_contents(const void *hdr); #ifndef USE_HOSTCC -static inline int image_check_target_arch (const image_header_t *hdr) +static inline int image_check_target_arch(const image_header_t *hdr) { #ifndef IH_ARCH_DEFAULT # error "please define IH_ARCH_DEFAULT in your arch asm/u-boot.h" @@ -524,14 +524,14 @@ static inline int image_check_target_arch (const image_header_t *hdr) #define FIT_MAX_HASH_LEN 20 /* max(crc32_len(4), sha1_len(20)) */ /* cmdline argument format parsing */ -inline int fit_parse_conf (const char *spec, ulong addr_curr, +inline int fit_parse_conf(const char *spec, ulong addr_curr, ulong *addr, const char **conf_name); -inline int fit_parse_subimage (const char *spec, ulong addr_curr, +inline int fit_parse_subimage(const char *spec, ulong addr_curr, ulong *addr, const char **image_name); -void fit_print_contents (const void *fit); -void fit_image_print (const void *fit, int noffset, const char *p); -void fit_image_print_hash (const void *fit, int noffset, const char *p); +void fit_print_contents(const void *fit); +void fit_image_print(const void *fit, int noffset, const char *p); +void fit_image_print_hash(const void *fit, int noffset, const char *p); /** * fit_get_end - get FIT image size @@ -540,9 +540,9 @@ void fit_image_print_hash (const void *fit, int noffset, const char *p); * returns: * size of the FIT image (blob) in memory */ -static inline ulong fit_get_size (const void *fit) +static inline ulong fit_get_size(const void *fit) { - return fdt_totalsize (fit); + return fdt_totalsize(fit); } /** @@ -552,9 +552,9 @@ static inline ulong fit_get_size (const void *fit) * returns: * end address of the FIT image (blob) in memory */ -static inline ulong fit_get_end (const void *fit) +static inline ulong fit_get_end(const void *fit) { - return (ulong)fit + fdt_totalsize (fit); + return (ulong)fit + fdt_totalsize(fit); } /** @@ -565,63 +565,63 @@ static inline ulong fit_get_end (const void *fit) * NULL, on error * pointer to node name, on success */ -static inline const char *fit_get_name (const void *fit_hdr, +static inline const char *fit_get_name(const void *fit_hdr, int noffset, int *len) { - return fdt_get_name (fit_hdr, noffset, len); + return fdt_get_name(fit_hdr, noffset, len); } -int fit_get_desc (const void *fit, int noffset, char **desc); -int fit_get_timestamp (const void *fit, int noffset, time_t *timestamp); - -int fit_image_get_node (const void *fit, const char *image_uname); -int fit_image_get_os (const void *fit, int noffset, uint8_t *os); -int fit_image_get_arch (const void *fit, int noffset, uint8_t *arch); -int fit_image_get_type (const void *fit, int noffset, uint8_t *type); -int fit_image_get_comp (const void *fit, int noffset, uint8_t *comp); -int fit_image_get_load (const void *fit, int noffset, ulong *load); -int fit_image_get_entry (const void *fit, int noffset, ulong *entry); -int fit_image_get_data (const void *fit, int noffset, +int fit_get_desc(const void *fit, int noffset, char **desc); +int fit_get_timestamp(const void *fit, int noffset, time_t *timestamp); + +int fit_image_get_node(const void *fit, const char *image_uname); +int fit_image_get_os(const void *fit, int noffset, uint8_t *os); +int fit_image_get_arch(const void *fit, int noffset, uint8_t *arch); +int fit_image_get_type(const void *fit, int noffset, uint8_t *type); +int fit_image_get_comp(const void *fit, int noffset, uint8_t *comp); +int fit_image_get_load(const void *fit, int noffset, ulong *load); +int fit_image_get_entry(const void *fit, int noffset, ulong *entry); +int fit_image_get_data(const void *fit, int noffset, const void **data, size_t *size); -int fit_image_hash_get_algo (const void *fit, int noffset, char **algo); -int fit_image_hash_get_value (const void *fit, int noffset, uint8_t **value, +int fit_image_hash_get_algo(const void *fit, int noffset, char **algo); +int fit_image_hash_get_value(const void *fit, int noffset, uint8_t **value, int *value_len); -int fit_set_timestamp (void *fit, int noffset, time_t timestamp); -int fit_set_hashes (void *fit); -int fit_image_set_hashes (void *fit, int image_noffset); -int fit_image_hash_set_value (void *fit, int noffset, uint8_t *value, +int fit_set_timestamp(void *fit, int noffset, time_t timestamp); +int fit_set_hashes(void *fit); +int fit_image_set_hashes(void *fit, int image_noffset); +int fit_image_hash_set_value(void *fit, int noffset, uint8_t *value, int value_len); -int fit_image_check_hashes (const void *fit, int noffset); -int fit_all_image_check_hashes (const void *fit); -int fit_image_check_os (const void *fit, int noffset, uint8_t os); -int fit_image_check_arch (const void *fit, int noffset, uint8_t arch); -int fit_image_check_type (const void *fit, int noffset, uint8_t type); -int fit_image_check_comp (const void *fit, int noffset, uint8_t comp); -int fit_check_format (const void *fit); +int fit_image_check_hashes(const void *fit, int noffset); +int fit_all_image_check_hashes(const void *fit); +int fit_image_check_os(const void *fit, int noffset, uint8_t os); +int fit_image_check_arch(const void *fit, int noffset, uint8_t arch); +int fit_image_check_type(const void *fit, int noffset, uint8_t type); +int fit_image_check_comp(const void *fit, int noffset, uint8_t comp); +int fit_check_format(const void *fit); -int fit_conf_get_node (const void *fit, const char *conf_uname); -int fit_conf_get_kernel_node (const void *fit, int noffset); -int fit_conf_get_ramdisk_node (const void *fit, int noffset); -int fit_conf_get_fdt_node (const void *fit, int noffset); +int fit_conf_get_node(const void *fit, const char *conf_uname); +int fit_conf_get_kernel_node(const void *fit, int noffset); +int fit_conf_get_ramdisk_node(const void *fit, int noffset); +int fit_conf_get_fdt_node(const void *fit, int noffset); -void fit_conf_print (const void *fit, int noffset, const char *p); +void fit_conf_print(const void *fit, int noffset, const char *p); #ifndef USE_HOSTCC -static inline int fit_image_check_target_arch (const void *fdt, int node) +static inline int fit_image_check_target_arch(const void *fdt, int node) { return !fit_image_check_arch(fdt, node, IH_ARCH_DEFAULT); } #endif /* USE_HOSTCC */ #ifdef CONFIG_FIT_VERBOSE -#define fit_unsupported(msg) printf ("! %s:%d " \ +#define fit_unsupported(msg) printf("! %s:%d " \ "FIT images not supported for '%s'\n", \ __FILE__, __LINE__, (msg)) -#define fit_unsupported_reset(msg) printf ("! %s:%d " \ +#define fit_unsupported_reset(msg) printf("! %s:%d " \ "FIT images not supported for '%s' " \ "- must reset board to recover!\n", \ __FILE__, __LINE__, (msg)) diff --git a/tools/default_image.c b/tools/default_image.c index f5bad47..6ea3b46 100644 --- a/tools/default_image.c +++ b/tools/default_image.c @@ -33,7 +33,7 @@ static image_header_t header; -static int image_check_image_types (uint8_t type) +static int image_check_image_types(uint8_t type) { if ((type > IH_TYPE_INVALID) && (type < IH_TYPE_FLATDT)) return EXIT_SUCCESS; @@ -41,14 +41,14 @@ static int image_check_image_types (uint8_t type) return EXIT_FAILURE; } -static int image_check_params (struct mkimage_params *params) +static int image_check_params(struct mkimage_params *params) { return ((params->dflag && (params->fflag || params->lflag)) || (params->fflag && (params->dflag || params->lflag)) || (params->lflag && (params->dflag || params->fflag))); } -static int image_verify_header (unsigned char *ptr, int image_size, +static int image_verify_header(unsigned char *ptr, int image_size, struct mkimage_params *params) { uint32_t len; @@ -62,10 +62,10 @@ static int image_verify_header (unsigned char *ptr, int image_size, * checksum field for checking - this can't be done * on the PROT_READ mapped data. */ - memcpy (hdr, ptr, sizeof(image_header_t)); + memcpy(hdr, ptr, sizeof(image_header_t)); if (be32_to_cpu(hdr->ih_magic) != IH_MAGIC) { - fprintf (stderr, + fprintf(stderr, "%s: Bad Magic Number: \"%s\" is no valid image\n", params->cmdname, params->imagefile); return -FDT_ERR_BADMAGIC; @@ -77,8 +77,8 @@ static int image_verify_header (unsigned char *ptr, int image_size, checksum = be32_to_cpu(hdr->ih_hcrc); hdr->ih_hcrc = cpu_to_be32(0); /* clear for re-calculation */ - if (crc32 (0, data, len) != checksum) { - fprintf (stderr, + if (crc32(0, data, len) != checksum) { + fprintf(stderr, "%s: ERROR: \"%s\" has bad header checksum!\n", params->cmdname, params->imagefile); return -FDT_ERR_BADSTATE; @@ -88,8 +88,8 @@ static int image_verify_header (unsigned char *ptr, int image_size, len = image_size - sizeof(image_header_t) ; checksum = be32_to_cpu(hdr->ih_dcrc); - if (crc32 (0, data, len) != checksum) { - fprintf (stderr, + if (crc32(0, data, len) != checksum) { + fprintf(stderr, "%s: ERROR: \"%s\" has corrupted data!\n", params->cmdname, params->imagefile); return -FDT_ERR_BADSTRUCTURE; @@ -97,36 +97,36 @@ static int image_verify_header (unsigned char *ptr, int image_size, return 0; } -static void image_set_header (void *ptr, struct stat *sbuf, int ifd, +static void image_set_header(void *ptr, struct stat *sbuf, int ifd, struct mkimage_params *params) { uint32_t checksum; image_header_t * hdr = (image_header_t *)ptr; - checksum = crc32 (0, + checksum = crc32(0, (const unsigned char *)(ptr + sizeof(image_header_t)), sbuf->st_size - sizeof(image_header_t)); /* Build new header */ - image_set_magic (hdr, IH_MAGIC); - image_set_time (hdr, sbuf->st_mtime); - image_set_size (hdr, sbuf->st_size - sizeof(image_header_t)); - image_set_load (hdr, params->addr); - image_set_ep (hdr, params->ep); - image_set_dcrc (hdr, checksum); - image_set_os (hdr, params->os); - image_set_arch (hdr, params->arch); - image_set_type (hdr, params->type); - image_set_comp (hdr, params->comp); - - image_set_name (hdr, params->imagename); - - checksum = crc32 (0, (const unsigned char *)hdr, + image_set_magic(hdr, IH_MAGIC); + image_set_time(hdr, sbuf->st_mtime); + image_set_size(hdr, sbuf->st_size - sizeof(image_header_t)); + image_set_load(hdr, params->addr); + image_set_ep(hdr, params->ep); + image_set_dcrc(hdr, checksum); + image_set_os(hdr, params->os); + image_set_arch(hdr, params->arch); + image_set_type(hdr, params->type); + image_set_comp(hdr, params->comp); + + image_set_name(hdr, params->imagename); + + checksum = crc32(0, (const unsigned char *)hdr, sizeof(image_header_t)); - image_set_hcrc (hdr, checksum); + image_set_hcrc(hdr, checksum); } /* @@ -143,7 +143,7 @@ static struct image_type_params defimage_params = { .check_params = image_check_params, }; -void init_default_image_type (void) +void init_default_image_type(void) { - mkimage_register (&defimage_params); + mkimage_register(&defimage_params); }