CLK1 & CLK2 Frequency Setting Interaction Errors when Sharing PLLB #24

Closed
NT7S opened this Issue Aug 21, 2016 · 7 comments

Projects

None yet

2 participants

@NT7S
Member
NT7S commented Aug 21, 2016

The frequency setting of either CLK1 or CLK2 outputs significantly impacts the frequency of CLK2 or CLK1 respectively by 100 KHz or more when operating at HF frequencies. The CLK that is set last is accurately set but the previously set CLK sharing the same PLL is frequency shifted.

This problem only affects CLK outputs sharing the same PLL. Since there are two PLL's the issue is only significant for applications where 3 outputs are needed as two of the outputs must share a PLL.

(copied from @va7ta report on the AVR sister library NT7S/Si5351#11)

@NT7S NT7S added the bug label Aug 21, 2016
@NT7S NT7S added this to the v2.0.0 milestone Aug 21, 2016
@NT7S NT7S self-assigned this Aug 21, 2016
@NT7S
Member
NT7S commented Nov 6, 2016

Initial v2.0.0 commit in libupdate branch includes a new tuning algorithm which addresses this problem.

@va7ta
va7ta commented Nov 7, 2016

Hi Jason,

 I received this notification - many thanks for following up on this 

bug!

 I decided to check out your new V2.0.0  but couldn't find it. 

Arduino Library Manager update shows latest version as 1.1.2 and github
zip files are dated last June.

 Does it usually take a while for the updates to show up on 

distribution sites?

 If you would like me to take it for a spin before it shows up in 

distribution please send me the files.

Best Regards,
Tom, VA7TA

On 16-11-06 12:32 PM, Jason Milldrum wrote:

Initial v2.0.0 commit in libupdate branch includes a new tuning
algorithm which addresses this problem.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#24 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFKe4vxJ3itbIWGS55HqqKIUtg_MMeoXks5q7jlIgaJpZM4JpXVm.

@NT7S
Member
NT7S commented Nov 7, 2016

Hi Tom,

You can find the branch here: https://github.com/etherkit/Si5351Arduino/tree/libupdate

You'll have to manually install the ZIP file in your library folder for now. It's still not ready to be published because I have more features to add, but it is definitely in a state that is suitable for testing with the A variant.

Please do take it for a spin and kick the wheels. I'm sure there are probably a few more bugs to work out. Please notice that the interface has changed a bit, and may change more in the future before release, so look a the README for the updated documentation.

Thanks,
Jason

@va7ta
va7ta commented Nov 7, 2016

Hi Jason,

  Many thanks!

 Just to let you know your new si5351example.ino file failed to 

compile because:

  • clock frequency CLOCKFREQ middle parameter needs to be added to line 30

#define CLOCKFREQ 25000000

  • setfreq lines 33 and 37 have too many parameters - middle parameter
    needs to be removed to match new lib, I think.

    Compiles OK now. Hope this is helpful.

Tom

@NT7S
Member
NT7S commented Nov 7, 2016

Sorry about that, I know that the example sketches are not in a good state yet, but I forgot to mention it. I'll get them tidied up very soon.

@va7ta
va7ta commented Nov 8, 2016

Hi Jason,

 Nothing to be sorry for - I understood that this is a work in 

progress - just wanted to let you know what I spotted.

 Gosh you write nice magic!!!

 I made the necessary changes to my WiFi server code to match your 

new lib and I am happy to report that the 5351 output frequencies are
now all precise! I calibrated the 5351 set to 100 MHz against my counter
which I have connected to an external frequency standard and got it to
within a few Hz. Then I found the other two outputs which were set to HF
frequencies were both within 1 Hz.

 Many thanks for your great work! Hopefully I will get another 

notification from github when you have it all finalized.

Thanks Again!
Tom, VA7TA

On 16-11-07 03:18 PM, Jason Milldrum wrote:

Sorry about that, I know that the example sketches are not in a good
state yet, but I forgot to mention it. I'll get them tidied up very soon.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#24 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFKe4tAHfZyxsKedUgt--wF-nqjLgyHdks5q77G4gaJpZM4JpXVm.

@NT7S
Member
NT7S commented Nov 12, 2016

Glad that it looks like it solves that nasty problem. I've got a little bit more code to add to this version, then I'll make an announcement of a beta version. Hopefully in no more than one week.

@NT7S NT7S referenced this issue Dec 1, 2016
Merged

v2.0.0 Update #26

@NT7S NT7S closed this Dec 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment