spl: fit: Warn if FIT contains "fpga" property in config node
authorAlexandru Gagniuc <mr.nuke.me@gmail.com>
Mon, 29 Mar 2021 17:05:13 +0000 (12:05 -0500)
committerTom Rini <trini@konsulko.com>
Wed, 14 Apr 2021 19:23:01 +0000 (15:23 -0400)
Commit 4afc4f37c70e ("doc: FIT image: Clarify format and simplify
syntax") requires that FPGA images be referenced through the
"loadables" in the config node. This means that "fpga" properties in
config nodes are deprecated.

Given that there are likely FIT images which use "fpga", let's not
break those right away. Print a warning message that such use is
deprecated, and give users a couple of releases to update their

Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
common/spl/spl_fit.c

index 514ec91..496fabc 100644 (file)
@@ -529,6 +529,12 @@ __weak bool spl_load_simple_fit_skip_processing(void)
        return false;
 }
 
+static void warn_deprecated(const char *msg)
+{
+       printf("DEPRECATED: %s\n", msg);
+       printf("\tSee doc/uImage.FIT/source_file_format.txt\n");
+}
+
 static int spl_fit_upload_fpga(struct spl_fit_info *ctx, int node,
                               struct spl_image_info *fpga_image)
 {
@@ -562,6 +568,8 @@ static int spl_fit_load_fpga(struct spl_fit_info *ctx,
        if (node < 0)
                return node;
 
+       warn_deprecated("'fpga' property in config node. Use 'loadables'");
+
        /* Load the image and set up the fpga_image structure */
        ret = spl_load_fit_image(info, sector, ctx, node, &fpga_image);
        if (ret) {