On 30.04.2015 20:49, Michael Welling wrote:
On Thu, Apr 30, 2015 at 07:45:54PM +0200, Sebastian Hesselbarth wrote:
When changing PLL rate significantly, PLLs have to be reset. Add a function
to perform and check for successful PLL reset.
Signed-off-by: Sebastian Hesselbarth <sebastian.hesselbarth@xxxxxxxxx>
---
[...]
@@ -519,6 +545,9 @@ static int si5351_pll_set_rate(struct clk_hw *hw, unsigned long rate,
SI5351_CLK_INTEGER_MODE,
(hwdata->params.p2 == 0) ? SI5351_CLK_INTEGER_MODE : 0);
+ /* reset pll after rate change */
+ si5351_pll_reset(hwdata);
+
What is the point of having a return code if it is not being used?
Good point, I'll pass that on to the return value of .set_rate()
callback for v2.
Sebastian
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
