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

[target] PYRODRONEF7 - fix compiler warnings / update Timers #985

Merged

Conversation

nerdCopter
Copy link
Member

@nerdCopter nerdCopter commented Mar 4, 2024

  • via BF's USE_TIMER_MAP_PRINT
  • also fixes compiler warnings:
./src/main/target/PYRODRONEF7/target.c:38:5: warning: excess elements in array initializer
   38 |     DEF_TIM(TIM4,  CH4,  PB9, TIM_USE_MOTOR,               0, 0), // NONE  TIM4_UP_D1-ST6     MOTOR6
      |     ^
./src/main/target/PYRODRONEF7/target.c:38:5: note: (near initialization for 'timerHardware')
./src/main/target/PYRODRONEF7/target.c:39:5: warning: excess elements in array initializer
   39 |     DEF_TIM(TIM8,  CH4,  PC9, TIM_USE_MOTOR,               0, 0), // D2-ST7                   MOTOR7
      |     ^
./src/main/target/PYRODRONEF7/target.c:39:5: note: (near initialization for 'timerHardware')
./src/main/target/PYRODRONEF7/target.c:41:5: warning: excess elements in array initializer
   41 |     DEF_TIM(TIM3,  CH4, PB1, TIM_USE_MOTOR | TIM_USE_LED,  0, 0), // D1-ST2                   LED/MOTOR5
      |     ^

@nerdCopter nerdCopter changed the title [target] PYRODRONEF7 - fix / update Timers [target] PYRODRONEF7 - fix compiler warnings / update Timers Mar 6, 2024
@nerdCopter nerdCopter merged commit ae97c69 into emuflight:master Mar 21, 2024
4 checks passed
@nerdCopter nerdCopter deleted the 20230304_PYRODRONEF7_fix_timer_count branch March 21, 2024 18:52
nerdCopter added a commit that referenced this pull request Apr 4, 2024
* [target] PYRODRONEF7 - fix / update Timers
@nerdCopter nerdCopter restored the 20230304_PYRODRONEF7_fix_timer_count branch June 26, 2024 14:35
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.

1 participant