PX4 on nuttx 7.27+ #11013
Closed
PX4 on nuttx 7.27+ #11013
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
davids5
force-pushed
the
master_nuttx_7.27+
branch
from
81b9a7f to
f25bf5f
Compare
|
Compressed ROMFS savings
|
|
Tested on pixhawk 4, on stabilized and altitude mode. good flight, no issues. |
No GPS? https://review.px4.io/plot_app?log=6637699e-fbee-49f6-b6ff-1d744503ed40#Nav-GPS-Uncertainty |
|
@dagar In the place where we do the bench tests the gps does not work |
|
Flash comparison of px4io-v2 to master.
|
|
@davids5 here's an example showing how to compress the defconfig in cmake that produces the same output as the NuttX helpers. https://github.com/PX4/NuttX/blob/a87082007a85fcf06ff1040dce472bc4bc48c827/cmake/defconfig.cmake#L146-L174 To reconstitute you just need to run |
|
@PX4/testflights could you expand testing a bit? Again take it very slow, but try something outdoors with GPS and also try both a pixhawk 4 and pixhawk 4 mini. |
@dagar lovely! I will pull this in today and get the reset of the config converted. |
|
Tested with Pixhawk 4. good flights, no issues. Flight on auto mode. |
|
Tested on pixhawk 4 mini. good flight, no issues. |
|
Sidenote related to the last pixhawk 4 mini test (https://review.px4.io/plot_app?log=1a2b42bb-ea49-499d-9432-9059e8765d7e) CPU Spikes in the pixhawk 4 mini log Not present in the pixhawk 4 logs https://review.px4.io/plot_app?log=a9d3ce11-9fb3-4af5-a9cf-e94ec3b4e88b @Avysuarez @Tony3dr as a quick test could you try swapping sdcards between the pixhawk 4 and 4 mini? |
|
Pixhawk 4 mini with the sd card of the pixhawk 4. |
@Avysuarez - thank you!, @dagar I guess that kills the SD theory. Let's list all the other key differences are:
|
davids5
force-pushed
the
master_nuttx_7.27+
branch
2 times, most recently
from
bb7d8d4 to
7f89090
Compare
|
@dagar - current status: All Nuttx build building locally. Not necessarily running! |
davids5
force-pushed
the
master_nuttx_7.27+
branch
from
7f89090 to
24cd8da
Compare
|
@dagar this has my first cut at the compressed config. We can not build with compressed config yet due to cmake needs for CONFIG_ARCH_FAMILY and CONFIG_ARCH_CHIP and something else that causes You can debug that here: #11086 I have checked that uncompressed .config in the build tree will be == to the defconfig found in this pr. |
davids5
force-pushed
the
master_nuttx_7.27+
branch
from
2bdc433 to
973f92c
Compare
davids5
changed the title
|
@PX4/testflights See #11013 (comment) - please expand testing to all Nuttx boards:
|
- needed if NuttX networking enabled
dagar
force-pushed
the
master_nuttx_7.27+
branch
from
c424407 to
2d694ca
Compare
- mixing NuttX's limited c++ standard library with the toolchain's full c++ standard library is potentially unsafe
|
NuttX version bumped to 7.28. Let's continue in #11256. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
All: All board configs have been upgraded. This is not complete in that the compressed defconfigs are a WIP. Not all boards have been bench vetted.
The Branch will be force pushed to do not relay on it!
@PX4/testflights - Please bench test as many builds - if it seems OK it can be tested on the ground but please do not assume it is safe, until it earns your trust.