Re: [PATCH V2 00/10] board/ti: k3 boards: Stop using findfdt

Roger Quadros
2024-01-11T10:17:44Z

On 09/01/2024 21:14, Nishanth Menon wrote:
> This is a wide cleanup to switch to setting fdtfile using env_set
> instead of scripted magic. 'fdtfile' is expected to be set by default.
> This allows the stdboot triggered efi loaders to find the correct OS
> device tree file even if regular boot process is interrupted by user
> intervention.
>
> This is a refresh of
> https://lore.kernel.org/all/86le9dwz4d.fsf@udb0321960.dhcp.ti.com/ which
> was the wrong approach.
>
> Updates from V1:
> * Renames of variables and macros for various review comments.
> * Commit message updates in some patches for clarity.
>
> Bootlogs: https://gist.github.com/nmenon/843e343cde645ec4aa57b60cece5256a
>
> based on master: c5e461fbf7cc Merge tag 'u-boot-imx-master-20240108' of https://gitlab.denx.de/u-boot/custodians/u-boot-imx
>
> NOTE: There are a couple of checkpatch WARN (around LATE_INIT) and
> CHECK (fdt_ops #ifdeffery) that on closer inspection looks fine and
> consistent with other similar usage.
>
> V1: https://lore.kernel.org/all/20240108173301.2692332-1-nm@ti.com/
>
> Nishanth Menon (10):
> board: ti: common: Introduce a common fdt ops library
> board: ti: am62ax: Set fdtfile from C code instead of findfdt script
> board: ti: am62x: Set fdtfile from C code instead of findfdt script
> board: ti: am64x: Set fdtfile from C code instead of findfdt script
> board: ti: am65x: Set fdtfile from C code instead of findfdt script
> board: ti: j721e: Set fdtfile from C code instead of findfdt script
> board: ti: j721s2: Set fdtfile from C code instead of findfdt script
> board: beagle: beagleboneai64: Set fdtfile from C code instead of
> findfdt script
> board: beagle: beagleplay: Set fdtfile from C code instead of findfdt
> script
> include: env: ti: Drop default_findfdt

For this series:

Reviewed-by: Roger Quadros <rogerq@kernel.org>

--
cheers,
-roger