cms-gem-daq-project / ctp7_modules

0 stars 13 forks source link

Fix introduced regression in resetDAQLinkLocal #153

Closed jsturdy closed 4 years ago

jsturdy commented 4 years ago

Description

resetDAQLinkLocal when migrated from cmsgemos introduced the bug that it would disable the DAQ link during a reset. This sequence was not fully validated, and that particular line was commented out in the original HwGenericAMC.cc function:

https://github.com/cms-gem-daq-project/cmsgemos/blob/a6a9ac740f30fd902e9f8628c067fde5d8ed2cfb/gemhardware/src/common/HwGenericAMC.cc#L484-L493

Types of changes

Motivation and Context

DAQ link was being disabled during the Start transition

How Has This Been Tested?

Tested with feature/gemcalib branch in 904, on integration stand