hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merge branch 'DEN-13834-dg_hd_dev-hd_dg_dvt-update-part-3' into develop

DEN-13834 updated the blood leak self test status set

Merge branch 'staging' into DEN-13427-sw_dev-sprint-75_mg

    • -103
    • +135
    /firmware/App/Services/SystemCommMessages.c
Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

DEN-13427: Addressed code review comments

DEN-13598 Update code for new SRS with alarms.

    • -81
    • +10
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +0
    /firmware/App/Controllers/DGInterface.h
Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merged DEN-13834

    • -34
    • +56
    /firmware/App/Controllers/DGInterface.c
    • -29
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -17
    • +13
    /firmware/App/Controllers/PresOccl.c
    • -4
    • +2
    /firmware/App/Controllers/SyringePump.c
    • -7
    • +6
    /firmware/App/Controllers/Temperatures.c
    • -0
    • +12
    /firmware/App/Drivers/SafetyShutdown.c
    • -5
    • +10
    /firmware/App/Services/SystemComm.c
DEN-13834 added the DG heaters broadcast message. Addded a function to check the status of the trimmer heater's state in DGInterface and keep sending the command from HD until the trimmer heater is in the right status. Enabled stop/start trimmer command in the modes

    • -1
    • +41
    /firmware/App/Controllers/DGInterface.c
    • -29
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -10
    • +0
    /firmware/App/Controllers/DialOutFlow.h
I think the style is to use set/get functions if needed by other modules, otherwise just access the variable directly.

I think the style is to use set/get functions if needed by other modules, otherwise just access the variable directly.

Need to initialize here at declaration or in init function. I see neither now.

Need to initialize here at declaration or in init function. I see neither now.

DEN-13460 addressed the code review comments

    • -29
    • +31
    /firmware/App/Controllers/BloodLeak.c
    • -11
    • +11
    /firmware/App/Controllers/DGInterface.c
    • -2
    • +2
    /firmware/App/Controllers/DialInFlow.c
    • -2
    • +2
    /firmware/App/Controllers/DialOutFlow.c
    • -3
    • +2
    /firmware/App/Controllers/SyringePump.c
    • -7
    • +6
    /firmware/App/Controllers/Temperatures.c
Thanks for the reminder.

Thanks for the reminder.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

More of a caution than an explicit fix. Dong's S77 branch has some of the changes that are reflected here. He also has the more recent commit. Be careful when both of you merge your branches.

More of a caution than an explicit fix. Dong's S77 branch has some of the changes that are reflected here. He also has the more recent commit. Be careful when both of you merge your branches.

remove blank line

remove blank line

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.