fix FEC registers cf ORC-490

Author: sy-c

doc/releaseNotes.md

@@ -61,3 +61,6 @@ This file describes the main feature changes for released versions of ReadoutCar
 
 ## v0.44.2 - 23/06/2023
 - Added a workaround to avoid HW lock issues for CRORC: prevent concurrency in the calls to feed superpages to the card. Added a check to detect write discrepencies.
+
+##
+- Fix registers for FEC status counters.

src/Cru/Constants.h

@@ -215,7 +215,9 @@ static constexpr Register CLOCK_PLL_CONTROL_ONU(0x00240018);
 static constexpr Register ONU_USER_LOGIC(0x0022a000);
 
 /// Register for getting FEC status
-static constexpr Register ONU_FEC_COUNTERS(0x0022200c);
+static constexpr Register ONU_FEC_COUNTERS_STATUS(0x0022A004);
+static constexpr Register ONU_FEC_COUNTERS_SERR(0x0022A020);
+static constexpr Register ONU_FEC_COUNTERS_DERR(0x0022A028);
 
 /// Registers for setting the Ref Gen
 static constexpr Register ONU_USER_REFGEN(0x0022c000);

src/Cru/Ttc.cxx

@@ -215,14 +215,14 @@ OnuStatus Ttc::onuStatus(bool monitoring)
 
 FecStatus Ttc::fecStatus()
 {
-  uint32_t fecStatus = mBar->readRegister(Cru::Registers::ONU_FEC_COUNTERS.index);
+  uint32_t fecStatus = mBar->readRegister(Cru::Registers::ONU_FEC_COUNTERS_STATUS.index);
   return {
     Utilities::getBit(fecStatus, 0) == 1,
     Utilities::getBit(fecStatus, 1) == 1,
-    Utilities::getBit(fecStatus, 7) == 1,
-    (uint8_t)Utilities::getBits(fecStatus, 8, 15),
-    (uint8_t)Utilities::getBits(fecStatus, 16, 23),
-    (uint8_t)Utilities::getBits(fecStatus, 24, 31)
+    0, // meaningless, kept for compatibility
+    (uint8_t)mBar->readRegister(Cru::Registers::ONU_FEC_COUNTERS_SERR.index),
+    (uint8_t)mBar->readRegister(Cru::Registers::ONU_FEC_COUNTERS_DERR.index),
+    0 // meaningless, kept for compatibility
   };
 }