[PATCH 21/24] clk: adi: Remove negative error returns from clk_get_rate
    Greg Malysa 
    malysagreg at gmail.com
       
    Fri Oct 17 06:02:55 CEST 2025
    
    
  
On Wed, Oct 15, 2025 at 10:33 AM Andrew Goodbody
<andrew.goodbody at linaro.org> wrote:
>
> clk_get_rate() returns a ulong so do not attempt to pass negative error
> codes through it.
>
> Signed-off-by: Andrew Goodbody <andrew.goodbody at linaro.org>
> ---
>  drivers/clk/adi/clk-shared.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clk/adi/clk-shared.c b/drivers/clk/adi/clk-shared.c
> index dcadcafa9d236882af8c7b397eeeeb44461a23cb..600e22940e8cb60b1e510a74ee8068ccf0c7848a 100644
> --- a/drivers/clk/adi/clk-shared.c
> +++ b/drivers/clk/adi/clk-shared.c
> @@ -16,7 +16,7 @@ static ulong adi_get_rate(struct clk *clk)
>
>         ret = clk_get_by_id(clk->id, &c);
>         if (ret)
> -               return ret;
> +               return 0;
>
>         return clk_get_rate(c);
>  }
>
> --
> 2.47.3
>
Reviewed-by: Greg Malysa <malysagreg at gmail.com>
Tested on my sc573 and sc598 boards, looks good.
Thanks,
Greg
    
    
More information about the U-Boot
mailing list