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

add BLADE_PRO_H7 #592

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

add BLADE_PRO_H7 #592

wants to merge 10 commits into from

Conversation

HAKRC
Copy link

@HAKRC HAKRC commented Oct 14, 2024

Add new target BLADE_PRO_H7

@haslinghuis
Copy link
Member

haslinghuis commented Oct 15, 2024

@HAKRC
Copy link
Author

HAKRC commented Oct 18, 2024

@HAKRC please check https://betaflight.com/docs/development/manufacturer/config-target-guidance

We have sent the schematic to [email protected] . Please check thanks.

@hakrc1
Copy link
Contributor

hakrc1 commented Nov 27, 2024

Dear Betaflight maintainers,

Thank you for reviewing my Pull Request. I have followed the Betaflight Config Target Guidance and submitted all required materials, including:

The configuration target files for the new flight controller.
The schematic, which has been sent to [email protected] as requested.
If there are any additional documents or modifications needed, please let me know.

Additionally, I would greatly appreciate it if you could help review and approve this Pull Request when convenient, as all checks have passed. Please feel free to reach out for further clarification or support.

Thank you for your time and assistance!

Best regards,
Hakrc
@haslinghuis @hydra @mosfet @raphaelcoeffic

configs/BLADE_PRO_H7/config.h Outdated Show resolved Hide resolved
configs/BLADE_PRO_H7/config.h Show resolved Hide resolved
configs/BLADE_PRO_H7/config.h Outdated Show resolved Hide resolved
@ot0tot
Copy link
Contributor

ot0tot commented Dec 21, 2024

Dear Betaflight maintainers,

Thank you for reviewing my Pull Request. I have followed the Betaflight Config Target Guidance and submitted all required materials, including:

The configuration target files for the new flight controller. The schematic, which has been sent to [email protected] as requested. If there are any additional documents or modifications needed, please let me know.

Additionally, I would greatly appreciate it if you could help review and approve this Pull Request when convenient, as all checks have passed. Please feel free to reach out for further clarification or support.

Thank you for your time and assistance!

Best regards, Hakrc @haslinghuis @hydra @mosfet @raphaelcoeffic

Still waiting for you to address the significant revisions requested on the Discord.

@HAKRC
Copy link
Author

HAKRC commented Dec 24, 2024

Dear Betaflight maintainers,
Thank you for reviewing my Pull Request. I have followed the Betaflight Config Target Guidance and submitted all required materials, including:
The configuration target files for the new flight controller. The schematic, which has been sent to [email protected] as requested. If there are any additional documents or modifications needed, please let me know.
Additionally, I would greatly appreciate it if you could help review and approve this Pull Request when convenient, as all checks have passed. Please feel free to reach out for further clarification or support.
Thank you for your time and assistance!
Best regards, Hakrc @haslinghuis @hydra @mosfet @raphaelcoeffic

Still waiting for you to address the significant revisions requested on the Discord.
Thanks, I've looked it up, and we'll make changes to the schematic and configuration files based on your suggested changes.

Comment on lines +57 to +58
#define SERVO3_PIN PD15
#define SERVO4_PIN PD14
Copy link
Contributor

Choose a reason for hiding this comment

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

These conflict with motors 7&8 on Timer 4

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.

4 participants