-
Notifications
You must be signed in to change notification settings - Fork 0
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
[NASA/NOS3#93] HWLIB Update #2
Conversation
…ux, added outside cfs build definitions to hwlib.h;
…tside cFS, updated bit_bit_order macro to latest cFS, and added arch_build.cmake;
* nos-linux | ||
|
||
Note that the hardware library does not include `cfe.h` or `osapi.h`. | ||
When using with cFS these includes must be in the `cpuX_device_cfg.h` file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the cavate to this update. If these updates migrate elsewhere be sure to do this!
FILE_NAME "device_cfg.h" | ||
MATCH_SUFFIX "device_cfg.h" | ||
PREFIXES ${BUILD_CONFIG} | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unsure why the cFS CMake requires this, but I'm glad you had figured it out previously @mgrubb-gsfc
…t usart_device[handle] to allow re-use of the port;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than a comment I made (where bytes is spelled as byter) everything looks fine, and that one issue is not anything of importance.
[#2] Updates for GTOSat RTEMS See merge request lems-a3/forks/hwlib!1
Update HWLIB from AETD and to allow for building a standalone checkout.
Proven working on nos3#93-SampleUpdate e5de02 branch.