bitdump / BLHeli

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

BLHeli_32.10 Low Voltage Protection Incorrect with LIHV Batteries #719

Open RNewDeal opened 8 months ago

RNewDeal commented 8 months ago

The low voltage protection thinks an LIHV battery is not an LIHV battery but a Lipo battery but cell count +1.

I tested a 3S LIHV with a voltage cutoff of 2.8V per cell and it would drop the throttle command to keep the total voltage at 11.2V, which is the low voltage cutoff if the battery was a 4S LiPo.

This problem was also seen on version 32.9

sskaug commented 8 months ago

Thank you for reporting. Yes, you are right - we have not done anything to the cell detect code for a long time, and at the time of design we did not consider LIHV as it was not much present. We can make a testcode that also considers LIHV - which exact firmware does your ESC use?

RNewDeal commented 8 months ago

Thank you for reporting. Yes, you are right - we have not done anything to the cell detect code for a long time, and at the time of design we did not consider LIHV as it was not much present. We can make a testcode that also considers LIHV - which exact firmware does your ESC use?

My ESC uses the Spedix 30A ESC firmware. An overall voltage cutoff instead of a per cell voltage cutoff would fix this also.

sskaug commented 8 months ago

Does it run the Spedix_GS30A_V1_1 code?

RNewDeal commented 8 months ago

Does it run the Spedix_GS30A_V1_1 code?

Yes

sskaug commented 8 months ago

Please try the testcode now posted here: https://github.com/bitdump/BLHeli/tree/master/BLHeli_32%20ARM/Misc%20testcodes

RNewDeal commented 8 months ago

Please try the testcode now posted here: https://github.com/bitdump/BLHeli/tree/master/BLHeli_32%20ARM/Misc%20testcodes

New testcode worked in my testing. I did not try it with a partially charged battery though.