[Patch] WiseGroup MP-8800 / MP-8866 (PS2 Joystick)

In the current cvs version, SDL doesn’t handle these Playstation2
controller => USB adapters correctly, in linux.

It will always assume that the maximum number of joysticks (2 in the
case of the MP-8866, 4 in the case of the 8800) are plugged in. This is
bad not only because it allows SDL to exaggerate the number of logical
joysticks, but primarily because the joystick axes are mapped
incorrectly, all over the place, such that the devices are effectively
unusable unless you have the maximum number of joysticks plugged in.

My changes to src/joystick/linux/SDL_sysjoystick.c build on another’s
previous work (which was a special case for this very joystick,
actually), and fix both of these problems, as well as making the current
code a little more general, to allow for others to more easily drop in
code for quirky joysticks such as these.

I’ve tested this code under 2.6.7 as well as 2.4.24… Both work as
advertised (provided you load the JOYDEV linux code as a module,
otherwise they won’t work at all, new code or old, but that’s another
issue entirely).

Though this sounds horribly formal, you have my permission to distribute
all of my work on this issue under the LGPL. So there.

Please let me know if you merge my changes into the cvs tree, or what
I’d need to change to get these fixes in (I attempted to mimic the style
as best as I could).

http://cs-people.bu.edu/chrisn1/SDL_sysjoystick.c

-Chris
-------------- next part --------------
A non-text attachment was scrubbed…
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: http://lists.libsdl.org/pipermail/sdl-libsdl.org/attachments/20040623/0ff98927/attachment.pgp

Chris Nelson wrote:

In the current cvs version, SDL doesn’t handle these Playstation2
controller => USB adapters correctly, in linux.

Thanks for the patch I hope that this patch makes it into SDL’s upcoming
release.

Thanks! I’ve placed your patch in CVS. Please let me know if there
are any bugfixes or problems with it.

See ya,
-Sam Lantinga, Software Engineer, Blizzard Entertainment