<div dir="ltr"><div><div>Hi Sebastian,<br><br></div>I posted patch for gap-less tuning. It is based on v5 of your driver.  This patch was tested for tuning with 1ppm clock step.<br><br></div><div>Best,<br></div>Michal.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 25, 2013 at 12:54 PM, Sebastian Hesselbarth <span dir="ltr"><<a href="mailto:sebastian.hesselbarth@gmail.com" target="_blank">sebastian.hesselbarth@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Mar 20, 2013 at 5:48 PM, Daniel Mack <<a href="mailto:zonque@gmail.com">zonque@gmail.com</a>> wrote:<br>

> On 20.03.2013 14:55, <a href="mailto:michal.bachraty@gmail.com">michal.bachraty@gmail.com</a> wrote:<br>
>> Thanks for writing this driver! I have tested your si5351 clock<br>
>> driver and his tuning capabilities. It works well, it generates<br>
>> proper clock frequency, but when new frequency is generated, little<br>
>> clock gap (1ms) is generated. Si5351 datasheet and WP claims, clock<br>
>> tuning can be without gaps -<br>
>> <a href="http://www.silabs.com/Support%20Documents/TechnicalDocs/Si5350-51-Frequency-Shifting-WP.pdf" target="_blank">http://www.silabs.com/Support%20Documents/TechnicalDocs/Si5350-51-Frequency-Shifting-WP.pdf</a><br>

>><br>
>> I made some tests with Si5351A chip and I found that when tuning touch<br>
>> only Multisynth registers, it can tune without gaps. There is no need<br>
>> for soft PLL reset. I found also, accessing Multisynth registers is not<br>
>> atomic, so there can be another frequency at output, while not all<br>
>> registers are written. Writing only to one register seems to be atomic.<br>
<br>
</div>Michael,<br>
<br>
if you don't configure the clock output to modify the pll, changing output<br>
frequency will not alter pll config and there will be no reset of pll.<br>
<div class="im"><br>
> Yeah, but limiting possible changes to the PLLs to one single register<br>
> also means that you cannot offer to generate all the frequencies any<br>
> more. What could probably be done is refine the algorithm so that it<br>
> stays 'as close as possible' to the former values, but I'm not sure how<br>
> much work that implies.<br>
><br>
> Can you provide a patch against Sebastian's v3 to do that? Then it can<br>
> be cleanly applied on top of the driver later.<br>
<br>
</div>Ack. Feel free to post a patch on top of v4 now.<br>
<span class="HOEnZb"><font color="#888888"><br>
Sebastian<br>
</font></span></blockquote></div><br></div>