fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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.

testConfig is an input, not an output.

testConfig is an input, not an output.

Change "release" to "test".

Change "release" to "test".

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.

Add check for Dialin logged in.

Add check for Dialin logged in.

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.

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

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

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.

Add blank line between banner and #define.

Add blank line between banner and #define.

Add blank line between banner and #define.

Add blank line between banner and #define.

Are DG and HD sharing this enum?

Are DG and HD sharing this enum?

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.

DG-DEN-15199_FW DN Sprint
DG-DEN-15199_FW DN Sprint
HD-DEN-15104_HD Fpga Communications
HD-DEN-15104_HD Fpga Communications
RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

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.

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.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

What update is required?

What update is required?

Fixed.

Fixed.

Fixed.

Fixed.

Update header, setUFVolStatus

Update header, setUFVolStatus

Update header

Update header

Update header.

Update header.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH