dgfirmware / firmware / App

Services

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 10 Jun
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 04 Jun 2021
12:58 pm

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Added alarm table size check

Options
    • -0
    • +3
    ./AlarmMgmt.c
  2. … 1 more file in changeset.
12:42 pm

Sean Nash

Is this needed. Should we just let Standby mode set actuators as it wants on entry?

Is this needed. Should we just let Standby mode set actuators as it wants on entry?

12:40 pm

Sean Nash

I think we should at least say that the mode variables are reset to re-start the mode or something.

I think we should at least say that the mode variables are reset to re-start the mode or something.

12:37 pm

Sean Nash

Why are some values aligned at left and others at right? I think all values should align at left.

Why are some values aligned at left and others at right? I think all values should align at left.

12:32 pm

Sean Nash

I'm assuming HET was enabled to get access to a GPIO pin? If so, do we need these notifications? Is there a place in HalCOGen to disable HET interrupts?

I'm assuming HET was enabled to get access to a GPIO pin? If so, do we need these notifications? Is there a place in HalCOGen to disable HET interrupts?

12:28 pm

Sean Nash

Not necessary - included in .h file.

Not necessary - included in .h file.

12:26 pm

Sean Nash

Please keep this case in last position in switch statement.

Please keep this case in last position in switch statement.

11:50 am

Sean Nash

Why commented out?

Why commented out?

10:41 am

Sean Nash

Remove comments or add TODO.

Remove comments or add TODO.

10:41 am

Sean Nash

I think we can remove this test code now.

I think we can remove this test code now.

Thursday 03 Jun 2021
10:36 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:36 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:36 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:35 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:35 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Monday 31 May 2021
4:33 pm

Dara Navaei

DEN-8030 tested modes and continued developing chemical disinfect

Options
    • -14
    • +1
    ./Reservoirs.c
    • -0
    • +56
    ./SystemCommMessages.c
    • -0
    • +6
    ./SystemCommMessages.h
  4. … 12 more files in changeset.
Sunday 30 May 2021
2:46 pm

Dara Navaei

DG-DEN-8030_DG-HD DEV Mode Chemical Disinfect
DG-DEN-8030_DG-HD DEV Mode Chemical Disinfect
Saturday 29 May 2021
2:05 am

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Added filter flush time period override

Options
    • -0
    • +4
    ./SystemComm.c
    • -0
    • +32
    ./SystemCommMessages.c
    • -0
    • +3
    ./SystemCommMessages.h
  4. … 2 more files in changeset.
1:16 am

qnguyen  (deleted user)

Good point. The module has been moved to fwcommon.

Good point. The module has been moved to fwcommon.

1:16 am

qnguyen  (deleted user)

Removed the wrapper function. For now, only support one type of 32-bit CRC algorithm.

Removed the wrapper function. For now, only support one type of 32-bit CRC algorithm.

12:58 am

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Moved integrity module to fwcommon

Options
    • -1
    • +0
    ./AlarmMgmtSWFaults.h
    • -3
    • +3
    ./FPGA.c
    • -135
    • +0
    ./Integrity.c
    • -40
    • +0
    ./Integrity.h
  5. … 1 more file in changeset.
Friday 28 May 2021
4:39 pm

qnguyen  (deleted user)

Done.

Done.

4:33 pm

qnguyen  (deleted user)

Noe said they updated the ID and this is the new ID for DG FPGA. The FPGA test will fail without this change.

Noe said they updated the ID and this is the new ID for DG FPGA.
The FPGA test will fail without this change.

3:20 pm

Sean Nash

Should mention somewhere that timer is counting ms.

Should mention somewhere that timer is counting ms.

3:19 pm

Sean Nash

Why did this change?

Why did this change?

3:18 pm

Sean Nash

Is there a reason why this module isn't in fwcommon? And if it can't be common for some reason, should this module (or group name) be named DGIntegrity?

Is there a reason why this module isn't in fwcommon? And if it can't be common for some reason, should this module (or group name) be named DGIntegrity?

3:18 pm

Sean Nash

Do we need to support more than one 32-bit CRC algorithm in firmware? If not, do we need this wrapper function?

Do we need to support more than one 32-bit CRC algorithm in firmware? If not, do we need this wrapper function?

Thursday 27 May 2021
10:34 am

qnguyen  (deleted user)

Done.

Done.

10:32 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.