dd-firmware / firmware / App / Modes

FPModes

Clone Tools

Stats

Commits this week: 1
Total Committers: 10
Last Commit: 14:33
Commits this week: 1

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 19 Mar
3:47 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
Wednesday 18 Mar
2:30 pm

Sean Nash and Michael Garthwaite

8:30 am

Raghu Kallala

LDT-3603 added condition to reset and start sub mode in signal function

Options
    • -1
    • +1
    ./FPOperationModes.c
    • -0
    • +15
    ./ModePreGenPermeate.c
    • -0
    • +7
    ./ModePreGenPermeateDefeatured.c
    • -0
    • +13
    ./StateInletPressureCheck.c
    • -0
    • +1
    ./StateInletPressureCheck.h
Thursday 12 Mar
1:14 pm

Raghu Kallala

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

Options
    • -1
    • +4
    ./ModeGenPermeateDefeatured.c
    • -1
    • +4
    ./ModePreGenPermeate.c
    • -1
    • +4
    ./ModePreGenPermeateDefeatured.c
9:24 am

Raghu Kallala

LDT-3603 addressed review comments and added missing code

Options
    • -0
    • +62
    ./FPModeFault.c
    • -1
    • +4
    ./FPModeFault.h
    • -0
    • +52
    ./FPModeStandby.c
    • -1
    • +2
    ./FPModeStandby.h
    • -88
    • +103
    ./FPOperationModes.c
    • -48
    • +55
    ./ModeGenPermeate.c
    • -2
    • +2
    ./ModeGenPermeate.h
    • -50
    • +55
    ./ModeGenPermeateDefeatured.c
    • -3
    • +2
    ./ModeGenPermeateDefeatured.h
    • -61
    • +67
    ./ModePreGenPermeate.c
    • -3
    • +3
    ./ModePreGenPermeate.h
    • -51
    • +57
    ./ModePreGenPermeateDefeatured.c
    • -2
    • +2
    ./ModePreGenPermeateDefeatured.h
7:21 am

Raghu Kallala

That's a question for me, will that be required? Fault has sub modes(3), Standby has 1 sub mode(I am adding code for this as this might be required) Init don't have any and service is not defined yet

That's a question for me, will that be required?

Fault has sub modes(3), Standby has 1 sub mode(I am adding code for this as this might be required)
Init don't have any and service is not defined yet

6:37 am

Raghu Kallala

Updated

Updated

6:37 am

Raghu Kallala

Updated

Updated

6:36 am

Raghu Kallala

Changed and left them as static, as we don't need these outside this unit

Changed and left them as static, as we don't need these outside this unit

6:27 am

Raghu Kallala

You are right, (missed that while duplicating the functions)

You are right, (missed that while duplicating the functions)

Wednesday 11 Mar
2:37 pm

Sean Nash

if ( TRUE == result )

if ( TRUE == result )

2:34 pm

Sean Nash

Since both states require no special handling, remove breaks and group with default - all modes.

Since both states require no special handling, remove breaks and group with default - all modes.

2:34 pm

Sean Nash

Remove comment.

Remove comment.

2:31 pm

Sean Nash

Add comment that all of these states require no special handling.

Add comment that all of these states require no special handling.

2:27 pm

Sean Nash

I think these functions (validate and signal) should be "test" functions and moved down to Dialin test functions section below - for all modes.

I think these functions (validate and signal) should be "test" functions and moved down to Dialin test functions section below - for all modes.

2:25 pm

Sean Nash

Add comment that no action is required for these modes - defaulting to invalid request.

Add comment that no action is required for these modes - defaulting to invalid request.

2:24 pm

Sean Nash

I think this switch should be on requested mode - we haven't changed modes yet.

I think this switch should be on requested mode - we haven't changed modes yet.

2:22 pm

Sean Nash

We have 1 param (and probably should have 2 params - requested mode and submode).

We have 1 param (and probably should have 2 params - requested mode and submode).

2:22 pm

Sean Nash

currentMode is an input.

currentMode is an input.

2:21 pm

Sean Nash

I think this validate function needs to know the requested mode as well - should have 2 params.

I think this validate function needs to know the requested mode as well - should have 2 params.

2:18 pm

Sean Nash

Are we not supporting sub mode requests for these modes?

Are we not supporting sub mode requests for these modes?

2:15 pm

Sean Nash

if ( TRUE == testChangeSubMode )

if ( TRUE == testChangeSubMode )

2:13 pm

Sean Nash

I think these 2 functions are in support of Dialin only, so should be prefixed with "test" as well.

I think these 2 functions are in support of Dialin only, so should be prefixed with "test" as well.

Tuesday 10 Mar
2:33 pm

Raghu Kallala

LDT-3603 fixed typo in comments

Options
    • -1
    • +1
    ./ModeGenPermeate.c
    • -1
    • +1
    ./ModeGenPermeateDefeatured.c
    • -1
    • +1
    ./ModePreGenPermeate.c
    • -1
    • +1
    ./ModePreGenPermeateDefeatured.c
1:50 pm

Raghu Kallala

LDT-3603 moved declaration to top of the scope

Options
    • -2
    • +5
    ./FPOperationModes.c
1:46 pm

Raghu Kallala

LDT-3603 added validate before making a request call in dialin test handler

Options
    • -53
    • +111
    ./FPOperationModes.c
    • -3
    • +1
    ./FPOperationModes.h
    • -4
    • +21
    ./ModeGenPermeate.c
    • -0
    • +1
    ./ModeGenPermeate.h
    • -4
    • +21
    ./ModeGenPermeateDefeatured.c
    • -0
    • +1
    ./ModeGenPermeateDefeatured.h
    • -14
    • +22
    ./ModePreGenPermeate.c
    • -0
    • +1
    ./ModePreGenPermeate.h
    • -7
    • +21
    ./ModePreGenPermeateDefeatured.c
    • -0
    • +1
    ./ModePreGenPermeateDefeatured.h
  11. … 1 more file in changeset.
Friday 06 Mar
2:48 pm

Sean Nash

See my comment above. We should return BOOL so Dialin request can be ACK/NAK'd.

See my comment above. We should return BOOL so Dialin request can be ACK/NAK'd.

2:46 pm

Sean Nash

No faults from Dialin commands - we never want Dialin to cause system to go to fault because it requested something invalid - we only NAK the invalid request. We should do whatever is needed (if an...

No faults from Dialin commands - we never want Dialin to cause system to go to fault because it requested something invalid - we only NAK the invalid request.
We should do whatever is needed (if anything) to start the mode normally in an else.
And we should return TRUE/FALSE in these signal functions so that when it's called from Dialin msg handler, the msg handler can return ACK/NAK to Dialin if sub mode is invalid.

2:42 pm

Sean Nash

We shouldn't request op mode until we're sure we're happy with the whole thing. Need to determine whether we are already in the op mode being requested. If so, no need to request it again - just re...

We shouldn't request op mode until we're sure we're happy with the whole thing.
Need to determine whether we are already in the op mode being requested. If so, no need to request it again - just request new sub mode if not already in it. If not, the currentSubMode checked below is not relevant (applies to current mode, not the requested mode) and we need to request new op mode and signal that new op mode that we will want to start in a specific sub mode.

2:38 pm

Sean Nash

Declarations should be at top of scope (right before memcpy).

Declarations should be at top of scope (right before memcpy).