Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create android/Pro Controller.cfg #1066

Closed
wants to merge 5 commits into from

Conversation

davidhedlund
Copy link
Contributor

@davidhedlund davidhedlund commented Dec 11, 2023

# Nintendo Switch Pro Controller (with nintendo-hid)
# 
# This is the first fully working Android autoconfig
# for Nintendo Switch Pro Controller as far as I know
# It requires Android 12.
# 
# Disclaimer:
# The Nintendo Switch Pro Controller has other binding
# for earliers Android versions. However, the Bluetooth
# connectivity issue in Android 11 and earlier makes the
# controller useless for gaming.
# 
# Evaluated by David Hedlund.

Generated from

  • OS: Google TV 12 (it's based on Android 12, se below)
  • RA installed via: Google Play
  • Retroarch version: 1.9.12

RetroArch:

  • Main Menu -> Information -> System Information -> Frontend Identifier: android [default]
  • Main Menu -> Information -> System Information -> Frontend Name: Chromecast HD
  • Main Menu -> Information -> System Information -> Frontend OS: Android (v12.0)
  • Settings -> Drivers -> Controller -> Controller: android [default]

IDs

The file is a modified copy from udev/Create Nintendo - Switch Pro Controller.cfg, including:

# Hex vid/pid: 057e:2009
input_vendor_id = "1406"
input_product_id = "8201"

However, currently, when I'm connecting the Nintendo Switch Pro Controller, RA pops up a "8BitDo SN30 Pro+ (Switch Mode) configured on port #1" (android/8BitDo_SN30_Pro_for_Android.cfg), and sometimes "Nintendo Switch Pro Controller configured on port #1" (sdl2/Nintendo Switch Pro Controller.cfg). So I'm a bit confused which ID that should be used.

# Nintendo Switch Pro Controller (with nintendo-hid)
# 
# This is the first fully working Android autoconfig
# for Nintendo Switch Pro Controller as far as I know
# It requires Android 12.
# 
# Disclaimer:
# The Nintendo Switch Pro Controller has other binding
# for earliers Android versions. However, the Bluetooth
# connectivity issue in Android 11 and earlier makes the
# controller useless for gaming.
# 
# Evaluated by David Hedlund.
# Requires Android 12 or later.
@davidhedlund davidhedlund marked this pull request as draft December 11, 2023 05:02
@LibretroAdmin
Copy link
Contributor

HI, what's the state of this? When is this PR ready for merge?

@davidhedlund
Copy link
Contributor Author

HI, what's the state of this? When is this PR ready for merge?

It's ready.

@davidhedlund davidhedlund marked this pull request as ready for review January 13, 2024 14:29
@hizzlekizzle
Copy link
Contributor

seems fine to me

@davidhedlund davidhedlund changed the title Create Nintendo_Switch_Pro_Controller.cfg Create Pro Controller.cfg Jan 15, 2024
@davidhedlund davidhedlund changed the title Create Pro Controller.cfg Create android/Pro Controller.cfg Jan 17, 2024
@davidhedlund
Copy link
Contributor Author

seems fine to me

From https://github.com/libretro/retroarch-joypad-autoconfig/pull/1066/checks

image

Some duplicate profiles were found
Duplicates inside 'android'
Vendor ID: 1406 - Product ID: 8201 - Name: Pro Controller
- android/8BitDo SN30 Pro+ (Switch Mode).cfg
- android/Pro Controller.cfg

How do we solve that?

@RobLoach
Copy link
Member

Duplicate vendor/product id pairs cause conflicts with RA. While it still "works", it can confuse the controller types. I'd honestly recommend just deleting android/8BitDo SN30 Pro (Switch Mode).cfg with this as its replacement.

People can use the 8BitDo controller in a different mode just fine.

Copy link
Member

@RobLoach RobLoach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete android/8BitDo SN30 Pro (Switch Mode).cfg .

@davidhedlund
Copy link
Contributor Author

davidhedlund commented Jan 25, 2024

Delete android/8BitDo SN30 Pro (Switch Mode).cfg .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants