[PATCH] clk: change return type of clk_get_parent_rate from long long to ulong

[Sean Anderson]

On 9/28/22 06:37, Michal Suchanek wrote:
> All functions getting and setting clock rate use ulong for rate, only
> clk_get_parent_rate is an exception. Change the return value to match
> other clock rate funcrions.
> 
> Most users directly assign the rate to unsigned long anyway, and the few
> users that use u64 (not s64) multiply the rate so they may need the
> extra bits for the result in their use case.
> 
> Fixes: 4aa78300a0 ("dm: clk: Define clk_get_parent_rate() for clk operations")
> Signed-off-by: Michal Suchanek <msuchanek at suse.de>
> ---
> 
>   drivers/clk/clk-uclass.c | 2 +-
>   include/clk.h            | 4 ++--
>   2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c
> index b89c77bf79..4678ed43af 100644
> --- a/drivers/clk/clk-uclass.c
> +++ b/drivers/clk/clk-uclass.c
> @@ -505,7 +505,7 @@ struct clk *clk_get_parent(struct clk *clk)
>   	return pclk;
>   }
>   
> -long long clk_get_parent_rate(struct clk *clk)
> +ulong clk_get_parent_rate(struct clk *clk)
>   {
>   	const struct clk_ops *ops;
>   	struct clk *pclk;
> diff --git a/include/clk.h b/include/clk.h
> index 76bb64bb5e..41a8fb3419 100644
> --- a/include/clk.h
> +++ b/include/clk.h
> @@ -444,7 +444,7 @@ struct clk *clk_get_parent(struct clk *clk);
>    *
>    * Return: clock rate in Hz, or -ve error code.
>    */
> -long long clk_get_parent_rate(struct clk *clk);
> +ulong clk_get_parent_rate(struct clk *clk);
>   
>   /**
>    * clk_round_rate() - Adjust a rate to the exact rate a clock can provide
> @@ -577,7 +577,7 @@ static inline struct clk *clk_get_parent(struct clk *clk)
>   	return ERR_PTR(-ENOSYS);
>   }
>   
> -static inline long long clk_get_parent_rate(struct clk *clk)
> +static inline ulong clk_get_parent_rate(struct clk *clk)
>   {
>   	return -ENOSYS;
>   }

Reviewed-by: Sean Anderson <seanga2 at gmail.com>