Make -K -M and -G use AOA in OTG mode

For convenience, short options were added to select UHID input modes:
 - -K for --keyboard=uhid
 - -M for --mouse=uhid
 - -G for --gamepad=uhid

In OTG mode, UHID is not available, so the short options should select
AOA instead.
This commit is contained in:
Romain Vimont 2024-09-07 23:06:16 +02:00
parent b0b05909c0
commit ccc030625b
3 changed files with 26 additions and 10 deletions

View File

@ -176,8 +176,8 @@ Start in fullscreen.
Do not attempt to use "adb reverse" to connect to the device. Do not attempt to use "adb reverse" to connect to the device.
.TP .TP
.B \-K .B \-G
Same as \fB\-\-gamepad=uhid\fR. Same as \fB\-\-gamepad=uhid\fR, or \fB\-\-keyboard=aoa\fR if \fB\-\-otg\fR is set.
.TP .TP
.BI "\-\-gamepad " mode .BI "\-\-gamepad " mode
@ -196,7 +196,7 @@ Print this help.
.TP .TP
.B \-K .B \-K
Same as \fB\-\-keyboard=uhid\fR. Same as \fB\-\-keyboard=uhid\fR, or \fB\-\-keyboard=aoa\fR if \fB\-\-otg\fR is set.
.TP .TP
.BI "\-\-keyboard " mode .BI "\-\-keyboard " mode
@ -261,7 +261,7 @@ Default is 0 (unlimited).
.TP .TP
.B \-M .B \-M
Same as \fB\-\-mouse=uhid\fR. Same as \fB\-\-mouse=uhid\fR, or \fB\-\-mouse=aoa\fR if \fB\-\-otg\fR is set.
.TP .TP
.BI "\-\-max\-fps " value .BI "\-\-max\-fps " value

View File

@ -375,7 +375,7 @@ static const struct sc_option options[] = {
}, },
{ {
.shortopt = 'G', .shortopt = 'G',
.text = "Same as --gamepad=uhid.", .text = "Same as --gamepad=uhid, or --gamepad=aoa if --otg is set.",
}, },
{ {
.longopt_id = OPT_GAMEPAD, .longopt_id = OPT_GAMEPAD,
@ -397,7 +397,7 @@ static const struct sc_option options[] = {
}, },
{ {
.shortopt = 'K', .shortopt = 'K',
.text = "Same as --keyboard=uhid.", .text = "Same as --keyboard=uhid, or --keyboard=aoa if --otg is set.",
}, },
{ {
.longopt_id = OPT_KEYBOARD, .longopt_id = OPT_KEYBOARD,
@ -493,7 +493,7 @@ static const struct sc_option options[] = {
}, },
{ {
.shortopt = 'M', .shortopt = 'M',
.text = "Same as --mouse=uhid.", .text = "Same as --mouse=uhid, or --mouse=aoa if --otg is set.",
}, },
{ {
.longopt_id = OPT_MAX_FPS, .longopt_id = OPT_MAX_FPS,
@ -2264,7 +2264,7 @@ parse_args_with_getopt(struct scrcpy_cli_args *args, int argc, char *argv[],
args->help = true; args->help = true;
break; break;
case 'K': case 'K':
opts->keyboard_input_mode = SC_KEYBOARD_INPUT_MODE_UHID; opts->keyboard_input_mode = SC_KEYBOARD_INPUT_MODE_UHID_OR_AOA;
break; break;
case OPT_KEYBOARD: case OPT_KEYBOARD:
if (!parse_keyboard(optarg, &opts->keyboard_input_mode)) { if (!parse_keyboard(optarg, &opts->keyboard_input_mode)) {
@ -2286,7 +2286,7 @@ parse_args_with_getopt(struct scrcpy_cli_args *args, int argc, char *argv[],
} }
break; break;
case 'M': case 'M':
opts->mouse_input_mode = SC_MOUSE_INPUT_MODE_UHID; opts->mouse_input_mode = SC_MOUSE_INPUT_MODE_UHID_OR_AOA;
break; break;
case OPT_MOUSE: case OPT_MOUSE:
if (!parse_mouse(optarg, &opts->mouse_input_mode)) { if (!parse_mouse(optarg, &opts->mouse_input_mode)) {
@ -2671,7 +2671,7 @@ parse_args_with_getopt(struct scrcpy_cli_args *args, int argc, char *argv[],
opts->audio_dup = true; opts->audio_dup = true;
break; break;
case 'G': case 'G':
opts->gamepad_input_mode = SC_GAMEPAD_INPUT_MODE_UHID; opts->gamepad_input_mode = SC_GAMEPAD_INPUT_MODE_UHID_OR_AOA;
break; break;
case OPT_GAMEPAD: case OPT_GAMEPAD:
if (!parse_gamepad(optarg, &opts->gamepad_input_mode)) { if (!parse_gamepad(optarg, &opts->gamepad_input_mode)) {
@ -2795,7 +2795,12 @@ parse_args_with_getopt(struct scrcpy_cli_args *args, int argc, char *argv[],
if (opts->keyboard_input_mode == SC_KEYBOARD_INPUT_MODE_AUTO) { if (opts->keyboard_input_mode == SC_KEYBOARD_INPUT_MODE_AUTO) {
opts->keyboard_input_mode = otg ? SC_KEYBOARD_INPUT_MODE_AOA opts->keyboard_input_mode = otg ? SC_KEYBOARD_INPUT_MODE_AOA
: SC_KEYBOARD_INPUT_MODE_SDK; : SC_KEYBOARD_INPUT_MODE_SDK;
} else if (opts->keyboard_input_mode
== SC_KEYBOARD_INPUT_MODE_UHID_OR_AOA) {
opts->keyboard_input_mode = otg ? SC_KEYBOARD_INPUT_MODE_AOA
: SC_KEYBOARD_INPUT_MODE_UHID;
} }
if (opts->mouse_input_mode == SC_MOUSE_INPUT_MODE_AUTO) { if (opts->mouse_input_mode == SC_MOUSE_INPUT_MODE_AUTO) {
if (otg) { if (otg) {
opts->mouse_input_mode = SC_MOUSE_INPUT_MODE_AOA; opts->mouse_input_mode = SC_MOUSE_INPUT_MODE_AOA;
@ -2805,16 +2810,24 @@ parse_args_with_getopt(struct scrcpy_cli_args *args, int argc, char *argv[],
} else { } else {
opts->mouse_input_mode = SC_MOUSE_INPUT_MODE_SDK; opts->mouse_input_mode = SC_MOUSE_INPUT_MODE_SDK;
} }
} else if (opts->mouse_input_mode == SC_MOUSE_INPUT_MODE_UHID_OR_AOA) {
opts->mouse_input_mode = otg ? SC_MOUSE_INPUT_MODE_AOA
: SC_MOUSE_INPUT_MODE_SDK;
} else if (opts->mouse_input_mode == SC_MOUSE_INPUT_MODE_SDK } else if (opts->mouse_input_mode == SC_MOUSE_INPUT_MODE_SDK
&& !opts->video_playback) { && !opts->video_playback) {
LOGE("SDK mouse mode requires video playback. Try --mouse=uhid."); LOGE("SDK mouse mode requires video playback. Try --mouse=uhid.");
return false; return false;
} }
if (opts->gamepad_input_mode == SC_GAMEPAD_INPUT_MODE_AUTO) { if (opts->gamepad_input_mode == SC_GAMEPAD_INPUT_MODE_AUTO) {
// UHID does not work on all devices (with old Android // UHID does not work on all devices (with old Android
// versions), so it cannot be enabled by default // versions), so it cannot be enabled by default
opts->gamepad_input_mode = otg ? SC_GAMEPAD_INPUT_MODE_AOA opts->gamepad_input_mode = otg ? SC_GAMEPAD_INPUT_MODE_AOA
: SC_GAMEPAD_INPUT_MODE_DISABLED; : SC_GAMEPAD_INPUT_MODE_DISABLED;
} else if (opts->gamepad_input_mode
== SC_GAMEPAD_INPUT_MODE_UHID_OR_AOA) {
opts->gamepad_input_mode = otg ? SC_GAMEPAD_INPUT_MODE_AOA
: SC_GAMEPAD_INPUT_MODE_UHID;
} }
} }

View File

@ -142,6 +142,7 @@ enum sc_lock_video_orientation {
enum sc_keyboard_input_mode { enum sc_keyboard_input_mode {
SC_KEYBOARD_INPUT_MODE_AUTO, SC_KEYBOARD_INPUT_MODE_AUTO,
SC_KEYBOARD_INPUT_MODE_UHID_OR_AOA, // normal vs otg mode
SC_KEYBOARD_INPUT_MODE_DISABLED, SC_KEYBOARD_INPUT_MODE_DISABLED,
SC_KEYBOARD_INPUT_MODE_SDK, SC_KEYBOARD_INPUT_MODE_SDK,
SC_KEYBOARD_INPUT_MODE_UHID, SC_KEYBOARD_INPUT_MODE_UHID,
@ -150,6 +151,7 @@ enum sc_keyboard_input_mode {
enum sc_mouse_input_mode { enum sc_mouse_input_mode {
SC_MOUSE_INPUT_MODE_AUTO, SC_MOUSE_INPUT_MODE_AUTO,
SC_MOUSE_INPUT_MODE_UHID_OR_AOA, // normal vs otg mode
SC_MOUSE_INPUT_MODE_DISABLED, SC_MOUSE_INPUT_MODE_DISABLED,
SC_MOUSE_INPUT_MODE_SDK, SC_MOUSE_INPUT_MODE_SDK,
SC_MOUSE_INPUT_MODE_UHID, SC_MOUSE_INPUT_MODE_UHID,
@ -158,6 +160,7 @@ enum sc_mouse_input_mode {
enum sc_gamepad_input_mode { enum sc_gamepad_input_mode {
SC_GAMEPAD_INPUT_MODE_AUTO, SC_GAMEPAD_INPUT_MODE_AUTO,
SC_GAMEPAD_INPUT_MODE_UHID_OR_AOA, // normal vs otg mode
SC_GAMEPAD_INPUT_MODE_DISABLED, SC_GAMEPAD_INPUT_MODE_DISABLED,
SC_GAMEPAD_INPUT_MODE_UHID, SC_GAMEPAD_INPUT_MODE_UHID,
SC_GAMEPAD_INPUT_MODE_AOA, SC_GAMEPAD_INPUT_MODE_AOA,