-
Notifications
You must be signed in to change notification settings - Fork 18k
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
AP_HAL_ChibiOS: Pixhawk6X peripheral power on at startup #26753
AP_HAL_ChibiOS: Pixhawk6X peripheral power on at startup #26753
Conversation
What about radios that use RTS for bootloader hold? |
Hi @amilcarlucas, Indeed, I saw that comment and it is in pretty much every board that delays the power on. Some boards do not delay the power on though so I suspect the comment is out-of-date. Another alternative is just to power on the CAN bus first (e.g. the 2nd definition only). I've tested that as well successfully. Maybe @tridge knows. |
I think we should at least apply on:
|
6c683d3
to
d4b82b9
Compare
d4b82b9
to
c69bc16
Compare
I've updated the hwdef files for the three boards mentioned (Pixhawk6C, Pixhawk5X, Durandal). As a drive-by change I also correct the comments for two other boards (CUAV-X7, CUAV-Nora) that seemed to be saying the peripherals were OFF at startup when they were actually ON. |
This is included in 4.5.2-beta1 |
This addresses the Hereflow startup issue recorded in the 4.5.0 issues list and discussed here on the forums.
The issue is that the HereFlow sensor connected to a Pixhawk6X does not start successfully with 4.5.0 but does with 4.4.4. The temporary work around is for users to set BRD_BOOT_DELAY to 3000 which slows AP's startup until after the flow sensor has initialised.
This PR includes a better fix by providing power to the Hereflow sensor as soon as the autopilot is powered on. Note that this change affects more than just the CAN port (but I'm not personally sure what other ports are affected). This change affects these boards:
This has been tested on real hardware including:
This PR changes both the "HIPOWER" and "PERIPH" definitions. In fact, only the 2nd definition must be changed to resolve the issue but I suspect they should be consistent.
From a quick search for "nVDD_5V_PERIPH_EN OUTPUT HIGH" these other boards may have similar issues:
I'm unsure if other boards might have similar problems.