hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Removed

Removed

Added space. I modified slightly, Since this is called outside of the interrupt where the sci2FEOEError is being set I believe it should get the value and only reset it if was TRUE.

Added space. I modified slightly, Since this is called outside of the interrupt where the sci2FEOEError is being set I believe it should get the value and only reset it if was TRUE.

They were used when FE OE generated software fault. Removed.

They were used when FE OE generated software fault. Removed.

Removed.

Removed.

Cut and paste error from DG. Corrected.

Cut and paste error from DG. Corrected.

Removed. Replaced fpgaReadByteSize with sizeof( FPGA_SENSORS_T ) in the two statements that it was being used.

Removed. Replaced fpgaReadByteSize with sizeof( FPGA_SENSORS_T ) in the two statements that it was being used.

I don't see these two functions in the .c file. Are they not yet implemented?

I don't see these two functions in the .c file. Are they not yet implemented?

Why 2 of these?

Why 2 of these?

What does this do? Why 64 (not 32)?

What does this do? Why 64 (not 32)?

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Nothing else needs to be done.

Nothing else needs to be done.

HD-DEN-15196_DIAL-251
HD-DEN-15196_DIAL-251
With that approach, we are likely to have pump/heater off for some time which will allow the dialysate to cool and perhaps cause a temperature alarm when we do start treatment. I think we should ha...

With that approach, we are likely to have pump/heater off for some time which will allow the dialysate to cool and perhaps cause a temperature alarm when we do start treatment.
I think we should have the pump/heater on when door is closed in patient connect states.

Next state only requires door closed after confirm (when state is done). In meantime, door can be opened while pump is on and no alarm will occur and pump stays on I would think.

Next state only requires door closed after confirm (when state is done). In meantime, door can be opened while pump is on and no alarm will occur and pump stays on I would think.

Remove blank line.

Remove blank line.

Do we need to check for these in out exec?

Do we need to check for these in out exec?

Are these used anywhere?

Are these used anywhere?

Add space between ( and TRUE. Or just remove condition and always set flag to FALSE here.

Add space between ( and TRUE. Or just remove condition and always set flag to FALSE here.

Remove blank line.

Remove blank line.

Use enums instead of literals.

Use enums instead of literals.

Don't we have an enum for these values?

Don't we have an enum for these values?

Is there more to be done here (TODO)?

Is there more to be done here (TODO)?

I think these variable declarations should stay here. Why move to function?

I think these variable declarations should stay here. Why move to function?

it is done in the next state - user confirm. May be a better idea to move this pump & heater on to that state after user confirm.

it is done in the next state - user confirm. May be a better idea to move this pump & heater on to that state after user confirm.

This is HD f/w. Why are you looking at DG mode?

This is HD f/w. Why are you looking at DG mode?

I did not want to have the pump turning on/off if opened more than once. Stop while the user performs the action, then turn on when they are done. Maybe wait to turn on until user confirm.

I did not want to have the pump turning on/off if opened more than once. Stop while the user performs the action, then turn on when they are done. Maybe wait to turn on until user confirm.

Why aren't these declared at top of file and initialized in init function like everything else.

Why aren't these declared at top of file and initialized in init function like everything else.