td-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'staging' into LDT-4028-pre-treatment---fw-implementation

LDT-4029:Merge lates staging

Revert "LDT-4029: Merge latest staging and addressed comments"

This reverts commit 069dee3a2428d3c6bb281db1844a372ae6e2063a.

    • -35
    • +17
    /firmware/App/Controllers/AirPump.c
    • -27
    • +17
    /firmware/App/Controllers/Valves.c
    • -98
    • +24
    /firmware/App/Modes/ModeStandby.c
    • -29
    • +28
    /firmware/App/Modes/StatePreTxHeparinSetup.c
    • -3
    • +3
    /firmware/App/Modes/StateTxBloodPrime.c
    • -30
    • +17
    /firmware/App/Modes/StateTxPaused.c
    • -397
    • +342
    /firmware/App/Services/AlarmMgmtTD.c
    • -71
    • +179
    /firmware/App/Services/StateServices/FluidBolus.c
  1. … 8 more files in changeset.
LDT-4029: Merge latest staging and addressed comments

    • -17
    • +35
    /firmware/App/Controllers/AirPump.c
    • -17
    • +27
    /firmware/App/Controllers/Valves.c
    • -24
    • +98
    /firmware/App/Modes/ModeStandby.c
    • -28
    • +29
    /firmware/App/Modes/StatePreTxHeparinSetup.c
    • -3
    • +3
    /firmware/App/Modes/StateTxBloodPrime.c
    • -17
    • +30
    /firmware/App/Modes/StateTxPaused.c
    • -342
    • +397
    /firmware/App/Services/AlarmMgmtTD.c
    • -179
    • +71
    /firmware/App/Services/StateServices/FluidBolus.c
  1. … 8 more files in changeset.
LDT-4149: changes plus test code in standby.

    • -106
    • +92
    /firmware/App/Drivers/PinchValve.c
Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

LDT-4028: Addressed Review Commentsfor Install state

LDT-4149: Driver Add ons

LDT-4149: Driver Add ons

LDT-4149: Driver Add ons

    • -193
    • +573
    /firmware/App/Drivers/PinchValve.c
Merge branch 'staging' into LDT-4028-pre-treatment---fw-implementation

Update all function headers in this file to Leahi style (e.g. \b before inputs/outputs).

Update all function headers in this file to Leahi style (e.g. \b before inputs/outputs).

Add extra line (should be 2 lines) after test support banner.

Add extra line (should be 2 lines) after test support banner.

Remove extra blank line.

Remove extra blank line.

Update function header format from Denali to Leahi style (e.g. \b for inputs/outputs) throughout this file.

Update function header format from Denali to Leahi style (e.g. \b for inputs/outputs) throughout this file.

currentInstallState is an input.

currentInstallState is an input.

Do we need doxygen comments for static function declarations? We will have a header in the definition.

Do we need doxygen comments for static function declarations?
We will have a header in the definition.

AirPump.h should be above AirTrap.h

AirPump.h should be above AirTrap.h

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

Need to set state to preload state I think. Otherwise you will go to interrupted state.

Need to set state to preload state I think. Otherwise you will go to interrupted state.

Why did you remote call to init function here? Is it initialized somewhere else?

Why did you remote call to init function here? Is it initialized somewhere else?

Use ternary operation to assign TRUE or FALSE explicitly.

Use ternary operation to assign TRUE or FALSE explicitly.

This is at end of enum list above. So it should be at end of table here.

This is at end of enum list above. So it should be at end of table here.

LDT-4029: Addressed Review comments for Heparin Setup State

    • -124
    • +69
    /firmware/App/Modes/StatePreTxHeparinSetup.c
    • -12
    • +0
    /firmware/App/Modes/StatePreTxHeparinSetup.h
it will be adjusted when I merge latest staging

it will be adjusted when I merge latest staging

not required anymore, forgot to remove

not required anymore, forgot to remove

I think a comment here is needed. Keep it and maybe mention rinseback alarm responses too.

I think a comment here is needed. Keep it and maybe mention rinseback alarm responses too.

Maybe this should be handled in OperationModes exec since it applies to all confirmations (not just alarm related ones).

Maybe this should be handled in OperationModes exec since it applies to all confirmations (not just alarm related ones).