I'm not sure we need these "Detected" functions. I would expect that when this controller gets new readings, it will check for these alarm conditions in the check state and trigger alarm if appropriate. So if we are triggering alarms in this controller, there is no need for outside code to ask if these alarms are detected.
I don't think these definitions need to be public. Some of these should probably be inside the FpgaTD.c and some should probably be inside your driver's .c file.
