dd-firmware / firmware / App / Modes / FPModes

ModePreGenPermeateDefeatured.c

Clone Tools

Stats

Commits this week: 2
Total Committers: 7
Last Commit: 12:47
Commits this week: 2

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
Today
4 hrs ago

Sean Nash

See my comment above. We should return BOOL so Dialin request can be ACK/NAK'd.

See my comment above. We should return BOOL so Dialin request can be ACK/NAK'd.

4 hrs ago

Sean Nash

No faults from Dialin commands - we never want Dialin to cause system to go to fault because it requested something invalid - we only NAK the invalid request. We should do whatever is needed (if an...

No faults from Dialin commands - we never want Dialin to cause system to go to fault because it requested something invalid - we only NAK the invalid request.
We should do whatever is needed (if anything) to start the mode normally in an else.
And we should return TRUE/FALSE in these signal functions so that when it's called from Dialin msg handler, the msg handler can return ACK/NAK to Dialin if sub mode is invalid.

4 hrs ago

Sean Nash

We shouldn't request op mode until we're sure we're happy with the whole thing. Need to determine whether we are already in the op mode being requested. If so, no need to request it again - just re...

We shouldn't request op mode until we're sure we're happy with the whole thing.
Need to determine whether we are already in the op mode being requested. If so, no need to request it again - just request new sub mode if not already in it. If not, the currentSubMode checked below is not relevant (applies to current mode, not the requested mode) and we need to request new op mode and signal that new op mode that we will want to start in a specific sub mode.

4 hrs ago

Sean Nash

Declarations should be at top of scope (right before memcpy).

Declarations should be at top of scope (right before memcpy).

4 hrs ago

Sean Nash

Do we need this function? Can we do these 2 assignments in the Dialin msg handler function?

Do we need this function? Can we do these 2 assignments in the Dialin msg handler function?

4 hrs ago

Sean Nash

When moving this code to a function, the (requested) mode and sub mode should be passed in as parameters.

When moving this code to a function, the (requested) mode and sub mode should be passed in as parameters.

4 hrs ago

Sean Nash

I don't think this code should be here. Should be moved to a function and called - probably not from here though. Can we call function with this code from the Dialin msg handler?

I don't think this code should be here. Should be moved to a function and called - probably not from here though. Can we call function with this code from the Dialin msg handler?

6 hrs ago

Raghu Kallala

Shall I add the SW_Fault as an else case if an invalid submode was requested for an opmode? Sean NashMichael Garthwaite please confirm

Shall I add the SW_Fault as an else case if an invalid submode was requested for an opmode?
Sean NashMichael Garthwaite please confirm

6 hrs ago

Raghu Kallala

LDT-3603 converted all signal functions to void functions

Options
    • -6
    • +2
    ./ModePreGenPermeateDefeatured.c
  2. … 7 more files in changeset.
6 hrs ago

Raghu Kallala

I was thinking we could return BOOL to check if the request was successful, added it. But as you clarified the dialin response will only confirm the delivery of message successfully, I can change ...

I was thinking we could return BOOL to check if the request was successful, added it.

But as you clarified the dialin response will only confirm the delivery of message successfully, I can change all these signal functions to void
Thanks!

9 hrs ago

Michael Garthwaite

Yes since you're not using the return value in execOpModes()

Yes since you're not using the return value in execOpModes()

11 hrs ago

Raghu Kallala

LDT-3603 initial draft for FP set operation sub mode

Options
    • -0
    • +41
    ./ModePreGenPermeateDefeatured.c
  2. … 10 more files in changeset.
Thursday 05 Mar
7:32 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3603_Ability to set operation sub mode on IOFP sub system
LEAHI-DD-FIRMWARE-LDT-3603_Ability to set operation sub mode on IOFP sub system
Friday 16 Jan
10:35 am

Raghu Kallala

10:35 am

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    ./ModePreGenPermeateDefeatured.c
  2. … 12 more files in changeset.
10:31 am

Sean Nash , Dara Navaei and Michael Garthwaite

9:50 am

Raghu Kallala

LDT-3238 renamed module names and updated as needed.

Options
    • -2
    • +2
    ./ModePreGenPermeateDefeatured.c
  2. … 23 more files in changeset.
9:16 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3238_Update unit names for submodes to have a prefix "state"
LEAHI-DD-FIRMWARE-LDT-3238_Update unit names for submodes to have a prefix "state"
8:09 am

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    ./ModePreGenPermeateDefeatured.c
  2. … 16 more files in changeset.
Thursday 15 Jan
4:08 pm

Raghu Kallala

4:08 pm

Michael Garthwaite and Sean Nash

4:06 pm

Raghu Kallala

resolved merge conflicts

Options
    • -1
    • +1
    ./ModePreGenPermeateDefeatured.c
  2. … 15 more files in changeset.
10:59 am

Raghu Kallala

added param and output

added param and output

10:52 am

Raghu Kallala

Added struct packing

Added struct packing

9:41 am

Sean Nash

Update function header to include message param.

Update function header to include message param.

9:40 am

Sean Nash

This structure is for message payload. There is nothing to pack in this specific structure, but I think we should be in the habit of packing ALL payload structures in case a future structure change...

This structure is for message payload. There is nothing to pack in this specific structure, but I think we should be in the habit of packing ALL payload structures in case a future structure change does require packing.

9:15 am

Raghu Kallala

removed

removed

8:37 am

Raghu Kallala

LDT-2185 updated test function naming and messaging

Options
    • -2
    • +2
    ./ModePreGenPermeateDefeatured.c
  2. … 1 more file in changeset.
Wednesday 14 Jan
3:12 pm

Raghu Kallala

LDT-2185 resolved CR comments

Options
    • -1
    • +0
    ./ModePreGenPermeateDefeatured.c
  2. … 5 more files in changeset.
3:11 pm

Raghu Kallala

Fixed

Fixed