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 new target AOCODARCF7DUAL #7994

Merged
merged 4 commits into from
May 1, 2022
Merged

Add new target AOCODARCF7DUAL #7994

merged 4 commits into from
May 1, 2022

Conversation

dlt2018
Copy link
Contributor

@dlt2018 dlt2018 commented Apr 27, 2022

The code was compiled to version 4.1.0 firmware and has been tested for flight。

@DzikuVx
Copy link
Member

DzikuVx commented Apr 27, 2022

@dlt2018 The build is failing on the AOCODARCF7DUAL target. Please fix

/home/runner/work/inav/inav/src/main/target/AOCODARCF7DUAL/target.c:34:23: error: conflicting types for 'timerHardware'
   34 | const timerHardware_t timerHardware[] = {
      |                       ^~~~~~~~~~~~~
In file included from /home/runner/work/inav/inav/src/main/target/AOCODARCF7DUAL/target.c:25:
/home/runner/work/inav/inav/src/main/drivers/timer.h:139:24: note: previous declaration of 'timerHardware' was here
  139 | extern timerHardware_t timerHardware[];
      |                        ^~~~~~~~~~~~~

@dlt2018
Copy link
Contributor Author

dlt2018 commented Apr 28, 2022

@dlt2018在 AOCODARCF7DUAL 目标上构建失败。请修复

/home/runner/work/inav/inav/src/main/target/AOCODARCF7DUAL/target.c:34:23: error: conflicting types for 'timerHardware'
   34 | const timerHardware_t timerHardware[] = {
      |                       ^~~~~~~~~~~~~
In file included from /home/runner/work/inav/inav/src/main/target/AOCODARCF7DUAL/target.c:25:
/home/runner/work/inav/inav/src/main/drivers/timer.h:139:24: note: previous declaration of 'timerHardware' was here
  139 | extern timerHardware_t timerHardware[];
      |                        ^~~~~~~~~~~~~

ok, i fixed it.
There was an error because there was an extra “const”,but code compiles without error in my environment?

@DzikuVx DzikuVx added this to the 5.0 milestone May 1, 2022
@DzikuVx DzikuVx merged commit 7cd6614 into iNavFlight:master May 1, 2022
@DzikuVx DzikuVx added the Release Notes Add this when a PR needs to be mentioned in the release notes label May 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants