[U-Boot] [PATCH] vxworks: Make loading vxworks less verbose

[U-Boot] [PATCH] vxworks: Make loading vxworks less verbose

[Niklaus Giger]

Newer vxWorks 6.x images have over 20 different C++ segments.
This fills up the output with rarely used information.

Signed-off-by: Niklaus Giger <niklaus.giger@netstal.com>
---
 common/cmd_elf.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/common/cmd_elf.c b/common/cmd_elf.c
index 63f6fe7..bf7dd63 100644
--- a/common/cmd_elf.c
+++ b/common/cmd_elf.c
@@ -286,6 +286,7 @@ unsigned long load_elf_image (unsigned long addr)
 			continue;
 		}
 
+#ifdef DEBUG
 		if (strtab) {
 			debug("%sing %s @ 0x%08lx (%ld bytes)\n",
 				(shdr->sh_type == SHT_NOBITS) ?
@@ -294,6 +295,7 @@ unsigned long load_elf_image (unsigned long addr)
 				(unsigned long) shdr->sh_addr,
 				(long) shdr->sh_size);
 		}
+#endif
 
 		if (shdr->sh_type == SHT_NOBITS) {
 			memset ((void *)shdr->sh_addr, 0, shdr->sh_size);
-- 
1.6.3.3

[U-Boot] [PATCH] vxworks: Make loading vxworks less verbose

[Niklaus Giger]

Please ignore this patch. Came in by mistake. Functionality already
merged by commit 3c972849f2becbf19c13a24f090d293f37ecf616.

Sorry for the noise.

Best regards

Niklaus

[U-Boot] [PATCH] vxworks: Make loading vxworks less verbose

[Tom]

Niklaus Giger wrote:
> Please ignore this patch. Came in by mistake. Functionality already
> merged by commit 3c972849f2becbf19c13a24f090d293f37ecf616.
> 
> Sorry for the noise.
> 
Please ignore my response too :)
Tom

> Best regards
> 
> Niklaus
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] vxworks: Make loading vxworks less verbose

[Tom]

Niklaus Giger wrote:
> Newer vxWorks 6.x images have over 20 different C++ segments.
> This fills up the output with rarely used information.
> 
> Signed-off-by: Niklaus Giger <niklaus.giger@netstal.com>
> ---
>  common/cmd_elf.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/common/cmd_elf.c b/common/cmd_elf.c
> index 63f6fe7..bf7dd63 100644
> --- a/common/cmd_elf.c
> +++ b/common/cmd_elf.c
> @@ -286,6 +286,7 @@ unsigned long load_elf_image (unsigned long addr)
>  			continue;
>  		}
>  
> +#ifdef DEBUG
>  		if (strtab) {
>  			debug("%sing %s @ 0x%08lx (%ld bytes)\n",
>  				(shdr->sh_type == SHT_NOBITS) ?
> @@ -294,6 +295,7 @@ unsigned long load_elf_image (unsigned long addr)
>  				(unsigned long) shdr->sh_addr,
>  				(long) shdr->sh_size);
>  		}
> +#endif

I am not sure this will do anything.
 From the decl of 'debug'

#ifdef	DEBUG
#define debug(fmt,args...)	printf (fmt ,##args)
#define debugX(level,fmt,args...) if (DEBUG>=level) printf(fmt,##args);
#else
#define debug(fmt,args...)
#define debugX(level,fmt,args...)
#endif	/* DEBUG */

debug is already hidden by the DEBUG define.

Maybe a better choice would be to switch to debugX

Tom

>  
>  		if (shdr->sh_type == SHT_NOBITS) {
>  			memset ((void *)shdr->sh_addr, 0, shdr->sh_size);