Skip to content

Add 64b66b decoder and RD53 encoder

Marco Trovato requested to merge phase2/Add64b66bdecoderRD53encoder_cleanup into phase2/master

Hi Frans,

I have created Encoding/DecodingEgroup/EpathLPGBT files in order to decouple as much as possible the LPGBT flavor with the other flavors. That creates a lot of redundancy, which, I think, can be avoided by adding a number of generics. I can work on it if interested.

The CRFromAxis tiny change that we discussed today is not deployed (https://gitlab.cern.ch/atlas-tdaq-felix/firmware/-/blob/phase2/Add64b66bdecoderRD53encoder/sources/CRFromHostAxis/CRFromHostAxis.vhd#L282-290). I will let you and Marius handle that.

Please review the branch and add the changes for Sigasi and questasim implemented in https://gitlab.cern.ch/atlas-tdaq-felix/firmware/-/tree/phase2/Add64b66bdecoderRD53encoder_CleanupForMerge. You can directly change this branch, I won't mind.

Marco

Merge request reports