ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
10.81k stars 17.26k forks source link

Copter: reboot with MP will load for a long time when set CAN_P1_DRIVER to 1 #27608

Open cuav-chen2 opened 2 months ago

cuav-chen2 commented 2 months ago

Bug report

Issue details In current master branch if compile with copter, when set CAN_P1_DRIVER or CAN_P2_DRIVER to 1 and send a reboot command to autopilot, Mission Planner will reconnect automaticlly and enter a infinite loading state just like that: MP1

I' ve tested in CUAV-X7 and CUAVv5Nano autopilot and both had this issue. To solve this loading problem, you must disconnect and reconnect manually in MP, or click the Retry Now button in CONFIG tabs. When MP staying in loading state, the console window shows that it is reading a 'None' param. I guess it has something to do with the @PARAM/param.pack file downloaded by mavlink on connect. MP_Console

I'm not sure if this is a MP or Ardupilot issue. It seems happened after this PR https://github.com/ArduPilot/ardupilot/pull/26497 .

Version ArduCopter V4.6.0-dev Mission Planner 1.3.81 build 1.3.8741.25556

Platform [ ] All [ ] AntennaTracker [x] Copter [ ] Plane [ ] Rover [ ] Submarine

Logs no logs.

rmackay9 commented 2 months ago

Hi @cuav-chen2,

Thanks very much for the report. I suspect (but don't know for sure) that this is a Mission Planner issue. The way to be sure is for us to test with another ground station like MAVProxy or QGC

cuav-chen2 commented 2 months ago

Hi @rmackay9 , Yes, I have tested with MAVProxy and QGC and both of them worked fine. Maybe this is a MP issue. I also found that if change GPS1_TYPE to any value will solve this issue. But if you reset all paramters to default, the issue will reappear.

rmackay9 commented 2 months ago

Hi @cuav-chen2,

Thanks very much for doing that additional testing. We probably need to re-raise this issue over on the MP issues list then

cuav-chen2 commented 2 months ago

Sure, I'll do that later.