search

BossHobby / QUICKSILVER

Flight Controller Firmware
MIT License
171 stars 40 forks source link

Feature-motor-beep-on-aux-channel #38

Closed andreyk72 closed 2 years ago

andreyk72 commented 2 years ago

Allow to switch motor beep via aux channel AUX_BUZZER_ENABLE. This is useful when flying outdoor on low weight aircrafts where installing separate buzzer is not desirable. Works with DSHOT protocol only.

vidzo commented 2 years ago

It's probably worth adding a dis-armed flag so that it can't be used while motors are armed or in air

bkleiner commented 2 years ago

It's probably worth adding a dis-armed flag so that it can't be used while motors are armed or in air

the function is gated by (flags.arm_state == 0) || flags.failsafe || (state.throttle < 0.001f) on the calling side. so that aspect should be fine.

ultimately i would really like to see all control code removed from the dshot driver. its a driver it should not concern itself with things like failsafe or aux channels.

for now, i will merge this to the develop branch.