fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'DEN-5980-non-volatile-data-management-2-of-2' of ssh://dvm-linux02:7999/comm/fwcommon into DEN-5980-non-volatile-data-management-2-of-2

DEN-5980 addressed code review comments

This section is monitoring the RPM of the drain pump and makes decision accordingly. This must be commented out until we figure out how to convert counts into RPM properly. I added a TODO note for ...

This section is monitoring the RPM of the drain pump and makes decision accordingly. This must be commented out until we figure out how to convert counts into RPM properly. I added a TODO note for that.

Done.

Done.

Yes. This state machine is only used when a record data is scheduled to be sent to Dialin in batches.

Yes. This state machine is only used when a record data is scheduled to be sent to Dialin in batches.

Changed them to NVDataMgmt NVDataMgmt.

Changed them to NVDataMgmt NVDataMgmt.

These string fields are not null terminated. We will used them as characters and they are not used anywhere in the firmware.

These string fields are not null terminated. We will used them as characters and they are not used anywhere in the firmware.

I wanted to use enum specifically for calibration purposes. Their names also contain the word CAL.

I wanted to use enum specifically for calibration purposes. Their names also contain the word CAL.

Done.

Done.

Done.

Done.

Done.

Done.

I updated the comment.

I updated the comment.

Done.

Done.

Done.

Done.

To make sure dequeuing is not interrupted.

To make sure dequeuing is not interrupted.

Done.

Done.

It is volts. Everywhere else we use abbreviated units of measure as suffix (e.g. _ML).

It is volts. Everywhere else we use abbreviated units of measure as suffix (e.g. _ML).

What does the V stand for in the name of this function? Can we be more explicit? If voltage, then "Volt".

What does the V stand for in the name of this function? Can we be more explicit? If voltage, then "Volt".

Are we getting load cell weight in other units somewhere else? If not, I would suggest defining the units in the function header and removing it from the name of the function.

Are we getting load cell weight in other units somewhere else? If not, I would suggest defining the units in the function header and removing it from the name of the function.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Why do we need to disable interrupt? Is it a critical function?

Why do we need to disable interrupt? Is it a critical function?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Update function description.

Update function description.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.