dd-firmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-2004: reverted 2.0 fpga init changes.

LDT-2004: override fixes for v1 and v2 sensors.

    • -135
    • +12
    /firmware/App/Drivers/ConductivitySensors.c
    • -0
    • +73
    /firmware/App/Monitors/Conductivity.c
FALSE?

FALSE?

if fresh and spent pressure is not met after certain time, we should trigger an alarm and not to stuck in this state forever.

if fresh and spent pressure is not met after certain time, we should trigger an alarm and not to stuck in this state forever.

are we using this variable, if not, can we remove across the file?

are we using this variable, if not, can we remove across the file?

Add comment

Add comment

Remove the unused macros.

Remove the unused macros.

fixed.

fixed.

added.

added.

deleted

deleted

done.

done.

done.

done.

changed and updated all the other places.

changed and updated all the other places.

changed.

changed.

handled accordingly.

handled accordingly.

done

done

done

done

done

done

done.

done.

LDT-3603 changed code to return false when pause state is requested.

LDT-3103 addressed code review comments and changes

    • -3
    • +16
    /firmware/App/Modes/ModeGenDialysate.c
LDT-2004: fixes to v2 conductivity sensors during unit 5 bring up

I don't see a TODO. If we intend to uncomment this line out later, we should add a TODO comment to the right as a reminder. If we do not intend to uncomment this later, we should just delete it.

I don't see a TODO. If we intend to uncomment this line out later, we should add a TODO comment to the right as a reminder. If we do not intend to uncomment this later, we should just delete it.

Fix alignment of "=".

Fix alignment of "=".

Add a param line in this header.

Add a param line in this header.

Can we delete this commented out line?

Can we delete this commented out line?

Add a params line in function header.

Add a params line in function header.

Why did we change this spelling? Now it's wrong.

Why did we change this spelling? Now it's wrong.

Can we delete this commented out line?

Can we delete this commented out line?

Keep this blank line.

Keep this blank line.