fwcommon

Clone Tools

Stats

Commits this week: 0
Total Committers: 13
Last Commit: 19 Nov
Commits this week: 0

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 26 Mar 2021
5:30 pm

Sean Nash

Fixed merge conflicts.

Options
5:28 pm

Sean Nash

DEN-7091: fixes to accel alarms.

Options
    • -1
    • +1
    /Accel.c
5:25 pm

Sean Nash

Fixed merge conflicts.

Options
    • -13
    • +5
    /Accel.c
4:50 pm

Sean Nash

Fixed merge conflicts.

Options
    • -7
    • +15
    /Accel.c
2:08 pm

Sean Nash

DEN-7091: pulled in fixes to Accelerometer driver from DEN-7117.

Options
    • -2
    • +28
    /Accel.c
10:38 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:37 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:37 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:36 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Thursday 25 Mar 2021
10:42 am

Sean Nash

Yes, jobs are added to queue from General Task and removed from queue from Background Task. Need queue operations to be protected.

Yes, jobs are added to queue from General Task and removed from queue from Background Task. Need queue operations to be protected.

Wednesday 24 Mar 2021
12:39 pm

Dara Navaei

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

Options
    • -7
    • +11
    /Accel.c
12:38 pm

Dara Navaei

DEN-5980 addressed code review comments

Options
    • -7
    • +5
    /Accel.c
    • -9
    • +8
    /NVDataMgmt.c
    • -2
    • +2
    /NVDataMgmtDGRecords.h
    • -2
    • +2
    /NVDataMgmtHDRecords.h
    • -14
    • +1
    /RTC.c
12:37 pm

Dara Navaei

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.

12:26 pm

Dara Navaei

Done.

Done.

12:25 pm

Dara Navaei

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.

12:23 pm

Dara Navaei

Changed them to NVDataMgmt NVDataMgmt.

Changed them to NVDataMgmt NVDataMgmt.

12:23 pm

Dara Navaei

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.

12:21 pm

Dara Navaei

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.

12:20 pm

Dara Navaei

Done.

Done.

12:19 pm

Dara Navaei

Done.

Done.

12:18 pm

Dara Navaei

Done.

Done.

12:18 pm

Dara Navaei

I updated the comment.

I updated the comment.

12:15 pm

Dara Navaei

Done.

Done.

12:14 pm

Dara Navaei

Done.

Done.

12:11 pm

Dara Navaei

To make sure dequeuing is not interrupted.

To make sure dequeuing is not interrupted.

Tuesday 23 Mar 2021
2:05 pm

Sean Nash

Done.

Done.

2:02 pm

Sean Nash

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).

1:25 pm

pmontazemi  (deleted user)

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".

1:11 pm

pmontazemi  (deleted user)

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.

11:18 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.