mavlink / qgroundcontrol

Cross-platform ground control station for drones (Android, iOS, Mac OS, Linux, Windows)
http://qgroundcontrol.io
3.1k stars 3.46k forks source link

Configurable serial ports #6905

Open dagar opened 5 years ago

dagar commented 5 years ago

In PX4 (https://github.com/PX4/Firmware/pull/10337) we have different modules and drivers that can be configured to use serial ports. As a result it's currently possible to have a conflict configured via parameters.

On the QGC side I'd like to add a mechanism to configure the usage per port, where QGC handles the deconfliction.

PX4

For example in PX4 we have the following parameters to configure the ports used by different mavlink instances or different GPS. Each parameter has the same drop down list of serial ports.

QGC

On the QGC side I'd like to invert the configuration and prevent conflicts. Each serial port should only appear in a single parameter.

hamishwillee commented 5 years ago

Just FMI, but would this be the only tuning page for those parameters, or would people still be able to see the underlying MAV_3_CONFIG, MAV_2_CONFIG etc?

jafrado commented 5 years ago

Would be great to have this ... As it stands, it's documented like this, but no way to configure com ports now: https://docs.px4.io/en/peripherals/serial_configuration.html https://docs.px4.io/en/peripherals/mavlink_peripherals.html

hamishwillee commented 5 years ago

As it stands, it's documented like this, but no way to configure com ports now:

@jafrado What do you mean - those documents show exactly how to configure them. It just isn't very intuitive.

All this issue asks is to be able to choose the associated service running on particular port (rather than starting selecting the port used by each service.)

jafrado commented 5 years ago

Doesn't work with 1.8.2 and v3.5, tried today. As it stands I added mavlink commands to my extras.txt

hamishwillee commented 5 years ago

@jafrado For 1.8.2 you would use SYS_COMPANION for this. In 1.9 and later you use the docs above. For a bit of guidance you can see the PR for the change here: https://github.com/PX4/px4_user_guide/pull/352/files

@dagar Can we run out versions for PX4 User Guide too so I can point to correct docs for this stuff?

jafrado commented 5 years ago

Got it, thanks.

DronecodeBot commented 12 months ago

This issue has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there:

https://discuss.px4.io/t/confusion-on-definition-of-telem-1-telem-2-etc/33424/1