hdfirmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-12224 updated the syscomm messages

DEN-12224 updated the files

    • -398
    • +398
    /firmware/App/Controllers/Buttons.c
    • -1349
    • +1349
    /firmware/App/Services/SystemCommMessages.c
DEN-12224 aligned the spaces in between the variables and datatypes

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Why isn't this command allowed for release version?

Why isn't this command allowed for release version?

Is this zero temporary? If so, add TODO.

Is this zero temporary? If so, add TODO.

DEN-12847 Add broadcast debug message for collecting stabel load cell drift data.

I don't think we should exit normal mode in TreatmentStop sub-mode.

I don't think we should exit normal mode in TreatmentStop sub-mode.

Should we exit blood leak normal mode in here somewhere?

Should we exit blood leak normal mode in here somewhere?

Ask Behrouz if UI is still looking for this. Remove if not.

Ask Behrouz if UI is still looking for this. Remove if not.

undoing DG messaging for door switches

Remove extra blank line. And why is there any blank line here?

Remove extra blank line. And why is there any blank line here?

Seems like we're setting a start time offset here, not a time stamp.

Seems like we're setting a start time offset here, not a time stamp.

Should be rpmAlarmStartTime

Should be rpmAlarmStartTime

Did we want this offset to revert to zero here or should Dialin have to do a reset?

Did we want this offset to revert to zero here or should Dialin have to do a reset?

I don't think this results in an elapsed time. It's just a different start time. The calcTimeSince below gives you the elapsed time.

I don't think this results in an elapsed time. It's just a different start time. The calcTimeSince below gives you the elapsed time.

DEN-12845 resolved merge conflicts during cherry picking process

    • -0
    • +98
    /firmware/App/Controllers/Buttons.c
    • -1
    • +12
    /firmware/App/Controllers/Switches.c
    • -0
    • +5
    /firmware/App/Controllers/SyringePump.c
DEN-12847 Load cell drift check for stable reservoirs at full and empty.

    • -0
    • +32
    /firmware/App/Controllers/DGInterface.c
    • -0
    • +2
    /firmware/App/Controllers/DGInterface.h
Merge branch 'DEN-12224-dg_hd_dev-switches-monitor' of ssh://dvm-linux02:7999/hd/hdfirmware into DEN-12224-dg_hd_dev-switches-monitor

DEN-12224 updated the messages

Should we set this flag only if we are in normal state?

Should we set this flag only if we are in normal state?

Remove extra blank line.

Remove extra blank line.

Should this be set to FALSE?

Should this be set to FALSE?

"reset" should be "rest" I think.

"reset" should be "rest" I think.

Agree. Dara, make these look something like "C_CALIBRATION_CMD".

Agree. Dara, make these look something like "C_CALIBRATION_CMD".

We will want to tighten these eventually.

We will want to tighten these eventually.

It is used on lines 302, 1172, 1795, 1875.

It is used on lines 302, 1172, 1795, 1875.

We want release s/w to be fully enabled with no special test code. SW configs are structured such that a disabled config is what is wanted for release s/w and so forcing all s/w configs to be disab...

We want release s/w to be fully enabled with no special test code. SW configs are structured such that a disabled config is what is wanted for release s/w and so forcing all s/w configs to be disabled in a release build ensures we don't accidentally leave a config in the wrong state.