fwcommon

Clone Tools

Stats

Commits this week: 0
Total Committers: 13
Last Commit: 19 Nov 25
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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 10 Nov 2022
9:05 am

Dara Navaei

DEN-14175 moved the hardware configurations enum to the common from utilities

Options
    • -1
    • +11
    /Common.h
    • -10
    • +0
    /Utilities.h
Wednesday 09 Nov 2022
1:59 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:56 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:56 pm

Dara Navaei

DEN-14175 addressed a code review comment

Options
    • -2
    • +0
    /Common.h
1:55 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:54 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:54 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:23 pm

Dara Navaei

Done.

Done.

1:21 pm

Dara Navaei

DEN-14175 addressed a code review comment by removing the getHardwareConfig function

Options
    • -17
    • +0
    /Utilities.c
    • -4
    • +0
    /Utilities.h
1:13 pm

Dara Navaei

Done

Done

1:12 pm

Dara Navaei

Done.

Done.

1:11 pm

Dara Navaei

Done.

Done.

1:03 pm

Dara Navaei

DEN-14175 removed a blank line

Options
    • -1
    • +0
    /NVDataMgmt.c
8:44 am

Sean Nash

Still doesn't belong here. We can either have a new shared FPGA header file or we can have two of these - one for each sub-system (similar to how we have two FPGA.c/.h files).

Still doesn't belong here. We can either have a new shared FPGA header file or we can have two of these - one for each sub-system (similar to how we have two FPGA.c/.h files).

8:41 am

Sean Nash

Doesn't this command take a pointer to a structure now?

Doesn't this command take a pointer to a structure now?

Monday 07 Nov 2022
11:27 am

Dara Navaei

This is here so both stacks can use it.

This is here so both stacks can use it.

11:27 am

Dara Navaei

Done.

Done.

10:44 am

Sean Nash

Why is this in utilities module? Doesn't it make more sense in fpga module?

Why is this in utilities module? Doesn't it make more sense in fpga module?

10:43 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:42 am

Sean Nash

Remove blank line.

Remove blank line.

10:41 am

Sean Nash

None of these structures appear to require packing - all 32-bit fields.

None of these structures appear to require packing - all 32-bit fields.

10:40 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

10:40 am

Sean Nash

If this message is for DG CPLD only, should change enum to "MSG_ID_DG_CPLD_...".

If this message is for DG CPLD only, should change enum to "MSG_ID_DG_CPLD_...".

10:39 am

Dara Navaei

Merge branch 'staging' into DEN-14175-dg_hd_dev-hd_dg_dvt-update-part-5

Options
10:39 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:36 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

10:33 am

Dara Navaei

DEN-14175 added _RELEASE_ to check the Software Config self test check

Options
    • -0
    • +2
    /NVDataMgmt.c
    • -7
    • +7
    /NVDataMgmtDGRecords.h
10:21 am

wbracken  (deleted user)

Looks like blank line is still there.

Looks like blank line is still there.

9:37 am

Dara Navaei

Done.

Done.

9:33 am

Dara Navaei

Done.

Done.