-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
nxp/imxrt_common/main:Fix Breakage from a9962dc #23174
Conversation
|
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 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 470bea9 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. |
…table before jump to APP
c14f866
to
98267b9
Compare
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 |
This was merged and will not even compile. There are babies being thrown out in the bathwater! - CI needs to be Fixed.