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.

Sign up for GitHub

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

Jump to bottom

Require definition of CFE_CPU_ID_VALUE #1308

Closed
skliper opened this issue Apr 13, 2021 · 0 comments · Fixed by #1592 or #1619
Closed

Require definition of CFE_CPU_ID_VALUE #1308

skliper opened this issue Apr 13, 2021 · 0 comments · Fixed by #1592 or #1619
Labels
CFS-40 enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Apr 13, 2021

Is your feature request related to a problem? Please describe.
CPU_ID should be explicitly defined, defaulted here:

cFE/cmake/target/src/target_config.c

Lines 48 to 50 in e80aae9

#ifndef CFE_CPU_ID_VALUE
#define CFE_CPU_ID_VALUE 0
#endif

Describe the solution you'd like
Remove default

Describe alternatives you've considered
None

Additional context
From CFS-40 code review

Requester Info
Jacob Hageman - NASA/GSFC
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CFS-40 enhancement
Projects
None yet
Milestone
Caelum
1 participant
@skliper