"john.liuli" <john.li...@huawei.com> writes:
> From: Li Liu <john.li...@huawei.com>
>
> Signed-off-by: Li Liu <john.li...@huawei.com>
> ---
> 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);
> 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;
> }
A welcome improvement. Further improvement would be using
error_report().
