hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'DEN-13834-dg_hd_dev-hd_dg_dvt-update-part-3' into develop

DEN-13834 updated the semaphore

Is this request for DG only? If so, rename enum to MSG_ID_REQUEST_DG_CPLD_STATUS.

Is this request for DG only? If so, rename enum to MSG_ID_REQUEST_DG_CPLD_STATUS.

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merged DEN-13834

    • -7
    • +12
    /firmware/App/Controllers/DGInterface.c
    • -91
    • +36
    /firmware/App/Controllers/PresOccl.c
    • -17
    • +9
    /firmware/App/Controllers/SyringePump.c
DEN-13834 updated the FPGA persistent alarms

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

DEN-14001-hd-battery-driver-update. Addressed remaining code review comments. Changed I2C timeout from 1 ms to 2 ms. Updated headers in some functions. Removed extra blank lines.

BatteryStatusData is also an input

BatteryStatusData is also an input

Add 2 blank lines before test support banner.

Add 2 blank lines before test support banner.

Remove extra blank lines. Should only be 1 here.

Remove extra blank lines. Should only be 1 here.

Cases should be pulled back one indent (4 spaces) to align with others.

Cases should be pulled back one indent (4 spaces) to align with others.

Add blank line after declaration.

Add blank line after declaration.

Sort of. You will get something between the ms specified and 1 ms less because you don't know where you are in a given ms when you get your start time. Recommend changing this to 2 ms timeout.

Sort of. You will get something between the ms specified and 1 ms less because you don't know where you are in a given ms when you get your start time.
Recommend changing this to 2 ms timeout.

Why not checking payload length?

Why not checking payload length?

Keep blank line.

Keep blank line.

These FPGA registers are obsolete - h/w is no longer in our design. Need to continue to maintain register space since registers after these have not moved. You can think of these obsolete registers...

These FPGA registers are obsolete - h/w is no longer in our design. Need to continue to maintain register space since registers after these have not moved. You can think of these obsolete registers as "reserved" for future purpose.
Dara, I do think we could rename these register fields to something like Reserved1, Reserved2, ...

Monitor execs should be before operation modes exec.

Monitor execs should be before operation modes exec.

Why was blank line removed?

Why was blank line removed?

Align "=".

Align "=".

Add another blank line here.

Add another blank line here.

Idea is to re-purpose these (instead of adding new alarms to bottom of enum). We prefer not to collapse the table when an alarm ID goes obsolete so other alarm IDs do not change.

Idea is to re-purpose these (instead of adding new alarms to bottom of enum). We prefer not to collapse the table when an alarm ID goes obsolete so other alarm IDs do not change.

DEN-14001: Update battery driver.

DEN-14001: Updated Battery.c and Battery.h

    • -178
    • +203
    /firmware/App/Drivers/Battery.c
DEN-14001: Update Battery.c

Does didTimeout support a 1ms timeout?

Does didTimeout support a 1ms timeout?

What is the significance of (OBSOLETE)? Applies to several blocks.

What is the significance of (OBSOLETE)? Applies to several blocks.

Merge branch 'DEN-13903-sprint-80_mg_air_trap_optimization' into develop

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merge branch 'DEN-13903-s80-mg-DIAL-204' into develop

    • -1
    • +1
    /firmware/App/Controllers/DialOutFlow.c