image-fit: Don't display an error in fit_set_timestamp()
authorSimon Glass <sjg@chromium.org>
Sun, 1 May 2016 19:55:37 +0000 (13:55 -0600)
committerTom Rini <trini@konsulko.com>
Mon, 23 May 2016 15:50:18 +0000 (11:50 -0400)
This function returns an error code and its caller may be able to fix the
error. For example fit_handle_file() expands the device tree to fit if there
is a lack of space.

In this case the caller does not want an error displayed. It is confusing,
since it suggests that something is wrong, when it fact everything is fine.
Drop the error.

Signed-off-by: Simon Glass <sjg@chromium.org>
common/image-fit.c

index 25f8a1183d58de323bc4a66af34683db48471263..c86b7c6b1182e42e3febf94caaa3604628f809c6 100644 (file)
@@ -886,9 +886,9 @@ int fit_set_timestamp(void *fit, int noffset, time_t timestamp)
        ret = fdt_setprop(fit, noffset, FIT_TIMESTAMP_PROP, &t,
                                sizeof(uint32_t));
        if (ret) {
-               printf("Can't set '%s' property for '%s' node (%s)\n",
-                      FIT_TIMESTAMP_PROP, fit_get_name(fit, noffset, NULL),
-                      fdt_strerror(ret));
+               debug("Can't set '%s' property for '%s' node (%s)\n",
+                     FIT_TIMESTAMP_PROP, fit_get_name(fit, noffset, NULL),
+                     fdt_strerror(ret));
                return ret == -FDT_ERR_NOSPACE ? -ENOSPC : -1;
        }