Controllers

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-13460 addressed the code review comments

  1. … 1 more file in changeset.
This code is un-commented.

This code is un-commented.

Done.

Done.

In in DEN-13834.

In in DEN-13834.

Done.

Done.

Added TODO to un-comment this line of code.

Added TODO to un-comment this line of code.

"FAPI_CHECK_FSM_READY_BUSY" is a macro.

"FAPI_CHECK_FSM_READY_BUSY" is a macro.

Done.

Done.

DEN-13460 updated the heaters driver. Moved the heaters data broadcast structure to the fwcommon so the HD can use it too. Updated the temperature sensors FPGA persistent alarm

Done.

Done.

Merge branch 'DEN-13427-sw_dev-sprint-75_mg' into develop

DEN-13427: DIAL-201. Updated temp sensors broadcast message to include thd RTD.

Merge branch 'DEN-13786-hd_dg_dev-s78_bb' into develop

# Conflicts:

# firmware/App/Controllers/DialysateFlow.c

DEN-13786 DEN-13770 The persistent alarm interface was changed resulting in the immediate turning off of the UVReactors. Changed to correct function call. DEN-13786 DEN-13558 Correct getRsrvrDrainStatus and getRsrvrFillStatus per modes to Flush mode

  1. … 1 more file in changeset.
DEN-13460: make boolean explicit, update function header.

Commented out alarm 196

Align on "*" maybe?

Align on "*" maybe?

What is the value of SEMAPHORE_IN_USE_TIMEOUT_MS? The timeout appears to only be check when getSemaphore is called. Seems like you could acquire the semaphore and the process that acquired the sema...

What is the value of SEMAPHORE_IN_USE_TIMEOUT_MS? The timeout appears to only be check when getSemaphore is called. Seems like you could acquire the semaphore and the process that acquired the semaphore could hold onto it for a time much longer than SEMAPHORE_IN_USE_TIMEOUT.

Should this function handle invalid hex characters?

Should this function handle invalid hex characters?

Set "status" to FALSE.

Set "status" to FALSE.

Should this be removed?

Should this be removed?

Should variables in if be reversed?

Should variables in if be reversed?

Should this be in or not?

Should this be in or not?

Some values have "U"?

Some values have "U"?

Are we moving away from initializing via definition?

Are we moving away from initializing via definition?

Merged DEN-13460

  1. … 2 more files in changeset.
DEN-13460 updated the FPGA persistent alarm functions

  1. … 2 more files in changeset.
DEN-13460 updated the trimmer heater's duty cycle cap control

  1. … 1 more file in changeset.
Remove comment.

Remove comment.

When will it be uncommented?

When will it be uncommented?