Skip to content
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 - Fix VDD_3V3_SENSORS_EN polarity #15239

Merged
merged 1 commit into from
Jun 29, 2020

Conversation

jarivanewijk
Copy link
Contributor

@PetervdPerk-NXP reported that he had difficulty calibrating the sensors on a new NXP FMUK66 board (Rev. D). After further investigation he only measured about 2.6V on the 3V3_S domain on the NXP FMUK66. I measured a similar voltage level. It may or may not be the cause of the sensor calibration problems, but it's clear that something is not right here.

After studying the schematics and board init code, it seems that up until the Rev. B (not supported anymore) there was a PMOS that enabled the 3V3_S domain. Since Rev. C this voltage is coming from a dedicated low noise LDO. The enable signal is not connected to a PMOS anymore, but to the enable pin of the new LDO. The macro that is used to control the GPIO pin to enable the 3V3_S was never changed, though.

The schematics for different revisions are listed here: https://nxp.gitbook.io/hovergames/rddrone-fmuk66/schematics

This fix simply inverts the polarity for the particular GPIO pin. It will now be set to high when the macro is called. This pulls the enable pin on the LDO high. You can validate that it works by measuring the voltage at TP32. It should be about 2.5-2.6V without this fix. I am still not sure where this voltage is coming from, though.

@davids5 could you validate this?

FYI @igalloway @dk7xe - this one stayed under the radar for a LONG time. It's a miracle that there were no major problems for the participants of the first HoverGames challenge...

@dagar dagar requested a review from davids5 June 29, 2020 14:28
@davids5
Copy link
Member

davids5 commented Jun 29, 2020

@jarivanewijk it is most likely back feeding. I will review the off state and see if the correct nodes are being set correctly.

@davids5 davids5 merged commit fe7fcb1 into PX4:master Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants