On Mon, 9 May 2016 18:05:20 +0800
Chen-Yu Tsai [off-list ref] wrote:

quoted

+}
+
+static long ccu_fixed_factor_round_rate(struct clk_hw *hw,
+                                       unsigned long rate,
+                                       unsigned long *parent_rate)
+{
+       struct ccu_fixed_factor *fix = hw_to_ccu_fixed_factor(hw);
+
+       return *parent_rate / fix->div * fix->mult;

Why is this the other way around? With integer math it shouldn't be
interchangeable. (Though it seems clk_fixed_factor does the same...)

Also, clk_fixed_factor handles CLK_SET_RATE_PARENT. Do we need to do the
same here?

Agree. Otherwise, sound in the H3 does not work.

-- 
Ken ar c'henta?	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/