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

PX4 platform init #11269

Merged
merged 1 commit into from
Jan 22, 2019
Merged

PX4 platform init #11269

merged 1 commit into from
Jan 22, 2019

Conversation

dagar
Copy link
Member

@dagar dagar commented Jan 22, 2019

Slowly establishing a good definition of the "PX4 platform" and now gradually pulling all the common elements together so that a system intended to run PX4 modules has all the prerequisites in place.

Reference - #11232 (comment)
#11261

@dagar
Copy link
Member Author

dagar commented Jan 22, 2019

For the most part the layers start to become evident when you look at the dependencies as a graph.

  • modules and drivers should only interact through uORB and parameters
  • code that can be shared between multiple modules and drivers is a library
  • code that is shared between multiple modules, drivers, AND libraries forms the "PX4 Platform" layer

Other pieces to consider next:

  • move uORB to platform layer, autostart in px4_platform_init, split out command line for status, top, etc
  • move parameters to platform layer
  • move perf to platform layer
  • move systemlib to libs

@dagar dagar force-pushed the pr-px4_platform_init branch from 08350c9 to ce007ee Compare January 22, 2019 15:38
@dagar dagar force-pushed the pr-px4_platform_init branch from ce007ee to ff9b2d9 Compare January 22, 2019 15:42
Copy link
Member

@davids5 davids5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

@dagar dagar merged commit 320d2e9 into PX4:master Jan 22, 2019
@dagar dagar deleted the pr-px4_platform_init branch January 22, 2019 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants