Use SNAP board's 1pps rather than ADC's

realtimeradio / ata_snap

FPGA firmware for SNAP boards at the ATA

GNU General Public License v3.0

4 stars 5 forks source link

Use SNAP board's 1pps rather than ADC's #15

Closed wfarah closed 3 years ago

wfarah commented 4 years ago

As already noticed before, the ADC 1pps is dropping pulses. We changed the wiring such that we use the 1pps directly to the SNAP board, and it seems that it is working as expected with no pulses seen to be dropping.

A change of value to the sync_sel register must be performed somewhere to reflect this change and make it a default. We wired all the 1pps here accordingly

realtimeradio commented 3 years ago

Do you want this to be --

A config file parameter?
An initialization script command line parameter?
fixed so you never have to think about it?

wfarah commented 3 years ago

Fixed. Until we find a critical flaw in using the connection

realtimeradio commented 3 years ago

Should now be fixed in fee15a2

wfarah commented 3 years ago

A tiny bug in:

https://github.com/realtimeradio/ata_snap/blob/11c34b25ceb1ea36d18c681b723524102bdb86fe/sw/ata_snap/src/ata_snap_fengine.py#L141

_ in _pps_source needs to be removed