dgfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Should be s/w fault. Add TODO or trigger s/w fault now.

Should be s/w fault. Add TODO or trigger s/w fault now.

Blank line between declarations and code.

Blank line between declarations and code.

This assignment should be done above at declaration. Move calStatus declaration above result.

This assignment should be done above at declaration. Move calStatus declaration above result.

Keep this blank line.

Keep this blank line.

Do either of these 2 if statements need an else?

Do either of these 2 if statements need an else?

Blank line between declarations and code.

Blank line between declarations and code.

Prefer normal if else for calling functions. I see these throughout.

Prefer normal if else for calling functions. I see these throughout.

What is the point in having these locals? Why not just use the concentratePumps fields directly?

What is the point in having these locals? Why not just use the concentratePumps fields directly?

This blank line should stay - style is to have blank lines between declarations and code.

This blank line should stay - style is to have blank lines between declarations and code.

Define disable (even if not used) so it is clear which bit position controls enable/disable.

Define disable (even if not used) so it is clear which bit position controls enable/disable.

DG-DEN-12931_DG HD Dev HD DG Dvt Update
DG-DEN-12931_DG HD Dev HD DG Dvt Update
That would also require a change to the requirements.

That would also require a change to the requirements.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Alarm is not a fault. Updated clear immediate to false in AlarmDefs.h in Staging.

Alarm is not a fault. Updated clear immediate to false in AlarmDefs.h in Staging.

10-20 seems like a better number, but my sample size is small so I did not change it.

10-20 seems like a better number, but my sample size is small so I did not change it.

Updated

Updated

Updated

Updated

Seems like isADCValid should be returned.

Seems like isADCValid should be returned.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.

This will be addressed in DEN-12931.