Message ID | 1439090571-12069-1-git-send-email-prabhakar@freescale.com |
---|---|
State | Superseded |
Delegated to: | York Sun |
Headers | show |
Hi Prabhakar, Thanks for the patch. A few nitpicks. Otherwise: Reviewed-by: Bhupesh Sharma <bhupesh.sharma@freescale.com> > From: u-boot-release-bounces@linux.freescale.net [mailto:u-boot-release- > bounces@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@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 >
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"); 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; }
Apped "debug server image" in error message to make more informative. Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com> --- drivers/misc/fsl_debug_server.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)