leahi-common

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT:3970 Addressed review comments for Edit treatment parameters

Can we change the MACRO name to 64 step, if we are using 64 micro steps for HDF?

Can we change the MACRO name to 64 step, if we are using 64 micro steps for HDF?

Do we need to add the D92 pump speed read registers and handle appropriate monitor functions in Substitution Pump controller file?

Do we need to add the D92 pump speed read registers and handle appropriate monitor functions in Substitution Pump controller file?

Done

Done

Done

Done

add a blank line.

add a blank line.

Change to HDF pump. also update D92 instead of D76.

Change to HDF pump. also update D92 instead of D76.

Non-Dialin messages should be higher in this list than Dialin messages. Move this above all of the "test" handlers.

Non-Dialin messages should be higher in this list than Dialin messages. Move this above all of the "test" handlers.

If the check RAM ECC is disabled in the HALCoGen, this code will not be created.

If the check RAM ECC is disabled in the HALCoGen, this code will not be created.

Remove blank line

Remove blank line

If the check RAM ECC is disabled in the HALCoGen, this code will not be created.

If the check RAM ECC is disabled in the HALCoGen, this code will not be created.

If you disable mpu in the HALCoGen, this code will not be created.

If you disable mpu in the HALCoGen, this code will not be created.

This is auto generated code. How do we keep this code from re-appearing when somebody regenerates this code?

This is auto generated code. How do we keep this code from re-appearing when somebody regenerates this code?

same.

same.

same.

same.

We shouldn't delete Halcogen code. Add a #if 0 in this user code section and add a #endif in the user code section following _coreEnableFlashEcc() call.

We shouldn't delete Halcogen code.
Add a #if 0 in this user code section and add a #endif in the user code section following _coreEnableFlashEcc() call.

Should this be commented out before merge?

Should this be commented out before merge?

These are not in alphabetical order. See ModeStandby and ModeService and ModeFault.

These are not in alphabetical order. See ModeStandby and ModeService and ModeFault.

These should come before PC channels (higher priority).

These should come before PC channels (higher priority).

Done

Done

Done

Done

Done

Done

LDT-3259 merged staging and resolved the conflicts

Alphabetize.

Alphabetize.

Comment this out before merge.

Comment this out before merge.

Comment this out before merge.

Comment this out before merge.

LDT-2240-disposable-tube-set-self-test

Yes. The integrity module is in leahi-fwcommon repository, and it serves both TD and DD.

Yes. The integrity module is in leahi-fwcommon repository, and it serves both TD and DD.

These are located in the integrity.c

These are located in the integrity.c

Yes the ECC checks are done in the bootloader stack.

Yes the ECC checks are done in the bootloader stack.