dinyar / uGMTfirmware

Firmware for the CMS uGMT.
0 stars 4 forks source link

Disable bits too early in Stage1 sorter #310

Closed dinyar closed 8 years ago

dinyar commented 8 years ago

The disable bits from the Stage0 sorter need to be delayed by one BX before arriving at the Stage1 sorter.

dinyar commented 8 years ago

Side comment: It would be good to figure out why this worked in the tests without cancel out before. In principle these also contain the empty bits so we should have seen a problem earlier..

dinyar commented 8 years ago

The reason this was working before is due to https://github.com/dinyar/uGMTfirmware/blob/master/uGMT_algos/firmware/hdl/GhostBusting/WedgeCheckerUnit.vhd#L119. Here the empty bits are registered at the falling clock, so they were valid long enough for the stage 1 sorter to use them.