[U-Boot] [u-boot-release] [PATCH] driver: misc: debug server: Update Error message

Sharma Bhupesh bhupesh.sharma at freescale.com
Mon Aug 10 08:50:17 CEST 2015


Hi Prabhakar,

Thanks for the patch.

A few nitpicks. Otherwise:

Reviewed-by: Bhupesh Sharma <bhupesh.sharma at freescale.com>


> From: u-boot-release-bounces at linux.freescale.net [mailto:u-boot-release-
> bounces at linux.freescale.net] On Behalf Of Prabhakar Kushwaha
> Sent: Sunday, August 09, 2015 8:53 AM
> 
> Apped "debug server image" in error message to make more informative.

  ^^^^
Typo? I guess you meant Append here.

> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
> ---
>  drivers/misc/fsl_debug_server.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/misc/fsl_debug_server.c
> b/drivers/misc/fsl_debug_server.c index 44cd9b9..318626d 100644
> --- a/drivers/misc/fsl_debug_server.c
> +++ b/drivers/misc/fsl_debug_server.c
> @@ -60,29 +60,29 @@ int debug_server_parse_firmware_fit_image(const void
> **raw_image_addr,
>  	/* Check if Image is in FIT format */
>  	format = genimg_get_format(fit_hdr);
>  	if (format != IMAGE_FORMAT_FIT) {
> -		printf("Error! Not a FIT image\n");
> +		printf("Debug Server Image: Not a FIT image\n");

Can we change it to something like this instead (as similarly on the changes that follow):

Debug Server FW: Not a FIT image

>  		goto out_error;
>  	}
> 
>  	if (!fit_check_format(fit_hdr)) {
> -		printf("Error! Bad FIT image format\n");
> +		printf("Debug Server Image: Bad FIT image format\n");
>  		goto out_error;
>  	}
> 
>  	node_offset = fit_image_get_node(fit_hdr, uname);
>  	if (node_offset < 0) {
> -		printf("Error! Can not find %s subimage\n", uname);
> +		printf("Debug Server Image:Can not find %s subimage\n",
> uname);
>  		goto out_error;
>  	}
> 
>  	/* Verify Debug Server firmware image */
>  	if (!fit_image_verify(fit_hdr, node_offset)) {
> -		printf("Error! Bad Debug Server firmware hash");
> +		printf("Debug Server Image: Bad Debug Server firmware hash");
>  		goto out_error;
>  	}
> 
>  	if (fit_get_desc(fit_hdr, node_offset, &desc) < 0) {
> -		printf("Error! Failed to get Debug Server fw description");
> +		printf("Debug Server Image: Failed to get FW description");
>  		goto out_error;
>  	}
> 
> --
> 1.9.1
> 



More information about the U-Boot mailing list