Skip to content

Add CfsBridge::subscribeCfsCommand for subscribing to cFS command messages - #3

Open
devin-ai-integration[bot] wants to merge 2 commits into
develfrom
devin/1784747100-subscribe-cfs-command
Open

devin-ai-integration[bot] wants to merge 2 commits into
develfrom
devin/1784747100-subscribe-cfs-command

Conversation

@devin-ai-integration

Copy link
Copy Markdown

Summary

CfsBridge::subscribe(apid) can only produce telemetry-style message IDs (plus the special-cased FW_PACKET_COMMAND), so an application cannot subscribe to a standard cFS command MID such as a scheduler (SCH) wakeup message.

Adds:

CFE_Status_t subscribeCfsCommand(const ComCfg::Apid::T apid);
// MID = apid | CFS_BRIDGE_SPACE_PACKET_TYPE_MASK (0x1000) | CFS_BRIDGE_SPACE_PACKET_SEC_HDR_MASK (0x0800)

so e.g. APID 0x090 subscribes to MID 0x1890, matching cFS command messages carrying a command secondary header. Such messages then flow bridge → deframer → CfsRouter CFS_COMMAND route → SchAppDriver.

Used by JPL-Devin/fprime_cfs_reference#3 to drive the F Prime rate groups at 1 Hz from the real nasa/SCH app. Tested there against cFE main, draco-rc5, and v7.0.1.

Note: intended for fprime-community/fprime_cfs devel, but this session has no push/PR access to fprime-community — opened on this fork so it can be forwarded upstream.

Link to Devin session: https://nasa-jpl-demo.devinenterprise.com/sessions/6fed9a7881e24ac18ea71733c54508fa

…sages

Subscribes with the packet type bit and secondary header flag set in the
stream identifier, allowing the bridge to receive cFS command messages such
as scheduler (SCH) wakeup ticks.

Co-Authored-By: michael.d.starch <michael.d.starch@jpl.nasa.gov>
@devin-ai-integration

Copy link
Copy Markdown
Author
Original prompt from michael.d.starch

We've just done a bunch of work on fprime-cfs-reference. Would you please edit that reference to include the sched app (not the lab, the real one). Configure it to dispatch messages at 1Hz. Then test and make sure the F Prime GDS -> fprime_app in cFS works and is reporting telemetry on the rategroup?

Then make a PR for any changes

@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

…ry message type

Co-Authored-By: michael.d.starch <michael.d.starch@jpl.nasa.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants