dd-firmware / firmware / include

sci.h

Clone Tools

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 07 Jul
Commits this week: 0

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 16 Dec
1:25 pm

Vendor - TEL - Sameer Poyil

marking it as resolved since it is temporary driver

marking it as resolved since it is temporary driver

11:51 am

Vendor - TEL - Sameer Poyil

As per Sean's comment mark it as resolved

As per Sean's comment mark it as resolved

11:48 am

Vendor - TEL - Sameer Poyil

marking it as resolved since this is a temporary driver

marking it as resolved since this is a temporary driver

11:48 am

Vendor - TEL - Sameer Poyil

I am marking it as resolved since it is temporary driver

I am marking it as resolved since it is temporary driver

11:45 am

Vendor - TEL - Sameer Poyil

Its like we need to compile the safe library and include it. But if we are using only memcpy and memset, memcmp and static code analyzer check the destination buffer size , I am marking it as resolved

Its like we need to compile the safe library and include it. But if we are using only memcpy and memset, memcmp and static code analyzer check the destination buffer size , I am marking it as resolved

Monday 01 Dec
9:26 pm

Sean Nash

Why two output lines?

Why two output lines?

9:25 pm

Sean Nash

All cases seem a little off on indents, especially this one.

All cases seem a little off on indents, especially this one.

9:24 pm

Sean Nash

For default cases (throughout this unit), we either need a s/w fault (because this should never happen and indicates memory corruption) or a comment noting it is ok to ignore invalid cases (command...

For default cases (throughout this unit), we either need a s/w fault (because this should never happen and indicates memory corruption) or a comment noting it is ok to ignore invalid cases (commands in this case).

Friday 21 Nov
10:07 am

Sean Nash

Should we be calling get function in monitor instead of driver here?

Should we be calling get function in monitor instead of driver here?

10:06 am

Sean Nash

No overrides?

No overrides?

10:05 am

Sean Nash

Fix indent.

Fix indent.

10:05 am

Sean Nash

Use a #define instead of 1000.

Use a #define instead of 1000.

10:02 am

Sean Nash

If no parameters, just don't have any param lines (don't say none).

If no parameters, just don't have any param lines (don't say none).

10:01 am

Sean Nash

No param, so remove this line.

No param, so remove this line.

10:01 am

Sean Nash

measurementSettingsTX is an output.

measurementSettingsTX is an output.

10:00 am

Sean Nash

Seems like eePromDataTX is an output, not an input.

Seems like eePromDataTX is an output, not an input.

10:00 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:59 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:58 am

Sean Nash

Add comment to right.

Add comment to right.

9:57 am

Sean Nash

///

///

9:57 am

Sean Nash

Add space between } and type name.

Add space between } and type name.

9:56 am

Sean Nash

///

///

9:56 am

Sean Nash

Add space between } and type name.

Add space between } and type name.

9:56 am

Sean Nash

///

///

9:56 am

Sean Nash

Add space between } and type name.

Add space between } and type name.

9:55 am

Sean Nash

///

///

9:55 am

Sean Nash

///

///

9:55 am

Sean Nash

Use /// here so doxygen will pick it up.

Use /// here so doxygen will pick it up.

Thursday 20 Nov
9:23 am

Sean Nash

Comment seems backward. Uncommenting would enable the Teensy driver, right?

Comment seems backward. Uncommenting would enable the Teensy driver, right?

9:22 am

Sean Nash

Are we not supporting overrides with TEENSY driver?

Are we not supporting overrides with TEENSY driver?