Hello Michal,

Am Mon, Feb 03, 2025 at 10:28:15AM +0100 schrieb Michal Simek:
> do_fpga_loads() is not called from anywhere else that's why make it static.
> 
> Signed-off-by: Michal Simek <michal.si...@amd.com>
> ---
> 
>  cmd/fpga.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/cmd/fpga.c b/cmd/fpga.c
> index 212f421739fa..9dc7b63db5d1 100644
> --- a/cmd/fpga.c
> +++ b/cmd/fpga.c
> @@ -67,7 +67,8 @@ static int do_fpga_check_params(long *dev, long *fpga_data, 
> size_t *data_size,
>  }
>  
>  #if defined(CONFIG_CMD_FPGA_LOAD_SECURE)
> -int do_fpga_loads(struct cmd_tbl *cmdtp, int flag, int argc, char *const 
> argv[])
> +static int do_fpga_loads(struct cmd_tbl *cmdtp, int flag, int argc,
> +                      char *const argv[])
>  {
>       size_t data_size = 0;
>       long fpga_data, dev;

Reviewed-by: Alexander Dahl <a...@thorsis.com>

Build tested for our custom boards.

Greets
Alex

Reply via email to