dd-firmware / firmware / App / Controllers

ConcentratePumps.h

Clone Tools

Stats

Commits this week: 2
Total Committers: 5
Last Commit: 25 Nov 24
Commits this week: 2

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
Today
1 hr ago

Vendor - TEL - Sameer Poyil

Removing it now. will introduce it later if required

Removing it now. will introduce it later if required

1 hr ago

Vendor - TEL - Sameer Poyil

will finalize it when ocean spray logic is tested

will finalize it when ocean spray logic is tested

1 hr ago

Vendor - TEL - Sameer Poyil

dryBicartState is top level state machine which is different data type. removed dryBicartSubstate

dryBicartState is top level state machine which is different data type. removed dryBicartSubstate

1 hr ago

Vendor - TEL - Sameer Poyil

removed

removed

1 hr ago

Vendor - TEL - Sameer Poyil

This is not extra blank line its a line for TASK_GENERAL_INTERVAL )

This is not extra blank line its a line for TASK_GENERAL_INTERVAL )

1 hr ago

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

1 hr ago

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 hrs ago

Sean Nash

Remove this blank line.

Remove this blank line.

2 hrs ago

Sean Nash

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

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

2 hrs ago

Sean Nash

Remove blank line.

Remove blank line.

2 hrs ago

Sean Nash

Remove blank line.

Remove blank line.

2 hrs ago

Sean Nash

Add a blank line after declarations.

Add a blank line after declarations.

2 hrs ago

Sean Nash

Remove blank line.

Remove blank line.

2 hrs ago

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 hrs ago

Sean Nash

Remove extra blank lines.

Remove extra blank lines.

2 hrs ago

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?

3 hrs ago

Sean Nash

Why do we need this?

Why do we need this?

3 hrs ago

Sean Nash

Remove extra blank line.

Remove extra blank line.

3 hrs ago

Sean Nash

Remove extra blank line.

Remove extra blank line.

3 hrs ago

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.

3 hrs ago

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?

3 hrs ago

Sean Nash

Let's remove this commented line.

Let's remove this commented line.

6 hrs ago

Vendor - TEL - Sameer Poyil

I don't need to edit this file. I will check with Dara to exclude this file from code review.

I don't need to edit this file. I will check with Dara to exclude this file from code review.

6 hrs ago

Vendor - TEL - Sameer Poyil

only maximum 30 float data is allowed. If that is the case then I cannot broadcast the acid or bicarb control signals. need to split it or need two message id.

only maximum 30 float data is allowed. If that is the case then I cannot broadcast the acid or bicarb control signals.
need to split it or need two message id.

6 hrs ago

Vendor - TEL - Sameer Poyil

Its a problem . Vinay has to decide it

Its a problem . Vinay has to decide it

Thursday 02 Apr
10:22 am

Vendor - TEL - Jashwant Gantyada

10:02 am

Vinayakam Mani

Wednesday 01 Apr
11:34 am

Sean Nash

Algin "=" to be consistent with those below.

Algin "=" to be consistent with those below.

11:34 am

Sean Nash

Add blank line after declarations.

Add blank line after declarations.

11:33 am

Sean Nash

Add blank line after declarations.

Add blank line after declarations.