CS
CS copied to clipboard
Fix #26, Use `CFE_ES_MemAddress_t` instead of `cpuaddr` in tables/tlm/cmd structs
Checklist
- [x] I reviewed the Contributing Guide.
- [x] I signed and emailed the appropriate Contributor License Agreement to [email protected] and copied [email protected].
Describe the contribution
- Fixes #26
- Convert
cpuaddr
types in tables/cmd/tlm structs to fixed-sizeCFE_ES_MemAddress_t
- removed the
Filler32
inCS_Res_EepromMemory_Table_Entry_t
(I assume it was there to align to the 64-bitcpuaddr
which is is longer necessary (unless/untilCFE_ES_MemAddress_t
andCFE_ES_MemOffset_t
are redefined as 64-bits in https://github.com/nasa/cFE/issues/2093. Another option is to just leave it in now for future compatibility with 64-bit alignment.
- removed the
- Convert
Testing performed GitHub CI actions all passing successfully (incl. Build + Run, Unit/Coverage Tests etc.).
Expected behavior changes
Exposed structs will no longer change size based on the definition of cpuaddr
.
Contributor Info Avi Weiss @thnkslprpt