dd-firmware / firmware

App

Clone Tools

Stats

Commits this week: 37
Total Committers: 16
Last Commit: 16:11
Commits this week: 37

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
Tuesday 14 Apr
3:44 pm

Vinayakam Mani

LDT-3352 code review comments fix

Options
    • -1
    • +1
    ./Services/AlarmMgmtSWFaults.h
    • -1
    • +1
    ./Services/Messaging.c
3:33 pm

Vendor - TEL - Sameer Poyil

a macro used to create a contiguous bitmask starting at bit position l (low) and ending at bit position h

a macro used to create a contiguous bitmask starting at bit position l (low) and ending at bit position h

3:30 pm

Vendor - TEL - Sameer Poyil

nr means short form of bit 'number', unsigned long is portable, 32bit for 32-bit cpu and 64 bits for 64bit cpu whereas unsigned int is always 32 bits regardless of architecture.

nr means short form of bit 'number', unsigned long is portable, 32bit for 32-bit cpu and 64 bits for 64bit cpu whereas unsigned int is always 32 bits regardless of architecture.

2:35 pm

Sean Nash

Remove this blank line.

Remove this blank line.

2:35 pm

Sean Nash

Put () around each of the 2 conditions being ANDed.

Put () around each of the 2 conditions being ANDed.

2:34 pm

Sean Nash

Remove blank line.

Remove blank line.

2:33 pm

Sean Nash

Remove blank line.

Remove blank line.

2:31 pm

Sean Nash

Add a blank line after declarations.

Add a blank line after declarations.

2:31 pm

Sean Nash

Remove blank line.

Remove blank line.

2:31 pm

Sean Nash

Should we remove this else if or is it going to be restored later?

Should we remove this else if or is it going to be restored later?

2:27 pm

Sean Nash

Remove extra blank lines.

Remove extra blank lines.

2:26 pm

Sean Nash

Shouldn't we be assigning dryBicartState instead of dryBicartSubstate in all of these cases?

Shouldn't we be assigning dryBicartState instead of dryBicartSubstate in all of these cases?

2:26 pm

Sean Nash

Why do we need this?

Why do we need this?

2:24 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

2:23 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

2:19 pm

Sean Nash

Not clear what GENMASK is doing. What do h and l refer to. This doesn't look like a simple bit mask.

Not clear what GENMASK is doing. What do h and l refer to. This doesn't look like a simple bit mask.

2:16 pm

Sean Nash

Is this for 64 bit unsigned integer? If not, this is redundant with BIT_BY_POS above. What does nr stand for?

Is this for 64 bit unsigned integer? If not, this is redundant with BIT_BY_POS above.
What does nr stand for?

2:14 pm

Sean Nash

Let's remove this commented line.

Let's remove this commented line.

2:08 pm

Sean Nash

Add blank line before return.

Add blank line before return.

2:07 pm

Sean Nash

Add blank line after declarations.

Add blank line after declarations.

2:05 pm

Sean Nash

Why U32? Why not RECOVER_STATE_T?

Why U32? Why not RECOVER_STATE_T?

2:04 pm

Sean Nash

Lower case x

Lower case x

2:02 pm

Sean Nash

Is blood flow in need of a PID controller too?

Is blood flow in need of a PID controller too?

1:46 pm

Sean Nash

2 blank lines before test support banner

2 blank lines before test support banner

1:46 pm

Sean Nash

Add blank line before return.

Add blank line before return.

1:46 pm

Sean Nash

Add blank line before return.

Add blank line before return.

11:49 am

Vendor - TEL - Sameer Poyil

LDT-2004-Dry bicarb closed loop control

Options
    • -4
    • +4
    ./Services/Messaging.c
11:40 am

Vendor - TEL - Sameer Poyil

LDT-2004 dry bicart closed loop control

Options
    • -0
    • +3
    ./Controllers/BalancingChamber.c
    • -1
    • +1
    ./Controllers/ConcentratePumps.h
    • -77
    • +72
    ./Controllers/DryBiCart.c
    • -5
    • +4
    ./Services/Messaging.c
11:25 am

Michael Garthwaite

LDT-3218: fixed valve config call

Options
    • -1
    • +2
    ./Controllers/ROPump.c
    • -1
    • +0
    ./Controllers/Valves.h
11:23 am

Raghu Kallala

LDT-3904 added FP operation status event

Options
    • -2
    • +31
    ./Modes/FPModes/FPOperationModes.c