bitdump / BLHeli

BLHeli for brushless ESC firmware
GNU General Public License v3.0
1.96k stars 1.09k forks source link

Feature request: make an option to forcibly specify supported ESC protocol #384

Open voroshkov opened 5 years ago

voroshkov commented 5 years ago

Recently different people with different hardware started experiencing intermittent self-arming issues after crashes or drones being exposed to water.

What if we could forcibly set the DShot protocol as the only one accepted by ESC in BLHeli settings? Would it make our drones less prone to self-arming in damaged condition?

WhiteWind4 commented 5 years ago

Many times seen this problem on the wet drones.

sskaug commented 5 years ago

This is good input. Particularly the old pwm formats are susceptible to false detection. At the moment we have a Rev32.6.6 test code out that we plan to release as a Rev32.7 full release in a few weeks, after proper testing, and I do not want to delay that or put that at risk by adding more functionality. But after that rev is out, we can look into this.

voroshkov commented 5 years ago

@sskaug, Great! Thank you!

mikeller commented 5 years ago

Fully supporting this. We've had issues reported against Betaflight by users experiencing unrequested motor spinups when selecting Dshot1200, which on second thought could be caused by capacitive effects on the motor control line causing the Dshot1200 frame to be detected as a PWM pulse.