Add support for SkyRukh Surge H7 board. #31230
Conversation
There was a problem hiding this comment.
some initial comments before a detailed review:
- none of the default params are required....the ports are already defaulted to those
- SERIAL4 defaults to GPS2
- you say all serial ports have DMA...they dont
- you pinout and wiring images are too poor resolution to tell the pins....and their labels should reflect which UART the TX/RX pins are for
I will do a detailed review once these have been addressed...
mehmetdrnmrkt
force-pushed
the
pr_skyrukh_surge_h7
branch
from
f42bcb8 to
c0dd196
Compare
|
Thanks for the feedback! Board images have been updated with higher resolution. Default params revised — SERIAL4 now marked as None since there’s no second GPS port. DMA configuration corrected and debug UART assignment fixed (it was previously mislabeled). |
mehmetdrnmrkt
force-pushed
the
pr_skyrukh_surge_h7
branch
2 times, most recently
from
c18d8de to
39700f6
Compare
|
I’ve implemented your requested changes. Please review again when you have a chance. |
mehmetdrnmrkt
force-pushed
the
pr_skyrukh_surge_h7
branch
from
39700f6 to
b7bd575
Compare
|
@andypiper I think the BIDIR is mis-cofigured....please review |
|
|
||
| The SkyRukh Surge H7 supports OSD using OSD_TYPE 1 (MAX7456 driver). | ||
|
|
||
| ## PWM Output |
There was a problem hiding this comment.
once bidir is finalized will need to note which outputs support it
also should list the timer groups and not that all outputs in a group must be the same protocol
ie
Outputs 1-3 group1
Outputs 5 & 7 group 2
Outputs 4.7.8 group 3
Hi @andypiper, did you get a chance to review the BIDIR config? |
mehmetdrnmrkt
force-pushed
the
pr_skyrukh_surge_h7
branch
from
b7bd575 to
4399b89
Compare
3 participants
This PR adds hardware definition files for the SkyRukh Surge H7 flight controller. The board is based on the STM32H743 MCU, provides 8 PWM/DShot outputs, multiple UARTs, CAN bus, I2C/SPI connectivity. Basic functionality including bootloader, USB, serial communication, PWM outputs, and flight tests has been successfully verified.