Can I make a PR to add support to AllXF serial gimbal? #10857
Replies: 5 comments 5 replies
-
In my opinion, it looks like it might be a reasonable addition. It's not too much code - won't take up very much flash. At a glance, the code seems reasonable. If you make a PR, plan on some others may have a number of comments. Sometimes there is a lot of discussion about a change. |
Beta Was this translation helpful? Give feedback.
-
Do you have a link to this gimbal ? Is this not the XF robots C-20T ? If so this has been supported by INAV 8 since a while now and i am using 2 atm driven by inav via serial uart connection. Its also the same as the caddx gm3 model only differently branded. |
Beta Was this translation helpful? Give feedback.
-
sure ! Not at home right now as flying with it as we speak near the beach but will check when home. But it nothing special i remembered. Just standard rx/tc cross connection , set that port to serial gimbal and specify in the configurator on one of the pages which channel you want mapped to pitch and which to roll. Also make sure you have updated the gimbal to the latest firmware for it to work as inav support in one of the latest versions and have the correct wires connected to your uart
|
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
I found the issue. To check some stuff for you in the gimbal config, i saw mine was running 3.4. So i upgraded to 3.6 and i lost gimbal control. I rolled back to 3.4 it worked again. So 3.6 broke INAV support !!! Online i only found the 3.6 package so i have attached a version 3.4 i downloaded some time ago when i set it up. Will see if i can online inform the guys from XF robot as not sure if they know. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I've bought a AllXF C-20T gimbal, which is similar to Walksnail gimbal but have different data structure , I've add some code but not finished yet (typedef has been made), after I finished add support to AllXF's gimbal, would you ( INAV maintainers ) accept this kind of changes?
Here are my changes:
master...yukihiratype2:inav:yukihiratype2-allxf-serial-gimbal
Beta Was this translation helpful? Give feedback.
All reactions