Dshot not working on latest master build on G4

[DusKing1]

Describe the bug

Any Dshot settings (600/300, bidir\no-bidir) does not working on currently master build on G47x target.

To Reproduce

@SteveCEvans grab that AirFleetV11 dev board, flash with latest master build, dshot just won't work.

Expected behavior

Dshot300 can work both in bidir mode and no-bidir mode.

Flight controller configuration

Just use NERC-HYBRIDG4.config

[github-actions]

@DusKing1: This issue is being automatically closed because it does not follow the template.

When you open an issue or feature request you are presented with a template. Follow the guidelines.

You can edit your message to fix this and the issue will be automatically reopened.

[DusKing1]

@github-actions fk off

[DusKing1]

@SteveCEvans any spare time for this?

[SteveCEvans]

Confirmed, connected to a Tekko32 F3 Metal 4in1 ESC. Also not getting any ESC sensor data. BLHeli configurator is able to communicate though.

[SteveCEvans]

@DusKing1 Has this ever worked to your knowledge? If so please provide me with the hash of the build.

[DusKing1]

@SteveCEvans Can't provide commit hash, ATM, but I remember build 2476 can work nicely.

[SteveCEvans]

3d5a0e585cc6a3e9db50b3b1b50602e389511983

Not working for me I'm afraid. @DusKing1 @asizon please check.

# dshot_telemetry_info
Dshot reads: 318044
Dshot invalid pkts: 318044
Dshot directionChange cycles: 0, micros: 0

Motor      eRPM      RPM      Hz   Invalid
=====   =======   ======   =====   =======
    0         0        0       0   NO DATA
    1         0        0       0   NO DATA
    2         0        0       0   NO DATA
    3         0        0       0   NO DATA

0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 
0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 

Also, BLHeli configurator isn't working very well with that build.

[asizon]

Doesnt work for mine with this hash

# dshot_telemetry_info
Dshot reads: 0
Dshot invalid pkts: 0
Dshot directionChange cycles: 0, micros: 0

Motor      eRPM      RPM      Hz   Invalid
=====   =======   ======   =====   =======
    0         0        0       0   NO DATA
    1         0        0       0   NO DATA
    2         0        0       0   NO DATA
    3         0        0       0   NO DATA

0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 
0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 

[asizon]

with last master i received dshot reads:

# dshot_telemetry_info
Dshot reads: 243967
Dshot invalid pkts: 243967
Dshot directionChange cycles: 0, micros: 0

Motor      eRPM      RPM      Hz   Invalid
=====   =======   ======   =====   =======
    0         0        0       0   NO DATA
    1         0        0       0   NO DATA
    2         0        0       0   NO DATA
    3         0        0       0   NO DATA

0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 
0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 

[asizon]

Definitelly it works as expected with last master. Dshot bidir works!Now i have flashed my airflet (v4?) with netronrcG4Aio target.

# dshot_telemetry_info
Dshot reads: 1091419
Dshot invalid pkts: 43159
Dshot directionChange cycles: 396, micros: 2

[asizon]

Then @DusKing1 you should check this config DMA resources :) and dont forget to use last Configurator dev build.

[DusKing1]

@asizon no we are talking about a different target.

[DusKing1]

@SteveCEvans I'm tracing back, will give you the hash later

[asizon]

@DusKing1 no hash meeded, master is working correctly, you should chech your target dma config or hardware for this board.

[DusKing1]

@SteveCEvans how about this, the first version when we have the BMI270 driver code for all unified target. I did bunch of tests on this build. e0ad71db2b9fe9f561daed0345cd4f62ae5b8525

[SteveCEvans]

@dusking I've tested with both current master and the build you reference above, and neither work with bit banged dshot.

With set dshot_bitbang = OFF the motor RPM is reported as expected. With set dshot_bitbang = ON there is no motor output whatsoever (as seen on a logic analyzer connected to the MOTOR 1 pin). Investigating.

[SteveCEvans]

Bitbanged DSHOT doesn't work irrespective of bidirectional being used. The only difference is that the line sits low with is disabled, high with it enabled.

[SteveCEvans]

@haslinghuis I'm reopening this as this is broken functionality and the unified target change is no more than a work around.

There's some clearly bogus code HERE which I'm investigating.

[daleckystepan]

Should be fixed. Can you retest?

[DusKing1]

@daleckystepan I'm a bit outdated here and what exact test I should be do?

[daleckystepan]

@DusKing1 Probably testing dhsot and bi dshot on G4 target.

Question is if it works and if we can close this issue. Thank you.

[haslinghuis]

Please check with latest firmware / configurator

[DusKing1]

Ok I will try to arrange some tests for this, assign this to me please

[DusKing1]

Dshot works normally now.

But I found that OSD still have some problem. #11539