dgfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-5963 added other inlet and outlet fans RPM

    • -6
    • +6
    /firmware/App/Modes/ModeHeatDisinfect.c
DEN-5963 tested heat disinfect

    • -102
    • +229
    /firmware/App/Modes/ModeHeatDisinfect.c
    • -1
    • +2
    /firmware/App/Modes/ModeHeatDisinfect.h
    • -3
    • +21
    /firmware/App/Services/WatchdogMgmt.c
DEN-7091: Added acid and bicarb tests

    • -41
    • +158
    /firmware/App/Modes/ModeFill.c
DEN-5963 worked on the heat disinfect states

    • -23
    • +23
    /firmware/App/Controllers/DrainPump.c
    • -79
    • +121
    /firmware/App/Modes/ModeHeatDisinfect.c
DEN-7091: Fixed FPGA revision

DEN-7091: Fixed drain pump conversion error when pump is off

    • -2
    • +13
    /firmware/App/Controllers/DrainPump.c
DEN-7091: Expanded fill command to accept start/stop flag

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Removed.

Removed.

DEN-7091: Addressed code review

I think setActiveReservoirCMD() sets activeReservoir.data for you. Redundant.

I think setActiveReservoirCMD() sets activeReservoir.data for you. Redundant.

Due to changes from non-volatile data management, this message is no longer available. Need to remove to fix build.

Due to changes from non-volatile data management, this message is no longer available.
Need to remove to fix build.

Why removed?

Why removed?

DEN-5963 added Ilimit to the PIControllers.c, added feedback temperature sensor to the heaters.c, worked on heat disinfect mode

    • -33
    • +57
    /firmware/App/Controllers/Heaters.c
    • -17
    • +22
    /firmware/App/Modes/ModeHeatDisinfect.c
    • -0
    • +1
    /firmware/App/Modes/ModeHeatDisinfect.h
    • -6
    • +6
    /firmware/App/Services/PIControllers.c
DEN-5963 worked on RO pump and the pre heat disinfect states

    • -90
    • +136
    /firmware/App/Controllers/ROPump.c
    • -44
    • +47
    /firmware/App/Modes/ModeHeatDisinfect.c
    • -5
    • +5
    /firmware/App/Modes/ModeHeatDisinfect.h
    • -0
    • +1
    /firmware/App/Services/PIControllers.c
    • -1
    • +2
    /firmware/App/Services/PIControllers.h
    • -0
    • +12
    /firmware/App/Services/Reservoirs.c
DEN-5963 merged staging. Worked on RO pump

    • -19
    • +17
    /firmware/App/Controllers/DrainPump.c
    • -7
    • +48
    /firmware/App/Controllers/Heaters.c
    • -6
    • +22
    /firmware/App/Controllers/Heaters.h
    • -6
    • +3
    /firmware/App/Controllers/Thermistors.c
    • -26
    • +19
    /firmware/App/Modes/ModeHeatDisinfect.c
    • -7
    • +0
    /firmware/App/Services/MessagePayloads.h
    • -20
    • +42
    /firmware/App/Services/PIControllers.c
    • -107
    • +182
    /firmware/App/Services/Reservoirs.c
DEN-5963 worked on RO pump and the drain states of heat disinfect

    • -124
    • +6
    /firmware/App/Controllers/ROPump.c
    • -3
    • +4
    /firmware/App/Modes/ModeHeatDisinfect.c
    • -1
    • +0
    /firmware/App/Services/PIControllers.h
DG-DEN-7091_Pre Treatment Mode
DG-DEN-7091_Pre Treatment Mode
DEN-7091: Changed startup active reservoir to 1

DEN-5963 changed the RO pump control scheme from pressure control to flow control

    • -10
    • +1
    /firmware/App/Controllers/LoadCell.h
    • -91
    • +161
    /firmware/App/Controllers/ROPump.c
    • -11
    • +11
    /firmware/App/Modes/ModeHeatDisinfect.c
    • -1
    • +0
    /firmware/App/Services/PIControllers.c
DEN-7091: Added filter flush progress data broadcast

Merge branch 'DEN-6402-hd_dev-pre-treatment-self-tests' into staging

Do these structures require packing? Separate those that do from those that do not and pack only those that do.

Do these structures require packing? Separate those that do from those that do not and pack only those that do.

Why is this commented out?

Why is this commented out?

You can remove the else - no longer needed.

You can remove the else - no longer needed.

These cases should be in same order as enum. POST tests for drivers/controllers/monitors will need NV Data Mgmt to have already read and validated NV Data so they can get/apply calibration. So shou...

These cases should be in same order as enum.
POST tests for drivers/controllers/monitors will need NV Data Mgmt to have already read and validated NV Data so they can get/apply calibration. So should this POST be executed earlier?