Re: [Qemu-devel] [PATCH] device_tree.c: redirect load_device_tree err message to stderr

[Peter Crosthwaite]

On Mon, Aug 25, 2014 at 6:44 PM, john.liuli <address@hidden> wrote:
> From: Li Liu <address@hidden>
>
> Signed-off-by: Li Liu <address@hidden>
> ---
>  device_tree.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/device_tree.c b/device_tree.c
> index ca83504..ccdb039 100644
> --- a/device_tree.c
> +++ b/device_tree.c
> @@ -79,7 +79,7 @@ void *load_device_tree(const char *filename_path, int 
> *sizep)
>      *sizep = 0;
>      dt_size = get_image_size(filename_path);
>      if (dt_size < 0) {
> -        printf("Unable to get size of device tree file '%s'\n",
> +        fprintf(stderr, "Unable to get size of device tree file '%s'\n",
>              filename_path);
>          goto fail;
>      }
> @@ -92,20 +92,20 @@ void *load_device_tree(const char *filename_path, int 
> *sizep)
>
>      dt_file_load_size = load_image(filename_path, fdt);
>      if (dt_file_load_size < 0) {
> -        printf("Unable to open device tree file '%s'\n",
> +        fprintf(stderr, "Unable to open device tree file '%s'\n",
>                 filename_path);

Indent this line by one more space to keep continued fn args aligned
to the "(". The other two are currently broken wrt to this issue, but
you could fix them too.

Otherwise:

Reviewed-by: Peter Crosthwaite <address@hidden>

I had some code somewhere that coverts this API to use Error **'s but
your patch is definitely a step in the right direction, while (and if)
that happens. I suggest cc'ing qemu-trivial for a merge.

Regards,
peter

>          goto fail;
>      }
>
>      ret = fdt_open_into(fdt, fdt, dt_size);
>      if (ret) {
> -        printf("Unable to copy device tree in memory\n");
> +        fprintf(stderr, "Unable to copy device tree in memory\n");
>          goto fail;
>      }
>
>      /* Check sanity of device tree */
>      if (fdt_check_header(fdt)) {
> -        printf ("Device tree file loaded into memory is invalid: %s\n",
> +        fprintf(stderr, "Device tree file loaded into memory is invalid: 
> %s\n",
>              filename_path);
>          goto fail;
>      }
> --
> 1.7.9.5
>
>
>