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

Remove unused define CFE_PLATFORM_ES_RESET_AREA_SIZE #2327

Closed
pepepr08 opened this issue May 5, 2023 · 1 comment · Fixed by #2331
Closed

Remove unused define CFE_PLATFORM_ES_RESET_AREA_SIZE #2327

pepepr08 opened this issue May 5, 2023 · 1 comment · Fixed by #2331
Labels
enhancement Equuleus-rc1
Milestone
Equuleus

Comments

@pepepr08
Copy link
Contributor

pepepr08 commented May 5, 2023

The define is not needed anymore and is currently unused.

Now the "PSP (using sizeof(CFE_ES_ResetArea_t)) informs ES of what the total reserved memory area is, and ES informs PSP of how to organize/partition it."

cFE/modules/es/config/default_cfe_es_internal_cfg.h

Line 352 in e3b6767

#define CFE_PLATFORM_ES_RESET_AREA_SIZE (170 * 1024)

@pepepr08
Copy link
Contributor Author

pepepr08 commented May 5, 2023

Related #610

@pepepr08 pepepr08 mentioned this issue May 8, 2023
Merged
2 tasks
dzbaker added a commit that referenced this issue May 12, 2023
@dzbaker
Merge pull request #2331 from pepepr08/fix-2327-remove-unused-define
524da65 
Fix #2327, Remove unused CFE_PLATFORM_ES_RESET_AREA_SIZE
@dzbaker dzbaker mentioned this issue May 12, 2023
Merged
2 tasks
@dmknutsen dmknutsen added this to the Equuleus milestone May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Equuleus-rc1
Projects
None yet
Milestone
Equuleus
Development

Successfully merging a pull request may close this issue.

Fix #2327, Remove unused CFE_PLATFORM_ES_RESET_AREA_SIZE pepepr08/cFE
3 participants
@pepepr08 @dmknutsen @chillfig