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

Fixed build if none of USE_DSHOT, USE_LED_STRIP, USE_TRANSPONDER are defined. #8506

Merged
merged 1 commit into from Jun 30, 2019

Conversation

mikeller
Copy link
Member

No description provided.

@mikeller mikeller added this to the 4.0.4 milestone Jun 29, 2019
@etracer65
Copy link
Member

Does this open up another possible conflict of dmaGetOptionByTimer not being defined when it's needed here

config[configIndex].dmaopt = dmaGetOptionByTimer(configuredTimer);
with a different set of define criteria?

@mikeller mikeller force-pushed the fix_build_without_features branch 2 times, most recently from ccae0d1 to e8e899a Compare June 30, 2019 03:14
@mikeller
Copy link
Member Author

Well spotted @etracer65, this needed remodeling, as the reduced timer struct when Dshot, LED_STRIP, and transponder aren't used simply cannot support DMA for timers.

@mikeller mikeller merged commit 3936bba into betaflight:master Jun 30, 2019
@mikeller mikeller deleted the fix_build_without_features branch June 30, 2019 13:47
mikeller added a commit that referenced this pull request Jun 30, 2019
Fixed build if none of USE_DSHOT, USE_LED_STRIP, USE_TRANSPONDER are defined.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants