inav
inav copied to clipboard
iNavFlight
Bidirectional DShot
Current Behavior
To use RPM filter, the ESC telemetry must be connected to the flight controller.
Desired Behavior
Most ESCs with BLHeli_32 and BLHeli_S are supporting bidirectional DShot. The ESC telemetry will be delivered to the flight controller by the motor wire.
Suggested Solution
Betaflight is supporting bidirectional DShot since version 4.0/4.1. It would be great to implement/adapt this feature in iNav.
Who does this impact? Who is this for?
Mostly all multirotor builds which want to take advantage of RPM filter without wiring an additonal line from the ESC to the flight controller.
Additional context
Link to the Betaflight RPM Filter page: https://github.com/betaflight/betaflight/wiki/Bidirectional-DSHOT-and-RPM-Filter
I'm not able to work on this right now, but unless anyone else is working on this, I'm planning to tackle this next.
Be our guest mate!
pt., 23 kwi 2021, 18:48 użytkownik Harry Phillips @.***> napisał:
I'm not able to work on this right now, but unless anyone else is working on this, I'm planning to tackle this next.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/iNavFlight/inav/issues/5549#issuecomment-825784419, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHMBG6PHPTNYYV3EAHQZCLTKGQHVANCNFSM4LV6V4IQ .
@padcom I plan to do it, but it's quite complex and I don't understand the STM32's DMA architecture well enough at the moment. I've written an implementation for ESP32 so that I understand what goes on on the wire, I'm just fighting with DMA. Consider it a far plan, not a near one.
@harry1453 Hi bro ! Any News about this feature ? Are you still working on it ?
Lot of people like me are waiting this to switch from Betaflight to Inav.
Please, implement this option, it would be very useful. Even for esc like the Hobbywing rotor 60a blheli_32 which has no telemetry output.
Thanks guys!
Since now the blheli s esc with bluejay firmware and the blheli32 esc have begun to give not only RPM, but also temperature via bidirection dshot, performing this issue would be very useful.
There is a clone of the repository where this has already been done. Any chance to include it here?
2023, still don't have bdshot supported yet?
There is a clone of the repository where this has already been done. Any chance to include it here?
Hey guys, it's my first comment here. I just switched from Betaflight to iNav as especially as a beginner I love to have the position fix feature to pause or even land easily, quite nice and Betaflight guys seem not to want to invest into this as they "mainly focus on performance" - oh man. Anyway, the bdshot is something which seems you want to have and I was quite sad that this is not supported yet, quite old request though. Any update or plans? Quick question, as Betaflight has integrated it and it's open source as well, isn't it easy to get the underlaying idea from their source code? Why is it so hard to get the best out of these two worlds and let each other inspire a bit?
Hey guys, (...) I just switched from Betaflight to iNav (...) Quick question, as Betaflight has integrated it and it's open source as well, isn't it easy to get the underlaying idea from their source code?
Check this out: https://github.com/iNavFlight/inav/wiki/INAV-for-BetaFlight-users
It is issue is really completed? May be remove support blheli32 telemetry and add bdshot, that use and blheli32, and bluedjay, have rpm, voltage, currency and temperature?
Why remove the telemetry option? That works fine. It would make no sense to remove it.
@DzikuVx has said a few times in his live streams that there is no intent to implement bidirectional DSHOT. I’m sure he mentioned that is can cause issues with the communications. Especially if longer cable runs are used.
I am little confused, the feature request is closed without comment. Is that good or bad, does that mean that it will be implemented or that the feature request will be just ignored?
I am little confused, the feature request is closed without comment. Is that good or bad, does that mean that it will be implemented or that the feature request will be just ignored?
DShot telemetry is supported, but not Bi-directional single-wire telemetry. from https://github.com/iNavFlight/inav/wiki/INAV-for-BetaFlight-users
@DzikuVx @MrD-RC @course-up @r1000ru I think it's closed as inav'll NOT support this feature, PLEASE correct me, if I'm wrong.
a) As I didn't see any PR about bdshort supported right now. b) @MrD-RC has stated "might cause issues with the communications, especially if longer cable runs are used."
Note: Just use 32bit ESC, which has UART telemetry.
So when closing, @DzikuVx should add the correct label, in this case wontfix or similar.