Hi guys,
I just found today with playing with clock drivers that the patch clk: convert API to match reset/mailbox style (sha1: 135aa95002646c46e89de93fa36adad1b010548f)
added this part of code to fixed clock driver -static ulong clk_fixed_rate_get_rate(struct udevice *dev) +static ulong clk_fixed_rate_get_rate(struct clk *clk) { - return to_clk_fixed_rate(dev)->fixed_rate; -} + if (clk->id != 0) + return -EINVAL;
which is returning -EINVAL when ulong should be returned.
The next thing I have found is that fixed clock driver has no set_rate function which is fine but when I was testing one driver which tries to set rate then error code was generated but without any useful information what happened. Are you ok with adding empty set_rate function with returning error message that set rate is not supported for fixed clocks?
Thanks, Michal