Extend from 4 ADC channels to 6 (Matek H743 and others) #9180
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
ichi from the INAV Discord here and I tested this on my Matek H743 Wing V3 FC with the Matek Hall Current sensor 150A wired to the CU2 pad of the FC. This worked as expected and allowed the remapping of the current_adc_channel from channel 2 to channel 6 to be able to use the external/second current sensor pad on the FC. This should also allow remapping the vbat_adc_channel to channel 5 to be able to use the external/second voltage sensor pad on the FC as well. |
Closed
|
Vbat_adc_channel assignment works, but scaling is not working after reassignment. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some newer boards, such as the Matek H743, have 6 adc channels. ADC 6 is marked as the external current sensor pin on the Matek. This PR expands the current support for 4 ADC to 6. I made no changes to how it works, just added adc5 and adc6 where we already had adc1, 2, 3, and 4.
Tested by Ichi on the INAV Discord.