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 #165, Add CFE_PSP_GetProcessorName #189

Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Aug 24, 2020

Describe the contribution
Fix #165 - Add CFE_PSP_GetProcessorName

Testing performed
Added test code in sample_app to output SC ID, CPU ID, and CPU NAME
Build and ran (pc-linux only), tested with both default options and ./core-cpu1 --cpuid=2 --scid=40 --cpuname="bleh", confirmed expected output

Expected behavior changes
Now provides get processor name API

System(s) tested on

  • Hardware: cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: bundle main (+ cfe/osal main) + this change.

Additional context
nasa/cFE#827

Third party code
None

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

@skliper skliper added CCB:FastTrack CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) enhancement New feature or request labels Aug 24, 2020
@skliper
Copy link
Contributor Author

skliper commented Aug 24, 2020

Note - missing stub...

@skliper skliper changed the title Fix #65, Add CFE_PSP_GetProcessorName Fix #165, Add CFE_PSP_GetProcessorName Aug 24, 2020
@skliper skliper force-pushed the fix165-add-getprocessorname branch from 404ba01 to 3ae587c Compare August 25, 2020 20:50
@skliper
Copy link
Contributor Author

skliper commented Aug 25, 2020

Added stub and fixed original commit message, ready for review.

@astrogeco
Copy link
Contributor

CCB 2020-08-26 - APPROVED

@yammajamma yammajamma added CCB-20200826 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Aug 26, 2020
@yammajamma yammajamma changed the base branch from main to integration-candidate August 26, 2020 19:09
@yammajamma yammajamma merged commit d424374 into nasa:integration-candidate Aug 26, 2020
@skliper skliper deleted the fix165-add-getprocessorname branch February 1, 2021 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:FastTrack enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add CFE_PSP_GetProcessorName() API
3 participants