[PATCH 33/40] test: cedit: use allocated address for reading file

Simon Glass sjg at chromium.org
Fri Jul 26 01:32:36 CEST 2024


Hi Sughosh,

On Wed, 24 Jul 2024 at 00:05, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> Instead of a randomly selected address, use an LMB allocated one for
> reading the file into memory. With the LMB map now being persistent
> and global, the address used for reading the file might be already
> allocated as non-overwritable, resulting in a failure. Get a valid
> address from LMB and then read the file to that address.
>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
> Changes since rfc: None
>
>  test/boot/cedit.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>

No, we should not start putting lmb into tests like this.

Regards,
Simon


More information about the U-Boot mailing list