hdfirmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

fixed. set to no sub state (0xFF) once we transition to pretx recirc

fixed. set to no sub state (0xFF) once we transition to pretx recirc

Looks like this is only being set in wet self-tests substate of pre-tx prime. Once we leave wet self-tests, should this 4th level state be set back to zero or something?

Looks like this is only being set in wet self-tests substate of pre-tx prime. Once we leave wet self-tests, should this 4th level state be set back to zero or something?

Seems like we should be calling CPLD function here and in Fans, DialInFlow, and DGInterface too. Also recommend calling the isACPowerLost() function in just one place - maybe alarm mgmt exec to gi...

Seems like we should be calling CPLD function here and in Fans, DialInFlow, and DGInterface too.

Also recommend calling the isACPowerLost() function in just one place - maybe alarm mgmt exec to give you your 10 sec alarm block. Maybe rename the function to checkACPowerLost() with no return value and make it static (private).

It is a timer counter. Counts down to zero. And still need to address function header per Bill's first comment.

It is a timer counter. Counts down to zero.

And still need to address function header per Bill's first comment.

This seems unnecessary now with your new function doing this all the time. Consider removing.

This seems unnecessary now with your new function doing this all the time. Consider removing.

This is technically a timer counter based on task tick of 50ms.

This is technically a timer counter based on task tick of 50ms.

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

This was safer the other way. Why change?

How is this different than checking CPLD? Condition is cleared when CPLD no longer sees power loss (w/ 150ms delay I guess). Seems simpler to use CPLD function instead of checking 2 alarms.

How is this different than checking CPLD? Condition is cleared when CPLD no longer sees power loss (w/ 150ms delay I guess). Seems simpler to use CPLD function instead of checking 2 alarms.

Remove extra blank line.

Remove extra blank line.

HD-DEN-15317_Usability HD Door Open Alarm During Post Treatment
HD-DEN-15317_Usability HD Door Open Alarm During Post Treatment
A 10 second timeout timer, driven by the 50ms tick that drives the alarm state machine.

A 10 second timeout timer, driven by the 50ms tick that drives the alarm state machine.

HD-DEN-15451_Dpi Pump Does Not Resume After Door Opened Alarm Cleared IN Pre TX
HD-DEN-15451_Dpi Pump Does Not Resume After Door Opened Alarm Cleared IN Pre TX
FIXED IN CODE WALKTHROUGH

FIXED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

Add alarmsBlockedTimer to header. Is alarmsBlockedTimer a timer or is it a counter?

Add alarmsBlockedTimer to header. Is alarmsBlockedTimer a timer or is it a counter?

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Add current4thLevleState to header.

Add current4thLevleState to header.

Add current4thLevelState to header

Add current4thLevelState to header