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

Add Trooper 2 #838

Merged
merged 1 commit into from Jun 1, 2022
Merged

Add Trooper 2 #838

merged 1 commit into from Jun 1, 2022

Conversation

RobLoach
Copy link
Member

@RobLoach RobLoach commented May 8, 2022

Fixes #828

input_r_x_plus_btn = "1"
input_r_x_minus_btn = "1"
input_r_y_plus_btn = "1"
input_r_y_minus_btn = "1"
Copy link
Member Author

Choose a reason for hiding this comment

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

Missing labels.

Copy link
Member Author

Choose a reason for hiding this comment

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

Going to merge for now, we could fix later on.

Copy link

Choose a reason for hiding this comment

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

This file looks wrong for several reasons:

  1. why so many inputs (16 total) mapped to "1"?
  2. the physical joystick has NO analog inputs.
  3. device name is padded with spaces and ill-formed.
  4. duplicate vendor and product id (VID & PID) with "Hyperkin_Trooper_V2.cfg"
  5. no labels

Conclusion: this file should be deleted.

@RobLoach RobLoach merged commit 2c36ecb into master Jun 1, 2022
@RobLoach RobLoach deleted the trooper-2--the-empire-strikes-back branch January 26, 2024 08:34
@dcaputi1
Copy link

dcaputi1 commented Apr 7, 2024

The file added in this commit looks corrupt. There are 16 in total digital and analog inputs all assigned to "1". The VID and PID are identical to a similarly named file "Hyperkin Trooper V2". This was discussed here https://github.com/libretro/retroarch-joypad-autoconfig/issues/828 but for some reason was missed in the merge.

@RobLoach
Copy link
Member Author

RobLoach commented Apr 7, 2024

Good catch, @dcaputi1. Let's revert this one.

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.

Trooper 2 Lakka Support
2 participants