review Si5324 setup code #208

Closed
opened 2022-12-09 16:27:33 +08:00 by sb10q · 2 comments

Zynq codebase might also need those fixes:
261dc6b933
740543d4e2
3c7a394eff

Zynq codebase might also need those fixes: https://github.com/m-labs/artiq/commit/261dc6b9330e831720bcb0de116fab4ece2c8bf4 https://github.com/m-labs/artiq/commit/740543d4e284245248e3ff838c46505938dcae7a https://github.com/m-labs/artiq/commit/3c7a394eff553ab75a7ea78bdd17830366504dc6
dpn was assigned by sb10q 2022-12-09 16:27:33 +08:00

261dc6b933 was apparently already fixed in artiq-zynq.

740543d4e2 was also fixed a different way; unified with the upstream ARTIQ solution in 5c054cc901 just to keep the implementations close. (Any long-term plan here, by the way?)

261dc6b933 was apparently already fixed in artiq-zynq. 740543d4e2 was also fixed a different way; unified with the upstream ARTIQ solution in https://git.m-labs.hk/M-Labs/artiq-zynq/commit/5c054cc90194b7647f78168b51bc452eed42f33e just to keep the implementations close. (Any long-term plan here, by the way?)
dpn closed this issue 2022-12-13 07:29:35 +08:00
Poster
Owner

We should merge the firmware code and put everything into the main ARTIQ repository at some point.

We should merge the firmware code and put everything into the main ARTIQ repository at some point.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/artiq-zynq#208
There is no content yet.