Map users to core ports on first button press

The first user to press a button is mapped to core port 1 (player 1),
the second is mapped to core port 2 (player 2), and so on.

Configured by the input_remap_on_button_press option. When enabled, all
users are unmapped by default (assigned to 'None'/MAX_USERS).

Author: nijm

config.def.h

@@ -1124,6 +1124,10 @@
  * input remap file */
 #define DEFAULT_NOTIFICATION_SHOW_REMAP_LOAD true
 
+/* Display a notification when a user is mapped to a core
+ * port on first button press */
+#define DEFAULT_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT true
+
 /* Display a notification when loading a
  * configuration override file */
 #define DEFAULT_NOTIFICATION_SHOW_CONFIG_OVERRIDE_LOAD true
@@ -1620,6 +1624,10 @@
  * gamepads, plug-and-play style. */
 #define DEFAULT_INPUT_AUTODETECT_ENABLE true
 
+/* Map user to core port on first button press. First person
+ * to press a button is player 1, second is player 2, etc. */
+#define DEFAULT_INPUT_REMAP_PORTS_ON_BUTTON_PRESS false
+
 /* Enables accelerometer/gyroscope/illuminance
  * sensor input, if supported */
 #if defined(ANDROID)

configuration.c

@@ -1934,6 +1934,7 @@ static struct config_bool_setting *populate_settings_bool(
    SETTING_BOOL("notification_show_cheats_applied", &settings->bools.notification_show_cheats_applied, true, DEFAULT_NOTIFICATION_SHOW_CHEATS_APPLIED, false);
    SETTING_BOOL("notification_show_patch_applied", &settings->bools.notification_show_patch_applied, true, DEFAULT_NOTIFICATION_SHOW_PATCH_APPLIED, false);
    SETTING_BOOL("notification_show_remap_load",  &settings->bools.notification_show_remap_load, true, DEFAULT_NOTIFICATION_SHOW_REMAP_LOAD, false);
+   SETTING_BOOL("notification_show_user_mapped_to_core_port",  &settings->bools.notification_show_user_mapped_to_core_port, true, DEFAULT_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT, false);
    SETTING_BOOL("notification_show_config_override_load", &settings->bools.notification_show_config_override_load, true, DEFAULT_NOTIFICATION_SHOW_CONFIG_OVERRIDE_LOAD, false);
    SETTING_BOOL("notification_show_set_initial_disk", &settings->bools.notification_show_set_initial_disk, true, DEFAULT_NOTIFICATION_SHOW_SET_INITIAL_DISK, false);
    SETTING_BOOL("notification_show_disk_control", &settings->bools.notification_show_disk_control, true, DEFAULT_NOTIFICATION_SHOW_DISK_CONTROL, false);
@@ -2177,6 +2178,7 @@ static struct config_bool_setting *populate_settings_bool(
    SETTING_BOOL("input_turbo_allow_dpad",        &settings->bools.input_turbo_allow_dpad, true, DEFAULT_TURBO_ALLOW_DPAD, false);
    SETTING_BOOL("input_auto_mouse_grab",         &settings->bools.input_auto_mouse_grab, true, DEFAULT_INPUT_AUTO_MOUSE_GRAB, false);
    SETTING_BOOL("input_remap_binds_enable",      &settings->bools.input_remap_binds_enable, true, true, false);
+   SETTING_BOOL("input_remap_ports_on_button_press",   &settings->bools.input_remap_ports_on_button_press, true, DEFAULT_INPUT_REMAP_PORTS_ON_BUTTON_PRESS, false);
    SETTING_BOOL("input_remap_sort_by_controller_enable",      &settings->bools.input_remap_sort_by_controller_enable, true, false, false);
    SETTING_BOOL("input_hotkey_device_merge",     &settings->bools.input_hotkey_device_merge, true, DEFAULT_INPUT_HOTKEY_DEVICE_MERGE, false);
 #ifdef HAVE_MENU

configuration.h

@@ -702,6 +702,7 @@ typedef struct settings
       /* Input */
       bool input_remap_binds_enable;
       bool input_remap_sort_by_controller_enable;
+      bool input_remap_ports_on_button_press;
       bool input_autodetect_enable;
       bool input_sensors_enable;
       bool input_overlay_enable;
@@ -757,6 +758,7 @@ typedef struct settings
       bool notification_show_cheats_applied;
       bool notification_show_patch_applied;
       bool notification_show_remap_load;
+      bool notification_show_user_mapped_to_core_port;
       bool notification_show_config_override_load;
       bool notification_show_set_initial_disk;
       bool notification_show_disk_control;

input/input_driver.c

@@ -5990,6 +5990,104 @@ static void input_keys_pressed(
       input_st->input_hotkey_block_counter = 0;
 }
 
+/* Ignore d-pad buttons to avoid false positives.
+ * Some wireless gamepad receivers report 0 values for all axes when the
+ * gamepad is powered off or not yet connected. Since 0 represents the
+ * "left" and "up" positions for d-pad axes, this creates false button
+ * presses that could trigger unwanted input mapping.
+ */
+static bool any_button_ex_dpad_pressed(int32_t state)
+{
+   static const uint32_t button_mask =
+      (1 << RETRO_DEVICE_ID_JOYPAD_A)     |
+      (1 << RETRO_DEVICE_ID_JOYPAD_B)     |
+      (1 << RETRO_DEVICE_ID_JOYPAD_X)     |
+      (1 << RETRO_DEVICE_ID_JOYPAD_Y)     |
+      (1 << RETRO_DEVICE_ID_JOYPAD_L)     |
+      (1 << RETRO_DEVICE_ID_JOYPAD_R)     |
+      (1 << RETRO_DEVICE_ID_JOYPAD_L2)    |
+      (1 << RETRO_DEVICE_ID_JOYPAD_R2)    |
+      (1 << RETRO_DEVICE_ID_JOYPAD_L3)    |
+      (1 << RETRO_DEVICE_ID_JOYPAD_R3)    |
+      (1 << RETRO_DEVICE_ID_JOYPAD_START) |
+      (1 << RETRO_DEVICE_ID_JOYPAD_SELECT);
+
+   return (state & button_mask) != 0;
+}
+
+/* Returns MAX_USERS if all core ports are mapped to a user. */
+static unsigned get_first_free_core_port(settings_t *settings)
+{
+   unsigned core_port_idx;
+   for (core_port_idx = 0; core_port_idx < MAX_USERS; core_port_idx++)
+   {
+      unsigned *core_port_users = settings->uints.input_remap_port_map[core_port_idx];
+      if (core_port_users[0] == MAX_USERS)
+         return core_port_idx;
+   }
+
+   return MAX_USERS;
+}
+
+static void map_user_to_first_free_core_port(unsigned user_idx, settings_t *settings)
+{
+   unsigned core_port_idx;
+   unsigned joypad_idx;
+   const char *device_display_name;
+   char msg[128];
+
+   msg[0] = '\0';
+
+   if (!settings || user_idx >= MAX_USERS)
+   {
+      RARCH_ERR("[Automap] Invalid parameters\n");
+      return;
+   }
+
+   /* Check if user is already mapped */
+   if (settings->uints.input_remap_ports[user_idx] < MAX_USERS)
+   {
+      RARCH_LOG("[Automap] User %u already mapped to core port %u\n", 
+            user_idx, settings->uints.input_remap_ports[user_idx]);
+      return;
+   }
+
+   core_port_idx = get_first_free_core_port(settings);
+   joypad_idx = settings->uints.input_joypad_index[user_idx];
+   device_display_name = input_config_get_device_display_name(joypad_idx);
+
+   if (string_is_empty(device_display_name))
+      device_display_name = msg_hash_to_str(MENU_ENUM_LABEL_VALUE_NOT_AVAILABLE);
+
+   if (core_port_idx < MAX_USERS)
+   {
+      configuration_set_uint(settings,
+            settings->uints.input_remap_ports[user_idx], core_port_idx);
+      configuration_set_uint(settings,
+            settings->uints.input_libretro_device[core_port_idx], RETRO_DEVICE_JOYPAD);
+
+      input_remapping_update_port_map();
+
+      RARCH_LOG("[Automap] %s (user %u) mapped to core port %u\n",
+            device_display_name, user_idx, core_port_idx);
+
+      if (settings->bools.notification_show_user_mapped_to_core_port)
+      {
+         snprintf(msg, sizeof(msg),
+               msg_hash_to_str(MSG_USER_MAPPED_TO_CORE_PORT_NR),
+               device_display_name,
+               core_port_idx + 1);
+         runloop_msg_queue_push(msg, strlen(msg), 1, 180, true, NULL,
+               MESSAGE_QUEUE_ICON_DEFAULT, MESSAGE_QUEUE_CATEGORY_INFO);
+      }
+   }
+   else
+   {
+      RARCH_LOG("[Automap] No free core ports - %s (user %u) not mapped\n",
+            device_display_name, user_idx);
+   }
+}
+
 void input_driver_poll(void)
 {
    size_t i, j;
@@ -6006,9 +6104,15 @@ void input_driver_poll(void)
       *sec_joypad                 = NULL;
 #endif
    bool input_remap_binds_enable  = settings->bools.input_remap_binds_enable;
+   bool input_remap_ports_on_button_press
+                                  = settings->bools.input_remap_ports_on_button_press;
    float input_axis_threshold     = settings->floats.input_axis_threshold;
    uint8_t max_users              = (uint8_t)settings->uints.input_max_users;
 
+#ifdef HAVE_MENU
+   input_remap_ports_on_button_press = input_remap_ports_on_button_press && !(menu_state_get_ptr()->flags & MENU_ST_FLAG_ALIVE);
+#endif
+
    if (joypad && joypad->poll)
       joypad->poll();
    if (sec_joypad && sec_joypad->poll)
@@ -6040,6 +6144,22 @@ void input_driver_poll(void)
       joypad_info[i].joy_idx               = settings->uints.input_joypad_index[i];
       joypad_info[i].auto_binds            = input_autoconf_binds[joypad_info[i].joy_idx];
 
+      if (input_remap_ports_on_button_press)
+      {
+         if (!(settings->uints.input_remap_ports[i] < MAX_USERS))
+         {
+            int32_t buttons_st = input_state_wrap(input_st->current_driver, input_st->current_data,
+                     joypad, sec_joypad, &joypad_info[i],
+                     (*input_st->libretro_input_binds),
+                     (input_st->flags & INP_FLAG_KB_MAPPING_BLOCKED) ? true : false,
+                     (unsigned)i,
+                     RETRO_DEVICE_JOYPAD, 0, RETRO_DEVICE_ID_JOYPAD_MASK);
+
+            if (any_button_ex_dpad_pressed(buttons_st))
+               map_user_to_first_free_core_port(i, settings);
+         }
+      }
+
       input_st->turbo_btns.frame_enable[i] =
                (*input_st->libretro_input_binds[i])[button_id].valid
             && settings->bools.input_turbo_enable ?
@@ -6672,6 +6792,11 @@ void input_remapping_update_port_map(void)
          port_map_index[remap_port]++;
       }
    }
+
+   /* Changing mapped port may leave a core port unused;
+    * reinitialise controllers to ensure that any such
+    * ports are set to 'RETRO_DEVICE_NONE' */
+   command_event(CMD_EVENT_CONTROLLER_INIT, NULL);
 }
 
 void input_remapping_deinit(bool save_remap)
@@ -6690,6 +6815,29 @@ void input_remapping_deinit(bool save_remap)
                                |    RUNLOOP_FLAG_REMAPS_GAME_ACTIVE);
 }
 
+void input_remapping_set_remap_ports_defaults(void)
+{
+   unsigned i;
+   settings_t *settings = config_get_ptr();
+   bool remap_on_button_press = settings->bools.input_remap_ports_on_button_press;
+
+   for (i = 0; i < MAX_USERS; i++)
+   {
+      configuration_set_uint(settings,
+         settings->uints.input_remap_ports[i],
+         remap_on_button_press ? MAX_USERS : i);
+
+      configuration_set_uint(settings,
+         settings->uints.input_libretro_device[i],
+         remap_on_button_press ? RETRO_DEVICE_NONE : RETRO_DEVICE_JOYPAD);
+   }
+
+   /* Need to call 'input_remapping_update_port_map()'
+    * whenever 'settings->uints.input_remap_ports'
+    * is modified */
+   input_remapping_update_port_map();
+}
+
 void input_remapping_set_defaults(bool clear_cache)
 {
    unsigned i, j;
@@ -6714,16 +6862,9 @@ void input_remapping_set_defaults(bool clear_cache)
       for (j = RARCH_FIRST_CUSTOM_BIND; j < (RARCH_FIRST_CUSTOM_BIND + 8); j++)
          configuration_set_uint(settings,
                settings->uints.input_remap_ids[i][j], j);
-
-      /* Controller port remaps */
-      configuration_set_uint(settings,
-            settings->uints.input_remap_ports[i], i);
    }
 
-   /* Need to call 'input_remapping_update_port_map()'
-    * whenever 'settings->uints.input_remap_ports'
-    * is modified */
-   input_remapping_update_port_map();
+   input_remapping_set_remap_ports_defaults();
 
    /* Restore 'global' settings that were cached on
     * the last core init

input/input_remapping.h

@@ -92,6 +92,30 @@ void input_remapping_deinit(bool save_remap);
  */
 void input_remapping_set_defaults(bool clear_cache);
 
+/**
+ * Used to set the default port mapping values within the `settings` struct.
+ * Only sets port mapping related values, not button/keyboard/analog remaps.
+ *
+ * If settings->bools.input_remap_ports_on_button_press is true, leave all
+ * users/devices disconnected. Users will be mapped to a core port on first
+ * button press.
+ *
+ * user 0 -> core port MAX_USERS (RETRO_DEVICE_NONE)
+ * user 1 -> core port MAX_USERS (RETRO_DEVICE_NONE)
+ * ...
+ * user n -> core port MAX_USERS (RETRO_DEVICE_NONE)
+ *
+ * Else map each user to a core port and set all devices to RETRO_DEVICE_JOYPAD:
+ *
+ * user 0 -> core port 0 (RETRO_DEVICE_JOYPAD)
+ * user 1 -> core port 1 (RETRO_DEVICE_JOYPAD)
+ * ...
+ * user n -> core port n (RETRO_DEVICE_JOYPAD)
+ *
+ * (where n = MAX_USERS - 1)
+ */
+void input_remapping_set_remap_ports_defaults(void);
+
 /**
  * Checks `input_config_bind_map` for the requested `input_bind_map`, and if
  * the bind has been registered, returns its base.

intl/msg_hash_lbl.h

@@ -2170,6 +2170,10 @@ MSG_HASH(
    MENU_ENUM_LABEL_INPUT_REMAP_BINDS_ENABLE,
    "input_remap_binds_enable"
    )
+MSG_HASH(
+   MENU_ENUM_LABEL_INPUT_REMAP_PORTS_ON_BUTTON_PRESS,
+   "input_remap_ports_on_button_press"
+   )
 MSG_HASH(
    MENU_ENUM_LABEL_INPUT_REMAP_SORT_BY_CONTROLLER_ENABLE,
    "input_remap_sort_by_controller_enable"
@@ -6266,6 +6270,10 @@ MSG_HASH(
    MENU_ENUM_LABEL_NOTIFICATION_SHOW_REMAP_LOAD,
    "notification_show_remap_load"
    )
+MSG_HASH(
+   MENU_ENUM_LABEL_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT,
+   "notification_show_user_mapped_to_core_port"
+   )
 MSG_HASH(
    MENU_ENUM_LABEL_NOTIFICATION_SHOW_CONFIG_OVERRIDE_LOAD,
    "notification_show_config_override_load"

intl/msg_hash_us.h

@@ -3337,6 +3337,14 @@ MSG_HASH(
    MENU_ENUM_SUBLABEL_INPUT_REMAP_BINDS_ENABLE,
    "Override the input binds with the remapped binds set for the current core."
    )
+MSG_HASH(
+   MENU_ENUM_LABEL_VALUE_INPUT_REMAP_PORTS_ON_BUTTON_PRESS,
+   "Remap Ports on Button Press"
+   )
+MSG_HASH(
+   MENU_ENUM_SUBLABEL_INPUT_REMAP_PORTS_ON_BUTTON_PRESS,
+   "First user to press a button becomes player 1, second becomes player 2, etc."
+   )
 MSG_HASH(
    MENU_ENUM_LABEL_VALUE_INPUT_REMAP_SORT_BY_CONTROLLER_ENABLE,
    "Sort Remaps By Gamepad"
@@ -5993,6 +6001,14 @@ MSG_HASH(
    MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_REMAP_LOAD,
    "Display an on-screen message when loading input remap files."
    )
+MSG_HASH(
+   MENU_ENUM_LABEL_VALUE_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT,
+   "Remap Ports on Button Press Notifications"
+   )
+MSG_HASH(
+   MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT,
+   "Display an on-screen message when a user is mapped to a core port."
+   )
 MSG_HASH(
    MENU_ENUM_LABEL_VALUE_NOTIFICATION_SHOW_CONFIG_OVERRIDE_LOAD,
    "Config Override Loaded Notifications"
@@ -15542,6 +15558,10 @@ MSG_HASH(
    MSG_DEVICE_NOT_CONFIGURED_FALLBACK_NR,
    "%s (%u/%u) not configured, using fallback"
    )
+MSG_HASH(
+   MSG_USER_MAPPED_TO_CORE_PORT_NR,
+   "%s is Player %u"
+   )
 MSG_HASH(
    MSG_BLUETOOTH_SCAN_COMPLETE,
    "Bluetooth scan complete."

menu/cbs/menu_cbs_sublabel.c

@@ -687,6 +687,7 @@ DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_cheats_applied,  M
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_patch_applied,   MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_PATCH_APPLIED)
 #endif
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_remap_load,  MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_REMAP_LOAD)
+DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_user_mapped_to_core_port, MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_config_override_load, MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_CONFIG_OVERRIDE_LOAD)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_set_initial_disk, MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_SET_INITIAL_DISK)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_notification_show_disk_control, MENU_ENUM_SUBLABEL_NOTIFICATION_SHOW_DISK_CONTROL)
@@ -823,6 +824,7 @@ DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_replay_max_keep,               MENU_
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_replay_checkpoint_interval,    MENU_ENUM_SUBLABEL_REPLAY_CHECKPOINT_INTERVAL)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_replay_checkpoint_deserialize, MENU_ENUM_SUBLABEL_REPLAY_CHECKPOINT_DESERIALIZE)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_input_remap_binds_enable,      MENU_ENUM_SUBLABEL_INPUT_REMAP_BINDS_ENABLE)
+DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_input_remap_ports_on_button_press, MENU_ENUM_SUBLABEL_INPUT_REMAP_PORTS_ON_BUTTON_PRESS)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_input_remap_sort_by_controller_enable,      MENU_ENUM_SUBLABEL_INPUT_REMAP_SORT_BY_CONTROLLER_ENABLE)
 DEFAULT_SUBLABEL_MACRO(action_bind_sublabel_input_autodetect_enable,       MENU_ENUM_SUBLABEL_INPUT_AUTODETECT_ENABLE)
 #if defined(HAVE_DINPUT) || defined(HAVE_WINRAWINPUT)
@@ -4146,6 +4148,9 @@ int menu_cbs_init_bind_sublabel(menu_file_list_cbs_t *cbs,
          case MENU_ENUM_LABEL_INPUT_REMAP_BINDS_ENABLE:
             BIND_ACTION_SUBLABEL(cbs, action_bind_sublabel_input_remap_binds_enable);
             break;
+         case MENU_ENUM_LABEL_INPUT_REMAP_PORTS_ON_BUTTON_PRESS:
+            BIND_ACTION_SUBLABEL(cbs, action_bind_sublabel_input_remap_ports_on_button_press);
+            break;
          case MENU_ENUM_LABEL_INPUT_REMAP_SORT_BY_CONTROLLER_ENABLE:
             BIND_ACTION_SUBLABEL(cbs, action_bind_sublabel_input_remap_sort_by_controller_enable);
             break;
@@ -4534,6 +4539,9 @@ int menu_cbs_init_bind_sublabel(menu_file_list_cbs_t *cbs,
          case MENU_ENUM_LABEL_NOTIFICATION_SHOW_REMAP_LOAD:
             BIND_ACTION_SUBLABEL(cbs, action_bind_sublabel_notification_show_remap_load);
             break;
+         case MENU_ENUM_LABEL_NOTIFICATION_SHOW_USER_MAPPED_TO_CORE_PORT:
+            BIND_ACTION_SUBLABEL(cbs, action_bind_sublabel_notification_show_user_mapped_to_core_port);
+            break;
          case MENU_ENUM_LABEL_NOTIFICATION_SHOW_CONFIG_OVERRIDE_LOAD:
             BIND_ACTION_SUBLABEL(cbs, action_bind_sublabel_notification_show_config_override_load);
             break;