Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Power needs to be cycled after Write Calibration #15

Open
dbogdan opened this issue Jan 13, 2018 · 16 comments
Open

Power needs to be cycled after Write Calibration #15

dbogdan opened this issue Jan 13, 2018 · 16 comments

Comments

@dbogdan
Copy link
Contributor

dbogdan commented Jan 13, 2018

After writing new calibration values to the board, the outputs of the AWG are all messed up and you need to cycle power to the board to get it working again. After cycling the power, the board seems to be properly calibrated.

@damercer
Copy link
Contributor

@dbogdan, any thoughts on this? Should I wait for this to be fixed or reload both of these Rev F M1K boards I have been testing with v 2.09 as a fall back. ( the last version that seemed to work the best ).

Thanks

Doug

@dbogdan
Copy link
Contributor Author

dbogdan commented Jan 15, 2018

Unfortunately, we didn't find what can cause this behavior. The new coefficients seem to be correctly stored even without cycling the power.
By the way, did you see any other issues with v2.16?

@rgetz
Copy link

rgetz commented Jan 15, 2018

Sounds like we just need to dig deeper/longer.

@damercer
Copy link
Contributor

damercer commented Jan 15, 2018 via email

@damercer
Copy link
Contributor

damercer commented Jan 15, 2018 via email

@damercer
Copy link
Contributor

Just trying to be helpful here but it would seem to me that you could narrow things down a lot by comparing what you changed between 2.15 (works) and 2.16 (don't work). Could not have been very many things that were effected or am I wrong.

Doug

@dbogdan
Copy link
Contributor Author

dbogdan commented Jan 16, 2018

Hmmm... if I remember correctly, we've seen a different behavior starting with 2.14...

@damercer
Copy link
Contributor

damercer commented Jan 16, 2018 via email

@damercer
Copy link
Contributor

I tested the following version of the firmware to see at what point the write to calibration settings broke.
2.10, 2.11. 2.12, 2.13 ( didn't have 2.14 handy ) all work.
2.15 and 2.16 do not.

So my suggestion still might be helpful if you undo the changes one at a time between working and not working version you might find what was done to cause the break?

I'm a little disappointed that this problem was left in with out being notified that an issue existed for so long. It caused me much confusion.

Doug

@dbogdan
Copy link
Contributor Author

dbogdan commented Jan 18, 2018

Sorry to hear that, Doug. We discovered the issue only recently too, but we checked all the new versions to find the cause.
In order to be able to add support for newer hardware revisions, we had to optimize more the timer interrupt handler - it seems that this was the point where this strange behavior appeared. Since we thought that the calibration will not be done too often (for many users probably never - if the board comes calibrated from factory), we decided that this is not something critical.

@damercer
Copy link
Contributor

damercer commented Jan 18, 2018 via email

@dbogdan
Copy link
Contributor Author

dbogdan commented Jan 18, 2018

libsmu should (I guess that it already does this) take care of saving and restoring the calibration coefficients when a firmware upgrade is done using the library. Unfortunately, that memory can't be protected.
For the moment, every user should unplug/plug the board after a new calibration is done.
Otherwise, I agree with what you said.

@rgetz
Copy link

rgetz commented Jan 18, 2018

@dbogdan : "For the moment"; means we acknowledge it's a bug, and we are working on solving it? Or that is the permanent solution?

@damercer
Copy link
Contributor

damercer commented Jan 18, 2018 via email

@dbogdan
Copy link
Contributor Author

dbogdan commented Jan 19, 2018

Certainly, we don't want this to be a permanent solution: we will still investigate what can be done.

@damercer
Copy link
Contributor

Has this issue been resolved in the version released today?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants