Skip to content

docs(boards): Add PWM Outputs section#26818

Open
hamishwillee wants to merge 2 commits into
mainfrom
pwm_outputs_fcs
Open

docs(boards): Add PWM Outputs section#26818
hamishwillee wants to merge 2 commits into
mainfrom
pwm_outputs_fcs

Conversation

@hamishwillee
Copy link
Copy Markdown
Contributor

@hamishwillee hamishwillee commented Mar 19, 2026

This adds a section PWM Outputs to all FC docs for which one can be generated - using #27103

This is generated from a Claude-created script. It looks at timers etc to determine the output. It generates some JSON in the file first.
parse_pwm_outputs.py
pwm_outputs_1.md

@dakejahl This all looks pretty sensible and I've had happy results with Claude, but I am not sure how to verify that it hasn't gone off the rails. Could you spot check this - for example look at a few of the Ark and Pixhawk cases?
There are also 4 cases which already had entries, and these have changed. I am presuming as a resutl of your Dhot changes.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 19, 2026

/en/flight_controller/corvon_743v1.md

| UART8 | /dev/ttyS7 | GPS2 |

## PWM Output
## PWM Outputs
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one needs to be fixed up manually - the labeling of the ports was lost, and there is also electrical data that was replaced.

| UART8 | /dev/ttyS5 | SERIAL4 |

## PWM Output
## PWM Outputs
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be updated manually. The data might be correct but we want to capture the labelling.

@hamishwillee hamishwillee force-pushed the pwm_outputs_fcs branch 3 times, most recently from 35a734d to 45ab9bd Compare April 2, 2026 00:08
The remaining 6 PWM ports are still being adapted (so it is not compatible with VOLT at time of writing).
:::

## PWM Outputs {#pwm_outputs}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cuav-chen2 When you have a moment, can you please check the information on the CUAV boards for PWM outputs? This is script generated, and has been updated following the information you provided in #26882 (comment)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No errors in the description.

Comment on lines +43 to +54
## PWM Outputs {#pwm_outputs}

This flight controller supports up to 6 FMU PWM outputs (AUX) and 8 IO PWM outputs (MAIN).

All FMU outputs support [DShot](../peripherals/dshot.md) and [Bidirectional DShot](../peripherals/dshot.md#bidirectional-dshot-telemetry).

The 6 outputs are in 2 groups:

- Outputs 1-4 in group1 (Timer1)
- Outputs 5-6 in group2 (Timer4)

All outputs within the same group must use the same output protocol and rate.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@julianoes Do you think you could sanity check the cubepilot information for PWM outputs?

Claude wrote me a script to generate this based on information in timers and so on. It's been valided against a number of boards that already documented this stuff, but I don't deeply understand it.

No great urgency, but if you can't please let me know.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, that's correct.

@dakejahl
Copy link
Copy Markdown
Contributor

dakejahl commented Apr 2, 2026

Sorry I missed this. I will review it tomorrow.

dakejahl
dakejahl previously approved these changes Apr 3, 2026
Copy link
Copy Markdown
Contributor

@dakejahl dakejahl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ARK docs are correct. Thanks!


Additional information can be found in the [Pixhawk 4 Technical Data Sheet](https://github.com/PX4/PX4-Autopilot/blob/main/docs/assets/flight_controller/pixhawk4/pixhawk4_technical_data_sheet.pdf).

## PWM Outputs {#pwm_outputs}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vincentpoont2 Can you please sanity check this new section for the Holybro flight controllers. This is produced via a claude produced script. I have got verification that it worked for Ark and Cube, and it seems to work for CUAV.

If it works for yours to I will assume it can be trusted and merge all of this :-)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HB-SP-P Can you review this?

Comment thread docs/en/flight_controller/pixhawk4.md Outdated

FMU Outputs:

- Outputs 1-4 support [DShot](../peripherals/dshot.md).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pixhawk 4 is STM32F7, not H7, so won't have DShot at all.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This bit fixed - i.e. tool filters on F7, F4 as not having support. I'm still playing with the words though to indicate support or no support.

Comment thread docs/en/flight_controller/pixhawk4.md Outdated
- Outputs 5-11 do not support DShot.
- Outputs 1-4 support [Bidirectional DShot](../peripherals/dshot.md#bidirectional-dshot-telemetry).

The 11 outputs are in 4 groups:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's 8 outputs and 3 inputs, although I'm not sure where the inputs physically are, maybe on the "CAP & ADC IN" pins.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thin this is fixed. It now states


This flight controller supports up to 8 FMU PWM outputs (AUX) and 8 IO PWM outputs (MAIN).

DShot is not supported.

The 8 outputs are in 3 groups:

  • Outputs 1-4 in group1 (Timer1)
  • Outputs 5-6 in group2 (Timer4)
  • Outputs 7-8 in group3 (Timer12)

All outputs within the same group must use the same output protocol and rate.

Comment thread docs/en/flight_controller/pixhawk5x.md Outdated
Comment on lines +120 to +122
- Outputs 1-6 support [DShot](../peripherals/dshot.md).
- Outputs 7-9 do not support DShot.
- Outputs 1-6 support [Bidirectional DShot](../peripherals/dshot.md#bidirectional-dshot-telemetry).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also an F7, no DSHot support as of now.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also fixed - it recogises not Dshot on F7.

Comment thread docs/en/flight_controller/pixhawk5x.md Outdated
- Outputs 1-4 in group1 (Timer1)
- Outputs 5-6 in group2 (Timer4)
- Outputs 7-8 in group3 (Timer12)
- Output 9 in group4 (Timer5)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Output 9 is an input.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. Globally it now sees that inputs are not outputs. I haven't fixed it up to capture what the inputs are yet though.

Comment on lines +129 to +131
- Outputs 1-8 support [DShot](../peripherals/dshot.md).
- Outputs 9-12 do not support DShot.
- Outputs 1-8 support [Bidirectional DShot](../peripherals/dshot.md#bidirectional-dshot-telemetry).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure, maybe. I'd have to test.

Comment thread docs/en/flight_controller/pixhawk4.md Outdated

## PWM Outputs {#pwm_outputs}

This flight controller supports up to 11 FMU PWM outputs (AUX) and 8 IO PWM outputs (MAIN).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be 8 FMU PWM outputs and 8 IO outs for all Holybro Pixhawk controllers

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.


## PWM Outputs {#pwm_outputs}

This flight controller supports up to 8 FMU PWM outputs (AUX) and 8 IO PWM outputs (MAIN).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For 6C Mini, there are only 6 FMU outputs

Copy link
Copy Markdown
Contributor Author

@hamishwillee hamishwillee Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HB-SP-P According to claude:

The board source files (board_config.h and timer_config.cpp) both define 8 FMU channels for the Pixhawk 6C.

IF this is correct, and there are only 6 FMU channels, doesn't that mean the base port code is wrong? Perhaps makes no difference in practice though.

@github-actions
Copy link
Copy Markdown
Contributor

/en/flight_controller/corvon_743v1.md

Comment thread docs/en/flight_controller/pixhawk5x.md Outdated

FMU Outputs:

- Outputs 1-6 support [DShot](../peripherals/dshot.md).
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doh. Claude not very bright.

…nerator script

docs(update): Regenerate PWM Outputs sections using updated fc_doc_generator script

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Max input voltage: 5.4V
- USB Power Input: 4.75\~5.25V
- Servo Rail Input: 0\~9.9V
<!-- pwm_outputs-proposed
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, note screwup in merge.

@github-actions github-actions Bot added the kind:chore Maintenance work that keeps the project healthy. label May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:chore Maintenance work that keeps the project healthy. scope:docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants