PX4-Autopilot
PX4-Autopilot copied to clipboard
nxp/imxrt_common/main:Fix Breakage from a9962dc
This was merged and will not even compile. There are babies being thrown out in the bathwater! - CI needs to be Fixed.
~~This is waiting on a nuttx change.......~~ Done
Oh dear. Thanks @davids5 !
Ok so once this compiled is did not boot. The Cause was that the last commit on Nuttx 0f401a6062653795b6355c420ea8b0e72578c204 cause the ITCM to be write protected, which caused fault on reboot.
The Fix is adding an additional mpu_configure_region(IMXRT_ITCM_BASE, 256 * 1024, before jumping to the APP.
But a bug in NuttX was preventing the MPU reset prior to reinitialization. - This has been fixed, unstreamed and backported.
@igalloway @PetervdPerk-NXP @vincentpoont2 : Bootloader replacement is needed! This PR provides a working bootloader and APP and should installed on all HW. Using PX4 after 470bea9ba84ac6202a707b263bbc2ddee4305b93 will fail to boot without the bootloader from this PR.
Thanks David, Peter.
@dk7xe we will need to offline plan for updating the bootloader in the existing validation units.
Thanks for the build and CI fix David! Is this still blocked or verified to solve the issues now? As far as I can see the remaining CI errors are not related. Let's work through and get them all fixed. We can coordinate in the dev call in a few minutes.
Thanks for the build and CI fix David! Is this still blocked or verified to solve the issues now? As far as I can see the remaining CI errors are not related. Let's work through and get them all fixed. We can coordinate in the dev call in a few minutes.
@MaEtUgR It is ready to come in and get backported to the 1.15 backports
Thanks again 👍