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

Create YYRC-EAGLEF722 #465

Merged
merged 12 commits into from
Jul 12, 2024
Merged

Conversation

tianv2mu0897
Copy link
Contributor

Hello, I would like to create a branch and look forward to merging it; As I am creating it for the first time, if there are any issues, I am very much looking forward to you telling me the problem points; Thank you!

@haslinghuis haslinghuis changed the title Create YYRC-EAGLEF722V0 Create YYRC-EAGLEF722 Jun 16, 2024
@nerdCopter
Copy link
Member

@tianv2mu0897 , may need to provide schematics and samples to dev-team. may need to join Discord and contact @sugaarK for such.

configs/YYRC-EAGLEF722V0 Outdated Show resolved Hide resolved
configs/YYRC-EAGLEF722V0 Outdated Show resolved Hide resolved
@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 18, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 18, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 18, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 18, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 19, 2024 via email

configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
@haslinghuis haslinghuis requested a review from ot0tot June 19, 2024 21:13
@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 19, 2024 via email

@haslinghuis
Copy link
Member

Have made the updates already

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 19, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 19, 2024 via email

configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 19, 2024 via email

Copy link
Contributor

@ot0tot ot0tot left a comment

Choose a reason for hiding this comment

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

Two minor changes suggested, otherwise it looks good!

configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 20, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 20, 2024 via email

PPM is delete,  DShot burst  set to AUTO,and adding #define DEFAULT_DSHOT_BITBANG DSHOT_BITBANG_ON.
@nerdCopter nerdCopter requested a review from ot0tot June 20, 2024 12:57
@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 28, 2024 via email

@ot0tot
Copy link
Contributor

ot0tot commented Jun 28, 2024

Awaiting schematics and dev samples. Have you joined the Discord and contacted @sugaarK ?

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jun 28, 2024 via email

Add PINIO TO PC1
@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jul 12, 2024 via email

Copy link
Member

@haslinghuis haslinghuis left a comment

Choose a reason for hiding this comment

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

Need to change filename to match board_name

configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
@nerdCopter
Copy link
Member

Need to change filename to match board_name

@tianv2mu0897 , in your repo, (assuming linux)

mkdir configs/EAGLEF722
git mv configs/YYRC-EAGLEF722 configs/EAGLEF722/config.h

then commit/push

@nerdCopter
Copy link
Member

nerdCopter commented Jul 12, 2024

83488d4 is still not proper, it should be configs/EAGLEF722/config.h

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jul 12, 2024 via email

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jul 12, 2024 via email

@nerdCopter
Copy link
Member

  • not possible via web-interface.
  • i fixed and pushed

@nerdCopter
Copy link
Member

@haslinghuis , please repeat last suggestion from new file. TY

@tianv2mu0897
Copy link
Contributor Author

tianv2mu0897 commented Jul 12, 2024 via email

configs/EAGLEF722/config.h Outdated Show resolved Hide resolved
configs/YYRC-EAGLEF722 Outdated Show resolved Hide resolved
@haslinghuis haslinghuis merged commit e8e9063 into betaflight:master Jul 12, 2024
1 check passed
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.

5 participants