firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-5980 addressed code review comments

    • -30
    • +1
    ./App/Services/SystemCommMessages.c
fwcommon had to be updated with latest changes. It is un-commented.

fwcommon had to be updated with latest changes. It is un-commented.

fwcommon had to be updated with latest changes. They are un-commented.

fwcommon had to be updated with latest changes. They are un-commented.

I just need to run NVDataMgmt POST and nothing else. I added a build switch.

I just need to run NVDataMgmt POST and nothing else. I added a build switch.

I am not sure. I un-commented it.

I am not sure. I un-commented it.

Done.

Done.

Done

Done

Done

Done

Done

Done

To be able to run the code on a board with no hardware. I added a build switch.

To be able to run the code on a board with no hardware. I added a build switch.

To be able to run the code on a board with no hardware attached. I added a build switch.

To be able to run the code on a board with no hardware attached. I added a build switch.

To be able to run the code on a board with no hardware attached. I added a build switch.

To be able to run the code on a board with no hardware attached. I added a build switch.

Done

Done

Done

Done

Done

Done

Done

Done

Yes it is part of fwcommon. To review fwcommon, please review DG-DEN-5980 code review.

Yes it is part of fwcommon. To review fwcommon, please review DG-DEN-5980 code review.

The calibration data is validated in NVDataMgmt driver.

The calibration data is validated in NVDataMgmt driver.

HD-DEN-7347_DG HD Dev Post
HD-DEN-7347_DG HD Dev Post
Added specific fault for each check.

Added specific fault for each check.

DEN-7091: Added fault for each self-test check

Either add TODO or delete commented line.

Either add TODO or delete commented line.

Not better to use this instead? bloodPumpMotorEdgeCount += (U16)(delta);

Not better to use this instead? bloodPumpMotorEdgeCount += (U16)(delta);

Removed. It seems nothing much we can do. The door switch is simple and just a GPIO read from FPGA.

Removed. It seems nothing much we can do. The door switch is simple and just a GPIO read from FPGA.

Straw door needs to be open for concentrate jugs' connection. Other than that, I am not aware of any flow path that requires closed door. Added TODO.

Straw door needs to be open for concentrate jugs' connection. Other than that, I am not aware of any flow path that requires closed door.
Added TODO.

Done.

Done.

Combined air trap and bubble detectors into one used cartridge check.

Combined air trap and bubble detectors into one used cartridge check.

Combined air trap and bubble detectors into one used cartridge check.

Combined air trap and bubble detectors into one used cartridge check.

DEN-7091: Addressed code review

Wrong valve state. Fixed.

Wrong valve state. Fixed.