docs(narinfc/h7): add flight controller documentation#26721
docs(narinfc/h7): add flight controller documentation#26721
Conversation
Add hardware documentation for the VOLOLAND NarinFC-H7 flight controller including pinouts, serial port mapping, PWM output groups, and analog inputs. Fix version badge to v1.17 and clarify RC_IN is wired directly to FMU via RC_INPUT driver. Based on work by @vololand and @hamishwillee from PR #26542. Signed-off-by: Ramon Roche <mrpollo@gmail.com>
|
No broken links found in changed files. |
Remove broken anchor link to modules_driver.md#rc-input which does not exist. Use inline code formatting instead. Signed-off-by: Ramon Roche <mrpollo@gmail.com>
hamishwillee
left a comment
There was a problem hiding this comment.
Docs look good to me @mrpollo . I'm happy for you to merge when you are ready.
|
|
||
| The NarinFC-H7 supports up to 14 PWM outputs. | ||
| All outputs except M13 and M14 support DShot. | ||
| Outputs 1-8 support Bi-Directional DShot. |
There was a problem hiding this comment.
According to Claude, only outputs 1–4 (Timer5) support bidirectional - since _bidi_timer_index = 0 is hardcoded and limits bidirectional to the first timer. @mrpollo
EDIT. Note, after looking at Jake's new PR it further says that this is the true statemnt:
Outputs 1–12 support DShot. Outputs 1–7 and 9–12 support Bidirectional DShot (output + eRPM). Output 8
supports Bidirectional DShot output only (no eRPM capture — STM32H7 Timer4 CH4 has no per-channel apture DMA).
There was a problem hiding this comment.
Note, even if this is true, it might be an accidental limitation. Until recently (#26263) you could only 4, but that limitation seems to have changed https://docs.px4.io/main/en/peripherals/dshot#hardware-support
Summary
Context
Supersedes #26542. This PR contains only the documentation changes, with unrelated submodule modifications (
Tools/simulation/gz,platforms/nuttx/NuttX/nuttx,src/modules/mavlink/mavlink) removed.Addresses review feedback from @hamishwillee:
RC_INPUTdriver)Based on work by @vololand and @hamishwillee.