On Sun, 30 Aug 2026 at 11:44, Heinrich Schuchardt
<[email protected]> wrote:
>
> All callers already write messages.
>
> * In efi_rng_register we have a log_warning().
> * In get_rng() we have an EFI_PRINT() statement.
>
> These different message types are adequate as get_rng() runs inside the API
> and efi_rng_register() runs on the U-Boot cli level.
>
> Remove the duplicate debug message in platform_get_rng_device()
>
> Signed-off-by: Heinrich Schuchardt <[email protected]>
> ---
> v2:
>         remove the message instead of replacing it
> ---

Reviewed-by: Ilias Apalodimas <[email protected]>
>  lib/efi_loader/efi_rng.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/lib/efi_loader/efi_rng.c b/lib/efi_loader/efi_rng.c
> index 7810b4e47ea..e920eec9d7b 100644
> --- a/lib/efi_loader/efi_rng.c
> +++ b/lib/efi_loader/efi_rng.c
> @@ -30,10 +30,8 @@ __weak efi_status_t platform_get_rng_device(struct udevice 
> **dev)
>         struct udevice *devp;
>
>         ret = uclass_get_device(UCLASS_RNG, 0, &devp);
> -       if (ret) {
> -               debug("Unable to get rng device\n");
> +       if (ret)
>                 return EFI_DEVICE_ERROR;
> -       }
>
>         *dev = devp;
>
> --
> 2.53.0
>

Reply via email to