addresses cms-gem-daq-project#297, bits [31:16] of HW_CHIP_ID are set…

… to zero if rawID is False

gempython/tools/vfat_user_functions_xhal.py

@@ -123,7 +123,9 @@ def getAllChipIDs(self, mask=0x0, rawID=False):
         Returns the chipIDs for all VFATs not in mask.
 
         mask - vfatMask, up to a 24 bit number, 1 in the N^th bit means skip this VFAT
-        rawID - If true returns the rawID and does not apply the Reed-Muller decoding
+        rawID - If True returns the rawID and does not apply the Reed-Muller decoding;
+                if False bits [31:16] of HW_CHIP_ID will be always zero in an attempt
+                to suppress errors from bit flips in non Reed-Muller Encoded VFATs.
         """
 
         chipIDData = (c_uint32 * vfatsPerGemVariant[self.gemType])()
@@ -132,6 +134,11 @@ def getAllChipIDs(self, mask=0x0, rawID=False):
         if rpcResp != 0:
             raise Exception("RPC response was non-zero, failed to get chipID data for OH{0}".format(self.parentOH.link))
 
+        # Suppress any potential bit flips in chipID (e.g. in non-reed muller encoded chips)
+        if rawID == False:
+            for vfat in len(chipIDData):
+                chipIDData[vfat] = chipIDData[vfat] & 0xffff
+
         return chipIDData
 
     def getAllChannelRegisters(self, mask=0x0):