Plane: added RTL_AUTOLAND=3

[tridge]

this allows for easy disabling of the new arming check

[IamPete1]

There is a command long too.

ardupilot/ArduPlane/GCS_Mavlink.cpp

Lines 959 to 965 in 73eabb1

	case MAV_CMD_DO_LAND_START:
	// attempt to switch to next DO_LAND_START command in the mission
	if (plane.mission.jump_to_landing_sequence()) {
	plane.set_mode(plane.mode_auto, ModeReason::GCS_COMMAND);
	return MAV_RESULT_ACCEPTED;
	}
	return MAV_RESULT_FAILED;

For 4.3 I would like to get "auto RTL" mode into plane to match copter, that will remove the RTL_AUTOLAND param.

[WickedShell]

Yeah, you can definetly opt out of this without rtl autoland enabled. I will need an option bit to not run the check, because we plan do land starts, because you can jump to it via mavlink, but on failsafe we want to go to a rally point.

…

On Mon, Mar 21, 2022, 20:09 Peter Hall ***@***.***> wrote: There is a command long too. https://github.com/ArduPilot/ardupilot/blob/73eabb1fd09d1ef28c6692d6746e4eeec4d027df/ArduPlane/GCS_Mavlink.cpp#L959-L965 For 4.3 I would like to get "auto RTL" mode into plane to match copter, that will remove the RTL_AUTOLAND param. — Reply to this email directly, view it on GitHub <#20345 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEKTCH2SQU7DQ6I3HKZ4QDVBE2VZANCNFSM5RJQO2KA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[tridge]

@WickedShell you can just set RTL_AUTOLAND to -1
I'll raise this for EU dev call, possibly drop this PR and create a value for -1 for RTL_AUTOLAND

[tridge]

i've added a RTL_AUTOLAND=3 to disable the warning

[timtuxworth]

Suggest the values for RTL_AUTOLAND are:

Value	Meaning
0	Disable
1	Home - Enable fly HOME or to a Rally point then land
2	Land - Enable go directly to landing sequence
3	Go Around - Disable but allow Arming with DO_LAND_START in the mission for go arounds.

This is because in QGC, you only see the first few letters of each option, as per this screen shot.

[hendjoshsr71]

// @Values: 0:Disable,1:Fly HOME then land,2:Go directly to landing sequence, 3:OnlyForGoAround

I think this should be renamed it confused me and I made a 4: DisableArmCheck or 4: DisableArmCheck_NoRTL. U

I had little reason for checking the extended descriptions since I didn't want a GoAround. I want it for ship landing scripting :)