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

On Wed, 28 Sept 2022 at 04:38, Michal Suchanek msuchanek@suse.de 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@suse.de

---

drivers/clk/clk-uclass.c | 2 +- include/clk.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-)

Reviewed-by: Simon Glass sjg@chromium.org