[PATCH v2 5/9] bootstd: Correct creating of bootdev sibling

[Bin Meng]

Hi Simon,

On Mon, Jul 31, 2023 at 1:15 AM Simon Glass <sjg at chromium.org> wrote:
>
> Use the correct function here, since there may be multiple IDE devices
> available.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Mattijs Korpershoek <mkorpershoek at baylibre.com>
> ---
>
> (no changes since v1)
>
>  drivers/block/ide.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/block/ide.c b/drivers/block/ide.c
> index 89201dd4d229..c698f9cbd558 100644
> --- a/drivers/block/ide.c
> +++ b/drivers/block/ide.c
> @@ -1059,9 +1059,9 @@ static int ide_probe(struct udevice *udev)
>                 desc->lba48 = pdesc.lba48;
>                 desc->type = pdesc.type;
>
> -               ret = bootdev_setup_for_dev(udev, "ide_bootdev");
> +               ret = bootdev_setup_for_sibling_blk(blk, "ide_bootdev");
>                 if (ret)
> -                       return log_msg_ret("bootdev", ret);
> +                       return log_msg_ret("bd", ret);

Why changes from bootdev to bd here? To me, bd is unclear, and sounds
like "boardinfo" in U-Boot.

>         }
>
>         return 0;
> --

Regards,
Bin