On 26 July 2018 at 08:16, Michal Simek <michal.si...@xilinx.com> wrote:
> The patch applied in 2010
> "cmd_fpga: cleanup help and check parameters"
> (sha1: a790b5b2326be9d7c9ad9e3d9b51a8bfabc62d07"
> was adding this checking
>
> + if (dev == FPGA_INVALID_DEVICE) {
> + puts("FPGA device not specified\n");
> + op = FPGA_NONE;
> + }
>
> which simply broke one command flow which was
> setenv fpga <dev>
> fpga loadmk <addr> // legacy image
> fpga loadmk <addr>:<fit_image_name> //fit image
>
> Also this sequence for FIT image is completely broken
> setenv fpga <dev>
> setenv fpgadata <addr>:<fit_image_name>
> fpga loadmk
> (Note: For legacy images this is working fine).
>
> even from code I don't think this has ever worked properly
> for fit image (dev = FPGA_INVALID_DEVICE should be rejected
> by fpga core). Fit image support was in 2008 added by:
> "[new uImage] Add new uImage fromat support to fpga command"
> (sha1: c28c4d193dbfb20b2dd3a5447640fd6de7fd0720)
>
> Just a summary of these facts that none found this for pretty long time
> it shouldn't be a problem to remove this flow (without fpga dev)
> completely to simplify the code.
>
> Signed-off-by: Michal Simek <michal.si...@xilinx.com>
> ---
>
> Changes in v1: None
>
> I am rewriting cmd/fpga.c file to use u-boot subcommand and I found
> that these flow are not working.
>
> ---
> cmd/fpga.c | 25 -------------------------
> 1 file changed, 25 deletions(-)
Reviewed-by: Simon Glass <s...@chromium.org>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
