Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #375, Separate network logic in pc-rtems to support generic targets #376

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 4, 2023

Checklist (Please check before submitting)

Describe the contribution

NOTE: really this is just a draft proof of concept to show this can be used generically if the network config is removed

Testing performed
Spot checked with LEON3 QEMU.

Expected behavior changes
Eventually plan to use source selection to include platform specific config... or consider making this generic in some other way. PSP module would be nice but I'm not aware of a way to select from a toolchain file or based on platform without making another psp.

System(s) tested on
LEON3 QEMU, see nasa/osal#1350

Additional context
Depends on an as-yet submitted toolchain file.

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper marked this pull request as draft January 4, 2023 22:39
@skliper skliper requested a review from pepepr08 January 4, 2023 22:39
@skliper
Copy link
Contributor Author

skliper commented Jan 4, 2023

@dmknutsen, @alanc98, @jphickey - another one for discussion if we want to go this route or if there are any better ideas

@skliper
Copy link
Contributor Author

skliper commented Aug 3, 2023

Note - discussed at 20230803 CCB, sounds like this is the right direction just need to separate so the PC specifics can be included in that case (source select)

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.

Separate network logic in pc-rtems to support generic targets
2 participants