dd-firmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-3372: code review comments

fixed.

fixed.

fixed.

fixed.

fixed.

fixed.

LDT-3372: code review comments

J1 - P9 J2 - P18 J3 - D17 J4 - D27 J5 - D29 J6 - D43 No sensor - D74 (redundant, so we should be able to live without it in short run)

J1 - P9
J2 - P18
J3 - D17
J4 - D27
J5 - D29
J6 - D43
No sensor - D74 (redundant, so we should be able to live without it in short run)

Remove extra blank line.

Remove extra blank line.

build switch fixes will be completed in the v2 conductivity sensor branch. Per what was agreed on back in November: J1 - P9 J2 - P18 J3 - D17 J4 - D27 J5 - D29 J6 - D43 No sensor - D74 (redundant,...

build switch fixes will be completed in the v2 conductivity sensor branch. Per what was agreed on back in November:

J1 - P9
J2 - P18
J3 - D17
J4 - D27
J5 - D29
J6 - D43
No sensor - D74 (redundant, so we should be able to live without it in short run)

Remove extra blank line.

Remove extra blank line.

LDT-3372: code review comments

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Add blank line after declarations.

Add blank line after declarations.

Same question.

Same question.

Why is D74 not a Teensy? And I though you were getting ride of this build switch and using test config (run time switch)?

Why is D74 not a Teensy? And I though you were getting ride of this build switch and using test config (run time switch)?

Remove extra blank line.

Remove extra blank line.

LDT-3103 merget latest staging

    • -26
    • +78
    /firmware/App/Modes/ModeGenDialysate.c
    • -2
    • +3
    /firmware/App/Modes/ModeGenDialysate.h
LDT-3103 implemented new pump changes

    • -8
    • +25
    /firmware/App/Modes/ModeGenDialysate.c
LDT-3218: fp tuning. WIP

Shall I add the SW_Fault as an else case if an invalid submode was requested for an opmode? Sean NashMichael Garthwaite please confirm

Shall I add the SW_Fault as an else case if an invalid submode was requested for an opmode?
Sean NashMichael Garthwaite please confirm

LDT-3603 converted all signal functions to void functions

    • -6
    • +2
    /firmware/App/Modes/FPModes/ModeGenPermeate.c
    • -1
    • +1
    /firmware/App/Modes/FPModes/ModeGenPermeate.h
I was thinking we could return BOOL to check if the request was successful, added it. But as you clarified the dialin response will only confirm the delivery of message successfully, I can change ...

I was thinking we could return BOOL to check if the request was successful, added it.

But as you clarified the dialin response will only confirm the delivery of message successfully, I can change all these signal functions to void
Thanks!

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

Merge branch 'LDT-3372-v1-conductivity-fixes' into develop

WIP on develop

LDT-3372: fixed error on release builds

Merge branch 'staging' into LDT-3372-add-ability-to-override-teensy

Merge branch 'staging' into LDT-3218-update-iofp-permeate-target-flow

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

Merge branch 'staging' into LDT-2004-v2-conductivity-sensors

Merge branch 'staging' into LDT-3372-v1-conductivity-fixes