[PATCH v3 2/2] configs: add defconfigs for the am625-lp-sk

[Tom Rini]

On Fri, May 03, 2024 at 11:44:29AM -0500, Bryan Brattlof wrote:

> The am62x-lp-sk is a package and reference board spin of the am62x-sk to
> showcase the low-power features of the am62x SoC family. Because it so
> closely resembles the am62x-sk board, use the preprocessor to inherit
> its configuration making the needed changes for this board where
> necessary.
> 
> Reviewed-by: Dhruva Gole <d-gole at ti.com>
> Signed-off-by: Bryan Brattlof <bb at ti.com>
> ---
>  configs/am62x_lp_sk_a53_defconfig | 3 +++
>  configs/am62x_lp_sk_r5_defconfig  | 2 ++
>  2 files changed, 5 insertions(+)
> 
> diff --git a/configs/am62x_lp_sk_a53_defconfig b/configs/am62x_lp_sk_a53_defconfig
> new file mode 100644
> index 0000000000000..904b2142b2f53
> --- /dev/null
> +++ b/configs/am62x_lp_sk_a53_defconfig
> @@ -0,0 +1,3 @@
> +#include <configs/am62x_evm_a53_defconfig>
> +CONFIG_DEFAULT_DEVICE_TREE="ti/k3-am62-lp-sk"
> +CONFIG_OF_UPSTREAM=y

So, there's a problem here. The #include trick for defconfig files isn't
working as intended, exactly. The example here doesn't work right.
First, it shows up as a variant of "sandbox" (as buildman will show and
leads to https://source.denx.de/u-boot/u-boot/-/jobs/835067#L119)

And this becomes clearer if you look at configs/am69_sk_r5_defconfig
which has to set some symbols already found in
configs/j784s4_evm_r5_defconfig in order to work. This is seemingly very
not equivalent to invoking "make foo_defconfig bar.config" to combine
things.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240515/117e536e/attachment.sig>