image: Drop #ifdefs for fit_print_contents()
authorSimon Glass <sjg@chromium.org>
Sun, 26 Sep 2021 01:43:34 +0000 (19:43 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 8 Oct 2021 19:53:27 +0000 (15:53 -0400)
Use a simple return to drop the unwanted code.

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

index f44f552..1e4099d 100644 (file)
@@ -170,7 +170,6 @@ int fit_get_subimage_count(const void *fit, int images_noffset)
        return count;
 }
 
-#if CONFIG_IS_ENABLED(FIT_PRINT)
 /**
  * fit_image_print_data() - prints out the hash node details
  * @fit: pointer to the FIT format image header
@@ -380,6 +379,9 @@ void fit_print_contents(const void *fit)
        const char *p;
        time_t timestamp;
 
+       if (!CONFIG_IS_ENABLED(FIT_PRINT))
+               return;
+
        /* Indent string is defined in header image.h */
        p = IMAGE_INDENT_STRING;
 
@@ -482,6 +484,9 @@ void fit_image_print(const void *fit, int image_noffset, const char *p)
        int ndepth;
        int ret;
 
+       if (!CONFIG_IS_ENABLED(FIT_PRINT))
+               return;
+
        /* Mandatory properties */
        ret = fit_get_desc(fit, image_noffset, &desc);
        printf("%s  Description:  ", p);
@@ -575,10 +580,6 @@ void fit_image_print(const void *fit, int image_noffset, const char *p)
                }
        }
 }
-#else
-void fit_print_contents(const void *fit) { }
-void fit_image_print(const void *fit, int image_noffset, const char *p) { }
-#endif /* CONFIG_IS_ENABLED(FIT_PRINT) */
 
 /**
  * fit_get_desc - get node description property