-
-
Notifications
You must be signed in to change notification settings - Fork 120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cannot connect to Mission Planner (works on QGroundControl) in ESPNOW and in WifiLR #93
Comments
Thank you for the detailed report! This is great :D I will have a look at it in the coming days. |
Hi! I was able to reproduce the error. It all works fine using the regular firmware + FTDI, but MissionPlanner failed when using the USBSerial firmware. |
Some additional information: |
Hi there, I'm one of the core developers at ArduPilot and we've also noticed this issue and would like to help figure out a fix. I've sent this over to Michael Oborne who knows best about Mission Planner to see if he has any ideas. |
I think I found the reason. It is a similar reason why you need to reboot the esp32 once you close QGroundControl. When MissionPlanner opens the serial port it triggers the RTS line of the USB. This reboots the ESP32. Alternatively (and this would be a hack, more or less), MissionPlanner could increase the timeout. The ESP32 will reboot and start sending anyway. The ESP32 will reboot when RTS is triggered. (Mission Planner when connecting & with "Connect Reset" not set) This behaviour is programmed into the chip more or less. There is nothing I can do on the ESP32 side. |
Thanks for the update. |
Hi @seeul8er, Thanks for the investigation! I'll pass this onto the MP developers to see if they have any ideas |
I talked with MichaelO and he thinks that "Connect Reset" checkbox should help but he also thinks that "dts" (aka RTS/CTS) shouldn't be enabled by default. This is the line in MissionPlanner that uses the checkbox In any case, we're going to test some more and I hope we can modify MP assuming that's where the issue is |
We've also found that the checkbox has no impact on the issue so looking into alternative fixes |
I tested this pull request since it looked promising: ArduPilot/MissionPlanner#3260 |
Hi @seeul8er, great, thanks! One of the MP devs is going to look into this as well once his ESP32-C3 modules arrive which should be in about a week. |
We've got a workaround on the MP side here now ArduPilot/MissionPlanner#3469 and an executable can be downloaded from this page. Hopefully this change will get incorporated into MP in the near future. I think EosBandi understands what's going on and also has some suggestions for the DroneBridge side to make it all work more smoothly. Anyway, I think that's coming in the near-ish future. |
@rmackay9 Fantastic! Thank you for pushing this topic. I will make sure to update the Wiki accordingly. |
Issue:
Using WifiLR in air and on ground serial USB ESP32.
Connects to QGroundcontrol but not to Mission Planner.
Using ESPnow in air and on ground serial USB ESP32:
Connects to QGroundcontrol but not to Mission Planner.
Computer:
Win 11 (Microsoft Surface Pro 9)
GCS software:
Mission Planner 1.3.82
QGroundControl 4.4.2
Flight controller and Ardupilot firmware version:
12/2/2024 4:05:29 PM : Frame: QUAD/X
12/2/2024 4:05:29 PM : RCOut: PWM:1-4 DS600:5-8 PWM:9-12
12/2/2024 4:05:29 PM : MatekF405-TE 0046003D 50315015 2033334E
12/2/2024 4:05:29 PM : ChibiOS: 6a85082c
12/2/2024 4:05:29 PM : ArduCopter V4.5.7 (2a3dc4b7)
Matek Flight Controller F405-WMN
Serial parameters:
SERIAL1_BAUD,57
SERIAL1_OPTIONS,0
SERIAL1_PROTOCOL,2
Dronebridge:
In the air:
DroneBridge for ESP32 - v2.0 RC4
ESP32C3 XIAO
Parameters exactly as shown on config page.
UART TX 5
UART RX 4
RTS PIN 0
CTS PIN 0
Protocol MAVLINK
Baud 57k
Max packet size 64
Serial read timeout (ms) 50
On the ground:
DroneBridge for ESP32 - v2.0 v2.0 RC4 SERIAL
ESP32C3 XIAO
Connected to PC on USB connector. (Not using FTDI adapter)
Parameters exactly as shown on config page.
Protocol MAVLINK
Baud 57k
Max packet size 64
Serial read timeout (ms) 50
The text was updated successfully, but these errors were encountered: