-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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_fmuk66 reboot command hangs if SD card popped in and out #11282
Comments
I'm going through some old NXP / FMUK66 related issues. I cannot reproduce this one, but I do get a mag timeout when an SD card is inserted while the FMU is powered.
nsh> ERROR [sensors] Mag #0 fail: TIMEOUT! |
@jarivanewijk I will have to determine the root cause. But I would suggest not making removal/insertion while powered a common practice. |
@davids5 Of course we should not make this a common practice. However, I think this might cause in flight issues when there is bad contact between SD card and the reader (vibrations?). You would (temporarily?) lose a sensor. We see quite a lot of logging drop outs with FMUK66, so this might not be too far fetched. |
@jarivanewijk yes I was thinking along the same lines, hence the comment on determining the root cause. I will not be able to get to this right away, but have it on my list. Have you tested on current master with the workqueue changes? |
@davids5 This issue has been open since January. If it was a big problem we would have come back to this earlier. There's no rush. ;) Just tested with latest master (c66fc85). I get the same behavior as yesterday (with 1.9.2 stable). I now always see both the mag and the baro fail.
|
We are getting some reports of flight logs being cut off mid flight. I think we need to have another look at this issue, because it is probably all related. Can we help in any way to find the root cause of these SD card issues? @davids5 |
I had the flight cut of twice now. Running 1.9.2 stable. Wondering if related to issue #11815 Here are my cut-off flight logs: |
@redxeth, @jarivanewijk Please test #13440 |
Thanks @davids5 ! Will try it. |
Thank you @davids5 - But I now get your original issue: the FMU hangs immediately when it is (re)booted without SD card inserted. Reinserting the SD card while PX4 is already running no longer gives the results I described in August (which was still the case for current master version). |
@davids5 - I just noticed that with PX4 1.10 the FMUK66 does not boot when no SD card is inserted. The message "Successfully bound SDHC to the MMC/SD driver" is printed in the debug console and then it hangs. I assume this is related to this issue and PR #13440. It was not the case for PX4 1.9.2. When the board is powered when an SD card is inserted, it works as it should (also with 1.10). |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
Closing, please re-open if this is still an issue. |
I tested this with a resent master. With no issue. @jarivanewijk does the unit in question have the latest bootloader? |
Sorry that it took me so long to get back to this, @davids5 - No, it was an old bootloader. We were still using the bootloader from May 2019, because of PX4/PX4-Bootloader#149 - It was fixed in November, but I forgot to test it and update the bootloader binary that we were distributing. With the latest bootloader and firmware I have no issues. Sorry for the inconvenience! |
I was too quick. It works fine with latest master firmware, but with latest stable (1.10.1) it still hangs after the bootloader (RGB LED remains white). @davids5 |
Thank you @jarivanewijk for retesting. Given the 1.11 is to be released soon, I would say it is good this is documented and will be resolved in the new release. |
Describe the bug
nxp_fmuk66 reboot command hangs if SD card popped in and out
To Reproduce
Steps to reproduce the behavior:
Expected behavior
shutdown line 201 never calls callback. May be related to card insert or logger. I am assuming the Work queue is blocked.
fyi @igalloway
The text was updated successfully, but these errors were encountered: