diff mbox series

spl: Kconfig: Record proper dependency for SPL_ATF

Message ID 428b9a306ba354b8956662e6077b6ead4122c5ca.1599127332.git.michal.simek@xilinx.com
State Accepted
Commit 975bacc16ebe0a334767a84ad0a3e147422d6b71
Delegated to: Tom Rini
Headers show
Series spl: Kconfig: Record proper dependency for SPL_ATF | expand

Commit Message

Michal Simek Sept. 3, 2020, 10:02 a.m. UTC
ATF support was all the time based on FIT image support but this dependency
is not recorded anywhere.
For !SPL_FIT && SPL_ATF there is compilation error:
common/spl/spl.c: In function 'board_init_r':
common/spl/spl.c:689:26: error: 'struct spl_image_info' has no member named 'fdt_addr'
  689 |   spl_fixup_fdt(spl_image.fdt_addr);

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 common/spl/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Glass Sept. 7, 2020, 1:43 a.m. UTC | #1
On Thu, 3 Sep 2020 at 04:02, Michal Simek <michal.simek@xilinx.com> wrote:
>
> ATF support was all the time based on FIT image support but this dependency
> is not recorded anywhere.
> For !SPL_FIT && SPL_ATF there is compilation error:
> common/spl/spl.c: In function 'board_init_r':
> common/spl/spl.c:689:26: error: 'struct spl_image_info' has no member named 'fdt_addr'
>   689 |   spl_fixup_fdt(spl_image.fdt_addr);
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  common/spl/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
Michal Simek Sept. 23, 2020, 12:28 p.m. UTC | #2
čt 3. 9. 2020 v 12:02 odesílatel Michal Simek <michal.simek@xilinx.com> napsal:
>
> ATF support was all the time based on FIT image support but this dependency
> is not recorded anywhere.
> For !SPL_FIT && SPL_ATF there is compilation error:
> common/spl/spl.c: In function 'board_init_r':
> common/spl/spl.c:689:26: error: 'struct spl_image_info' has no member named 'fdt_addr'
>   689 |   spl_fixup_fdt(spl_image.fdt_addr);
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  common/spl/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index af8255a8d6ab..a7fabda969d4 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -1271,7 +1271,7 @@ config SPL_YMODEM_SUPPORT
>
>  config SPL_ATF
>         bool "Support ARM Trusted Firmware"
> -       depends on ARM64
> +       depends on ARM64 && SPL_FIT
>         help
>           ATF(ARM Trusted Firmware) is a component for ARM AArch64 which
>           is loaded by SPL (which is considered as BL2 in ATF terminology).
> --
> 2.28.0
>

Applied.
M
diff mbox series

Patch

diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index af8255a8d6ab..a7fabda969d4 100644
--- a/common/spl/Kconfig
+++ b/common/spl/Kconfig
@@ -1271,7 +1271,7 @@  config SPL_YMODEM_SUPPORT
 
 config SPL_ATF
 	bool "Support ARM Trusted Firmware"
-	depends on ARM64
+	depends on ARM64 && SPL_FIT
 	help
 	  ATF(ARM Trusted Firmware) is a component for ARM AArch64 which
 	  is loaded by SPL (which is considered as BL2 in ATF terminology).