fwcommon

Clone Tools

Stats

Commits this week: 0
Total Committers: 13
Last Commit: 10 Jun
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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 21 Apr 2023
2:59 pm

Dara Navaei

Done

Done

2:46 pm

Dara Navaei

Correct. This common code in between the two stacks. I can separate them using #ifndef.

Correct. This common code in between the two stacks. I can separate them using #ifndef.

2:45 pm

Dara Navaei

Done

Done

2:45 pm

Dara Navaei

Done

Done

2:45 pm

Dara Navaei

I removed the memset macro.

I removed the memset macro.

2:44 pm

Dara Navaei

This function will not even be called if Dialin is not logged in. Did you want to check it here too?

This function will not even be called if Dialin is not logged in. Did you want to check it here too?

2:43 pm

Dara Navaei

Done

Done

2:43 pm

Dara Navaei

Done

Done

2:39 pm

Dara Navaei

This function is a Dialin command so it will not even be called if Dialin is not logged in. Did you want to check this again in this function?

This function is a Dialin command so it will not even be called if Dialin is not logged in. Did you want to check this again in this function?

11:41 am

Sean Nash

Note in brief that function will return FALSE if Dialin not logged in.

Note in brief that function will return FALSE if Dialin not logged in.

11:40 am

Sean Nash

testConfig is an input, not an output.

testConfig is an input, not an output.

11:39 am

Sean Nash

Change "release" to "test".

Change "release" to "test".

11:38 am

Sean Nash

Yes, s/w fault (for invalid config). Ignore command for not logged in.

Yes, s/w fault (for invalid config). Ignore command for not logged in.

11:37 am

Sean Nash

Add check for Dialin logged in.

Add check for Dialin logged in.

11:36 am

Sean Nash

Yes, s/w fault (for invalid config). Ignore command for not logged in.

Yes, s/w fault (for invalid config). Ignore command for not logged in.

11:34 am

Sean Nash

Also, Dialin must be logged in to accept a test config command.

Also, Dialin must be logged in to accept a test config command.

11:32 am

Sean Nash

memset wants a byte. Just say 0 instead of using KEY #define. Also, TEST_CONFIG_T is an enum, so size is probably 1 and so this probably won't initialize whole array.

memset wants a byte. Just say 0 instead of using KEY #define.
Also, TEST_CONFIG_T is an enum, so size is probably 1 and so this probably won't initialize whole array.

11:26 am

Sean Nash

Add blank line between banner and #define.

Add blank line between banner and #define.

11:26 am

Sean Nash

Add blank line between banner and #define.

Add blank line between banner and #define.

11:25 am

Sean Nash

Are DG and HD sharing this enum?

Are DG and HD sharing this enum?

11:17 am

Sean Nash

We want test configs be active only if explicitly enabled. Safest condition would be to say ( TRUE == getTest.... ). Make this change everywhere getTestConfigStatus function is used.

We want test configs be active only if explicitly enabled. Safest condition would be to say ( TRUE == getTest.... ).
Make this change everywhere getTestConfigStatus function is used.

Wednesday 19 Apr 2023
4:17 pm

Dara Navaei

DG-DEN-15199_FW DN Sprint
DG-DEN-15199_FW DN Sprint
2:25 pm

Michael Garthwaite

Monday 17 Apr 2023
10:26 am

Sean Nash

10:25 am

jtaylor  (deleted user) and wbracken  (deleted user)

Friday 14 Apr 2023
6:01 pm

Dara Navaei

HD-DEN-15104_HD Fpga Communications
HD-DEN-15104_HD Fpga Communications
9:23 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

9:21 am

Sean Nash

For init functions, I typically do not itemize variables (list too long) and just say initialize module.

For init functions, I typically do not itemize variables (list too long) and just say initialize module.

9:13 am

wbracken  (deleted user)

Variables in lines 153 to 165 should be in header... unless pre-treatment mode module initialized covers them.

Variables in lines 153 to 165 should be in header... unless pre-treatment mode module initialized covers them.

9:13 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH