diff mbox series

[U-Boot] apalis_imx6: add board_fit_config_name_match to support FIT in SPL

Message ID 20190902212327.8178-1-ricardo@foundries.io
State Accepted
Commit 6f8ef051c5dde7b6daeaf1a542c266d608fac783
Delegated to: Stefano Babic
Headers show
Series [U-Boot] apalis_imx6: add board_fit_config_name_match to support FIT in SPL | expand

Commit Message

Ricardo Salveti Sept. 2, 2019, 9:23 p.m. UTC
Only one dtb is currently supported, so match with imx6-apalis.

Signed-off-by: Ricardo Salveti <ricardo@foundries.io>
---
 board/toradex/apalis_imx6/apalis_imx6.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Peng Fan Sept. 3, 2019, 3:09 a.m. UTC | #1
> Subject: [PATCH] apalis_imx6: add board_fit_config_name_match to support
> FIT in SPL
> 
> Only one dtb is currently supported, so match with imx6-apalis.
> 
> Signed-off-by: Ricardo Salveti <ricardo@foundries.io>
> ---
>  board/toradex/apalis_imx6/apalis_imx6.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/board/toradex/apalis_imx6/apalis_imx6.c
> b/board/toradex/apalis_imx6/apalis_imx6.c
> index fa7fcc8d46..ef668fcedc 100644
> --- a/board/toradex/apalis_imx6/apalis_imx6.c
> +++ b/board/toradex/apalis_imx6/apalis_imx6.c
> @@ -1121,6 +1121,16 @@ void board_init_f(ulong dummy)
>  	board_init_r(NULL, 0);
>  }
> 
> +#ifdef CONFIG_SPL_LOAD_FIT
> +int board_fit_config_name_match(const char *name) {
> +	if (!strcmp(name, "imx6-apalis"))
> +		return 0;
> +
> +	return -1;
> +}
> +#endif
> +
>  void reset_cpu(ulong addr)
>  {
>  }

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> --
> 2.23.0
Max Krummenacher Sept. 3, 2019, 12:58 p.m. UTC | #2
On Mon, 2019-09-02 at 18:23 -0300, Ricardo Salveti wrote:
> Only one dtb is currently supported, so match with imx6-apalis.
> 
> Signed-off-by: Ricardo Salveti <ricardo@foundries.io>
> ---
>  board/toradex/apalis_imx6/apalis_imx6.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/board/toradex/apalis_imx6/apalis_imx6.c b/board/toradex/apalis_imx6/apalis_imx6.c
> index fa7fcc8d46..ef668fcedc 100644
> --- a/board/toradex/apalis_imx6/apalis_imx6.c
> +++ b/board/toradex/apalis_imx6/apalis_imx6.c
> @@ -1121,6 +1121,16 @@ void board_init_f(ulong dummy)
>  	board_init_r(NULL, 0);
>  }
>  
> +#ifdef CONFIG_SPL_LOAD_FIT
> +int board_fit_config_name_match(const char *name)
> +{
> +	if (!strcmp(name, "imx6-apalis"))
> +		return 0;
> +
> +	return -1;
> +}
> +#endif
> +
>  void reset_cpu(ulong addr)
>  {
>  }

I assume that you do not want to switch on CONFIG_SPL_LOAD_FIT
(or OP-TEE) for the apalis_imx6_defconfig but do this rather as
a preparation for a board variant.

Acked-by: Max Krummenacher <max.krummenacher@toradex.com>
diff mbox series

Patch

diff --git a/board/toradex/apalis_imx6/apalis_imx6.c b/board/toradex/apalis_imx6/apalis_imx6.c
index fa7fcc8d46..ef668fcedc 100644
--- a/board/toradex/apalis_imx6/apalis_imx6.c
+++ b/board/toradex/apalis_imx6/apalis_imx6.c
@@ -1121,6 +1121,16 @@  void board_init_f(ulong dummy)
 	board_init_r(NULL, 0);
 }
 
+#ifdef CONFIG_SPL_LOAD_FIT
+int board_fit_config_name_match(const char *name)
+{
+	if (!strcmp(name, "imx6-apalis"))
+		return 0;
+
+	return -1;
+}
+#endif
+
 void reset_cpu(ulong addr)
 {
 }