Re: [Buildroot] [PATCH v2 18/18] board/lx2160ardb: new platform

From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: buildroot@buildroot.org, Ioana Ciornei <ciorneiioana@gmail.com>,
	Brandon Maier <Brandon.Maier@collins.com>,
	Vincent Jardin <vjardin@free.fr>, Julien Olivain <ju.o@free.fr>,
	Romain Naour <romain.naour@smile.fr>,
	Akhilesh Nema <nemaakhilesh@gmail.com>
Subject: Re: [Buildroot] [PATCH v2 18/18] board/lx2160ardb: new platform
Date: Tue, 22 Apr 2025 23:45:18 +0200	[thread overview]
Message-ID: <20250422234518.2422983e@windsurf> (raw)
In-Reply-To: <20250328180427.3434693-19-olteanv@gmail.com>

Hello Vlad,

On Fri, 28 Mar 2025 20:04:26 +0200
Vladimir Oltean <olteanv@gmail.com> wrote:

> +The components from NXP are:
> +  - rcw, lf-6.12.3-1.0.0
> +  - atf (fork), lf-6.12.3-1.0.0
> +  - uboot (fork), lf-6.12.3-1.0.0
> +  - cadence-dp-firmware (blob), lf-6.12.3-1.0.0

So this line should be dropped, if I understand correctly?

> diff --git a/board/freescale/lx2160ardb/uboot-env.txt b/board/freescale/lx2160ardb/uboot-env.txt
> new file mode 100644
> index 000000000000..ad1e5882dd9e
> --- /dev/null
> +++ b/board/freescale/lx2160ardb/uboot-env.txt

I agree with Vincent that it would be nice to have a simpler U-Boot
environment if possible.

> +BR2_PACKAGE_QORIQ_RESTOOL=y

Due to restool not being applied (build issues), I can't apply this
defconfig just yet. Can you send a v3 with restool fixed? Thanks!

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot