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

improve cellPad [fixes Input regression] #3989

Merged
merged 3 commits into from Jan 13, 2018
Merged

Conversation

Megamouse
Copy link
Contributor

@Megamouse Megamouse commented Dec 26, 2017

fixes Super Street Fighter Turbo 2 HD
similar to prior Catherine fix

also fixes a bug where devices become useless when a "prior" device is disconnected during multiplayer leading to lowered now_connect count

@Megamouse Megamouse changed the title improve cellPadSetPressMode and cellPadSetSensorMode improve cellPad Dec 27, 2017
@Kravickas
Copy link
Contributor

FIFA 18 - press start works now
unknown

@Luffykun007
Copy link

with this PR FIFA World Cup Brazil 2014 going into the game now.

capturar

capturar2

@spyropt
Copy link

spyropt commented Dec 27, 2017

Super Street Fighter Turbo 2 HD Tested and working again :) thanks

@bobthecooldad
Copy link

bobthecooldad commented Dec 29, 2017

This fixes the simpsons game stuck at controller reconnect, allows to go ingame
Still has the issue that you cannot go up or down in the menu

@Asinin3
Copy link
Contributor

Asinin3 commented Dec 29, 2017

Simpsons can go ingame on master just need to unplug controller then plug it back in.

Fifa 17 can go ingame with this PR:
image

Nascar The Game: Inside Line still doesn't respond to inputs.

@SakataGintokiYT
Copy link
Contributor

SakataGintokiYT commented Jan 1, 2018

Ingame with this PR
bez tytulu

@Megamouse Megamouse changed the title improve cellPad improve cellPad [fixes Input regression] Jan 1, 2018
@Megamouse Megamouse force-pushed the cellpad branch 3 times, most recently from 9e07ef8 to f7f362f Compare January 2, 2018 11:57
@tge-was-taken
Copy link
Contributor

tge-was-taken commented Jan 3, 2018

BLUS30023 now goes ingame with this PR

@toccata10
Copy link
Contributor

toccata10 commented Jan 3, 2018

It fixes keyboard input on super street fighter 2, but if I change to an evdev joystick: no input at all. No input neither on my ds4 controller. The config gui shows that these controllers are detected and working. I can create a config file. It's just in the games that they're dead.
The logfile:
RPCS3.log-cellpad.gz

@Megamouse
Copy link
Contributor Author

@toccata10 try again

@toccata10
Copy link
Contributor

Can't build on my system

[ 78%] Building CXX object rpcs3/CMakeFiles/rpcs3.dir/evdev_joystick_handler.cpp.o
cc1plus: warning: /home/jo/tmp/rpcs3/megamouse/rpcs3_build/rpcs3/cotire/rpcs3_CXX_prefix.hxx.gch : non utilisé parce que « __STDC_CONSTANT_MACROS » est défini [-Winvalid-pch]
/home/jo/tmp/rpcs3/megamouse/rpcs3/rpcs3/evdev_joystick_handler.cpp: Dans la fonction membre « evdev_joystick_handler::EvdevDevice* evdev_joystick_handler::get_device(const string&) »:
/home/jo/tmp/rpcs3/megamouse/rpcs3/rpcs3/evdev_joystick_handler.cpp:237:31: error: pas de fonction concordante pour l'appel à « evdev_joystick_handler::update_device(evdev_joystick_handler::EvdevDevice&, bool) »
  if (!update_device(dev, false))
                               ^
/home/jo/tmp/rpcs3/megamouse/rpcs3/rpcs3/evdev_joystick_handler.cpp:113:6: note: candidate: bool evdev_joystick_handler::update_device(evdev_joystick_handler::EvdevDevice&)
 bool evdev_joystick_handler::update_device(EvdevDevice& device)
      ^~~~~~~~~~~~~~~~~~~~~~
/home/jo/tmp/rpcs3/megamouse/rpcs3/rpcs3/evdev_joystick_handler.cpp:113:6: note:   le candidat attend 1 argument, 2 fourni(s)
make[2]: *** [rpcs3/CMakeFiles/rpcs3.dir/build.make:7727: rpcs3/CMakeFiles/rpcs3.dir/evdev_joystick_handler.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:240: rpcs3/CMakeFiles/rpcs3.dir/all] Error 2
make: *** [Makefile:152: all] Error 2

@Megamouse
Copy link
Contributor Author

@toccata10 sorry. I forgot a line. I hope it works now

@toccata10
Copy link
Contributor

It works fine now. I tested with evdev, ds4 and keyboard. So the linux part seems ok. Probably just waiting for the merge now.

@Megamouse
Copy link
Contributor Author

as far as I can tell everything was tested now.
good thing this wasn't merged before (well actually i could've fixed all issues faster then hehe)

@LuismaSP89
Copy link

@Megamouse Syberia 2 still has no input even with this PR :S

@Megamouse
Copy link
Contributor Author

I know. but i can't find the problem. And I don't think I caused it

@toccata10
Copy link
Contributor

Syberia 2 [NPEB02016] has not input problems for me (just tested the beginning) on master. Tested with keyboard and ds4 on windows+linux.

@LuismaSP89
Copy link

@toccata10 Strange, i can´t pass the "Press start" screen, and i´m not the only one with this problem as you can see in the issue that i opened:

#3985

Mine is NPEB02016 too, and i tested with keyboard, xbox 360 and xbox one controllers.

@toccata10
Copy link
Contributor

I hope this gets merged soon as it improves master for several games. The syberia 2 pb is already in master, so there's no regression here, just improvement (from what I can see).

@LuismaSP89
Copy link

@toccata10 Syberia problem is solved (#3985)

inlcudes simpsons "hack" as comment
fix input regression and fix input for FIFA games
fix input in NASCAR [BLUS30932]
fix port status query -> disconnected devices don't cripple following devices by decreased now_connect
@Nekotekina Nekotekina merged commit 155f7a7 into RPCS3:master Jan 13, 2018
@Megamouse Megamouse deleted the cellpad branch January 14, 2018 21:08
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.

None yet