Should AND this bit off from register so we don't interfere with other bits that may have been set by other drivers (e.g. bubble detector). Should be something like fpgaActuatorSetPoints.fpgaSensorTest &= ~FPGA_BLOOD_LEAK_SELF_TEST_CMD;
Dara and I have found out that this code review for common is misconfigured. The source and destination branches are misconfigured. The branch to review should have been from DEN-11980_sw_dev_sprint_64 and the "branch from" should have been from staging. This code review has staging as the branch to review with master as the "branch review."
The software configurations and calibration excel reports are protected. They are protected because the format of the reports do not have to change the scripts rely on them to read the data back and send them to the firmware. The only cells that are not protected are the cells that the users can changed the values (i.e. the calibration values). This feature is optional a report can be created that is not protected.