[PATCH v2 27/32] test: cedit: use allocated address for reading file

Simon Glass sjg at chromium.org
Thu Aug 15 22:32:29 CEST 2024


Hi Sughosh,

On Wed, 14 Aug 2024 at 05:02, 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 V1:
> * Don't use the API version with flags parameter.
>
>  test/boot/cedit.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>

No, this address needs to work fine without using lmb. Same with any
other tests. Tests make use of the sandbox memory space memory
addresses and it makes things easier to code and debug.

Regards,
SImon


More information about the U-Boot mailing list