[PATCH] build: Revive and update LDR format support

Tom Rini trini at konsulko.com
Wed Mar 20 02:18:53 CET 2024


On Tue, Mar 19, 2024 at 09:09:22PM -0400, Greg Malysa wrote:

> LDR format files are used primarily by Analog Devices processors but may
> be of interest to other vendors. Previously support existed for this
> format as part of the U-Boot build, but it has been unmaintained and
> unused for a long time. In preparation for adding support for modern ADI
> processors that use LDR, modernize the LDR support:
> 
> - Introduce CONFIG_LDR_CPU as the CPU string recognized by the LDR tool
>   may not be the same as CONFIG_CPU
> - Add an SPL target that repackages u-boot-spl inside an LDR file
> 
> An almost identical target for packaging u-boot into an LDR file already
> exists and did not need to be updated
> 
> Co-developed-by: Nathan Barrett-Morrison <nathan.morrison at timesys.com>
> Signed-off-by: Nathan Barrett-Morrison <nathan.morrison at timesys.com>
> Signed-off-by: Ian Roberts <ian.roberts at timesys.com>
> Signed-off-by: Greg Malysa <greg.malysa at timesys.com>
> 
> ---
> 
>  Kconfig              | 9 ++++++++-
>  Makefile             | 2 +-
>  scripts/Makefile.spl | 5 +++++
>  3 files changed, 14 insertions(+), 2 deletions(-)
> 
> diff --git a/Kconfig b/Kconfig
> index 5710934000..3f3ce6054e 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -715,10 +715,17 @@ config SYS_CLK_FREQ
>  	  A static value for the CPU frequency.  Note that if not required
>  	  for a given SoC, this can be left at 0.
>  
> -source "api/Kconfig"
> +config LDR_CPU
> +	string "CPU name to be passed to LDR utility."
> +	help
> +	  Set the CPU name for the -T parameter in the LDR utility.  This is
> +	  generally used on processors from Analog Devices, but may be also
> +	  be useful for other vendors.
>  
>  endmenu		# General setup

This looks like a new prompted option, without a default, for every
platform. If so, as is it will cause everything in CI to fail to build.
This should "depend on" some relevant LDR symbol, once that's been
added, rather than as a ground-work patch. This rest seems fine, FWIW.

-- 
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/20240319/845f75af/attachment.sig>


More information about the U-Boot mailing list