[PATCH 03/10] docker: add ADI build-related tools to docker image

[Greg Malysa]

My apologies, I accidentally didn't hit reply to all, please see below.

---------- Forwarded message ---------
From: Greg Malysa <malysagreg at gmail.com>
Date: Tue, Jun 3, 2025 at 3:04 AM
Subject: Re: [PATCH 03/10] docker: add ADI build-related tools to docker image
To: Heinrich Schuchardt <xypron.glpk at gmx.de>


Hi Heinrich,

On Mon, Jun 2, 2025 at 4:11 PM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> Am 2. Juni 2025 21:44:41 MESZ schrieb Greg Malysa <malysagreg at gmail.com>:
> >ADI processors require packaging with the ldr utility for bootrom
>
> "ADI processors" is not self explanatory. Can we add a sentence like
>
> "LDR files are the executable format for the Analog Devices ADSP-SCxxx on-chip Boot ROM. Provide the ldr compiler for ARMv7 and ARMv8."

Yes I can update the message with text like this for v2.

>
> >
> >+# Build ldr tool for ADI boards and install to match expected abi name
> >+RUN git clone https://github.com/analogdevicesinc/lnxdsp-arm-poky-linux-gnueabi-ldr.git /tmp/lnxdsp-arm-poky-linux-gnueabi-ldr && \
> >+      cd /tmp/lnxdsp-arm-poky-linux-gnueabi-ldr/src/ldr && \
> >+      ./configure && \
> >+      make -j $(nproc) && \
> >+      cp ldr /usr/bin/arm-linux-gnueabi-ldr && \
> >+      cp ldr /usr/bin/aarch64-linux-ldr
>
> In the Docker image we tend to put such tools into a directory in /opt.

I will make this change as well and run through CI again on my runner.
I'll wait a week or so for feedback on the rest of the series before
resubmitting. Thanks for the feedback!

Thanks,
Greg