Skip to content

Add HAKRCH743 #757

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add HAKRCH743 #757

wants to merge 1 commit into from

Conversation

ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Apr 16, 2025

No description provided.

@ot0tot ot0tot marked this pull request as ready for review May 14, 2025 13:28
@HAKRC8899
Copy link

_ No description provided. _

Hello, friend, after testing the configuration of this target pull request, we have added the following configuration:
#define USE_MAG
#define USE_MAG_HMC5883
#define USE_MAG_QMC5883
The magnetometer can be displayed, but will it affect if this is not added? If it doesn't affect you, then all tests are normal, and We hope to be able to complete the subsequent upload work as soon as possible! We also responded to the questions in the Partner group, but we didn't get any responses

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.

Awaiting resolution for commented lines

@ot0tot
Copy link
Contributor Author

ot0tot commented Jul 9, 2025

_ No description provided. _

Hello, friend, after testing the configuration of this target pull request, we have added the following configuration: #define USE_MAG #define USE_MAG_HMC5883 #define USE_MAG_QMC5883 The magnetometer can be displayed, but will it affect if this is not added? If it doesn't affect you, then all tests are normal, and We hope to be able to complete the subsequent upload work as soon as possible! We also responded to the questions in the Partner group, but we didn't get any responses

Magnetometer defines are only included if the mag is built into the FC. Otherwise, it is a user option when building the firmware.

@HAKRC8899
Copy link

Awaiting resolution for commented lines

Hello, do you need any analysis from our side for the comment line?

@HAKRC8899
Copy link

_ No description provided. _

Hello, friend, after testing the configuration of this target pull request, we have added the following configuration: #define USE_MAG #define USE_MAG_HMC5883 #define USE_MAG_QMC5883 The magnetometer can be displayed, but will it affect if this is not added? If it doesn't affect you, then all tests are normal, and We hope to be able to complete the subsequent upload work as soon as possible! We also responded to the questions in the Partner group, but we didn't get any responses

Magnetometer defines are only included if the mag is built into the FC. Otherwise, it is a user option when building the firmware.

Ok, thanks for your answer

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.

3 participants