[PATCH] image: fix select_ramdisk for raw initrd

Simon Glass sjg at chromium.org
Mon Nov 1 00:47:04 CET 2021


Hi Artem,

On Fri, 15 Oct 2021 at 04:15, Artem Lapkin <email2tema at gmail.com> wrote:
>
> Problem
>
> We have unbootable raw initrd images because, select_ramdisk for raw
> initrd images ignore submited select addr and setup rd_datap value to 0
>
> Solution: setup rd_datap value from select
>
> Signed-off-by: Artem Lapkin <art at khadas.com>
> ---
>  common/image-board.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Do we need this patch as well as the other one? If so, can you combine them?

Also, is it possible to drop the assignment to rd_addr at the top of
the function?

>
> diff --git a/common/image-board.c b/common/image-board.c
> index e7660352e9..e3c6ea806a 100644
> --- a/common/image-board.c
> +++ b/common/image-board.c
> @@ -439,7 +439,7 @@ static int select_ramdisk(bootm_headers_t *images, const char *select, u8 arch,
>                                 end = strchr(select, ':');
>                         if (end) {
>                                 *rd_lenp = hextoul(++end, NULL);
> -                               *rd_datap = rd_addr;
> +                               *rd_datap = hextoul(select, NULL);
>                                 processed = true;
>                         }
>                 }
> --
> 2.25.1
>

Regards,
Simon


More information about the U-Boot mailing list