epics-modules / mrfioc2

EPICS driver for Micro Research Finland event timing system devices
http://epics-modules.github.io/mrfioc2/
Other
8 stars 29 forks source link

fix: Correct the setFrequency command #117

Closed agaget closed 6 months ago

agaget commented 6 months ago

The setFrequency command doesn't set the correct prescaler and so the frequency, because the prescaler is wrongly computed.

We need to use the m_owner->GetFrequency and not getFrequency(), otherwise we obtain a frequency that is not correct and change according to the prescaler computed.

image

It seems you are using git cliff, so I've tried to use a correct commit message, tell me if its ok otherwise I can change it.

jerzyjamroz commented 6 months ago

@agaget , please review: https://github.com/agaget/mrfioc2/pull/1

agaget commented 6 months ago

@agaget , please review: agaget#1

done