fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'staging' into DEN-15104-hd-fpga-communications

# Conflicts:

# Utilities.h

Use enums instead of literals.

Use enums instead of literals.

Don't we have an enum for these values?

Don't we have an enum for these values?

Is there more to be done here (TODO)?

Is there more to be done here (TODO)?

I think these variable declarations should stay here. Why move to function?

I think these variable declarations should stay here. Why move to function?

This is HD f/w. Why are you looking at DG mode?

This is HD f/w. Why are you looking at DG mode?

Why aren't these declared at top of file and initialized in init function like everything else.

Why aren't these declared at top of file and initialized in init function like everything else.

Merge branch 'staging' into DEN-15104-hd-fpga-communications

Why is this in two places? And why set every 10ms when it appears to be constant?

Why is this in two places? And why set every 10ms when it appears to be constant?

This doesn't seem to ever change. Why not just initialize this value once on reset?

This doesn't seem to ever change. Why not just initialize this value once on reset?

Make U32.

Make U32.

Merge branch 'staging' into DEN-15274-hd_dg_test_configuration_setup

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Merge branch 'staging' into DEN-15246-active-reservoir-empty-post-concentrate-alarm-recovery

Declarations before code.

Declarations before code.

Merge branch 'DEN-15246-active-reservoir-empty-post-concentrate-alarm-recovery' into develop

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Merge branch 'DEN-15196-Alarm-98-rework' into staging

Merge branch 'staging' into DEN-15196-Alarm-98-rework

Should be != TRUE?

Should be != TRUE?

loadcells[] is an output in this function.

loadcells[] is an output in this function.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Keep going. Now 95 is wrong. Need comments to align with actual values throughout.

Keep going. Now 95 is wrong. Need comments to align with actual values throughout.

Done

Done

Then the one above with 100 comment is wrong.

Then the one above with 100 comment is wrong.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.