Skip to content

Conversation

@JazzMaverick
Copy link
Contributor

Rev 16.71 (by JazzMac)
Add Reversed Dshot 300/600
Add Dshot tlm400/800, tlm always after a reversed dshot, no command to start/stop tlm
Rremove other protocols
Stop support BB1(24Mhz)

Rev 16.72
Change timming to tlm375/750
Modify delay between dshot/tlm to 30us to meet spec.
Fix dshot decoder timer0 wrapped issue (dshot packet loss issue).

@JyeSmith
Copy link

JyeSmith commented Dec 12, 2019

Tested 16.72 on SPEDIX ES30 HV 30A today and works as expected.

Edit - and the crazybee f4 Pro

@sskaug sskaug merged commit b73b9b9 into bitdump:master Dec 15, 2019
@Crazor
Copy link

Crazor commented Dec 19, 2019

@JyeSmith Interesting that it worked for you on the crazybee f4 pro. I tried 16.72 and while I got telemetry in Betaflight, I could not get the motors to run stable. In fact, I believe the ESCs crashed every couple of seconds, as I heard the startup tones from the motors. Also I noticed that the startup tones changed pitch (tones got higher), so maybe there's an issue with the µC clock frequency?
I will try 16.71 next..

@asizon
Copy link

asizon commented Dec 19, 2019

@Crazor he test the first 16.72 revision, you have tried 16.72c revision, it have some issues solves in 16.73

@JyeSmith
Copy link

The incorrect 16.72 was merged by accident and I tested an earlier version. Grab 16.73 from JazzMavericks repo and check out RCG for more info.

https://github.com/JazzMaverick/BLHeli/tree/JazzMaverick-patch-1/BLHeli_S%20SiLabs

https://www.rcgroups.com/forums/showthread.php?2640796-BLHeli_S-Smooth-as-Silk

@Crazor
Copy link

Crazor commented Dec 19, 2019

Ah great, 16.73 seems to work (as far as manually spinning up the motors via BetaFlight Configurator goes). Will try and fly this tomorrow. Thank you JyeSmith, and thank you JazzMaverick for your contribution!

@Crazor
Copy link

Crazor commented Dec 19, 2019

Of course I could not wait till tomorrow and already flew a 1s pack in my living room! Smooth as silk! Awesome!

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.

5 participants