diff options
author | Sean Anderson <sean.anderson@seco.com> | 2022-08-16 18:16:05 +0300 |
---|---|---|
committer | Peng Fan <peng.fan@nxp.com> | 2022-09-07 08:40:47 +0300 |
commit | 7b42bde075c9cec85aa200d2e093d3b2b5641d94 (patch) | |
tree | b5d3a63d3ee14262d923acc9cf817cd2334f3e67 | |
parent | bdbdaede671c5ad7b71737bd345e03eb7e99266d (diff) | |
download | u-boot-7b42bde075c9cec85aa200d2e093d3b2b5641d94.tar.xz |
cmd: fpga: Convert to use fit_get_data_node
This converts the FIT loading process of the fpga command to use
fit_get_data_node.
Signed-off-by: Sean Anderson <sean.anderson@seco.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
-rw-r--r-- | cmd/fpga.c | 24 |
1 files changed, 6 insertions, 18 deletions
diff --git a/cmd/fpga.c b/cmd/fpga.c index c4651dd403..9cf7651d8c 100644 --- a/cmd/fpga.c +++ b/cmd/fpga.c @@ -322,7 +322,7 @@ static int do_fpga_loadmk(struct cmd_tbl *cmdtp, int flag, int argc, case IMAGE_FORMAT_FIT: { const void *fit_hdr = (const void *)fpga_data; - int noffset; + int err; const void *fit_data; if (!fit_uname) { @@ -335,23 +335,11 @@ static int do_fpga_loadmk(struct cmd_tbl *cmdtp, int flag, int argc, return CMD_RET_FAILURE; } - /* get fpga component image node offset */ - noffset = fit_image_get_node(fit_hdr, fit_uname); - if (noffset < 0) { - printf("Can't find '%s' FIT subimage\n", fit_uname); - return CMD_RET_FAILURE; - } - - /* verify integrity */ - if (!fit_image_verify(fit_hdr, noffset)) { - puts("Bad Data Hash\n"); - return CMD_RET_FAILURE; - } - - /* get fpga subimage/external data address and length */ - if (fit_image_get_data_and_size(fit_hdr, noffset, - &fit_data, &data_size)) { - puts("Fpga subimage data not found\n"); + err = fit_get_data_node(fit_hdr, fit_uname, &fit_data, + &data_size); + if (err) { + printf("Could not load '%s' subimage (err %d)\n", + fit_uname, err); return CMD_RET_FAILURE; } |