lkml.org 
[lkml]   [2004]   [Dec]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] Hotplug support for several PSX controlers
Hello,

Since 2.6.9, several PSX controlers in the same time can be supported.
However, because of a bug, if not all the PSX controlers are pluged in
then nothing works. Typically, you load gamecon with options saying that
you have two PSX adapter ports and then you plug and unplug has many
controllers has you want. There is a bug which prevent keypress to be
detected when not all the controllers connected.

The problem was that when a port didn't have a controler pluged the
packet length to receive was read as very big, leading to a kind of
buffer overflow. This patch checks the packet length and if it is bigger
than the theoritical possible it considers that there is no controller
pluged on this port.

It probably works on a vanilla 2.6.10-rc3 but I highly recommand to use
the Vojtech's tree which contains an important fix about PSX DDR (cf
http://marc.theaimsgroup.com/?l=linux-kernel&m=110118014804716&w=2).

I've heard that Linus wants 2.6.10 ready for Christmas, this patch
should definitetly helps ;-)

Eric
--- linux-2.6.10.orig/drivers/input/joystick/gamecon.c 2004-10-20 21:25:06.000000000 +0200
+++ linux-2.6.10-rc3/drivers/input/joystick/gamecon.c 2004-12-12 14:58:35.000000000 +0100
@@ -241,7 +241,7 @@ static void gc_multi_read_packet(struct
#define GC_PSX_SELECT 0x02 /* Pin 3 */

#define GC_PSX_ID(x) ((x) >> 4) /* High nibble is device type */
-#define GC_PSX_LEN(x) ((x) & 0xf) /* Low nibble is length in words */
+#define GC_PSX_LEN(x) (((x) & 0xf) << 1) /* Low nibble is length in words */

static int gc_psx_delay = GC_PSX_DELAY;
module_param_named(psx_delay, gc_psx_delay, uint, 0);
@@ -259,7 +259,7 @@ static short gc_psx_ddr_btn[] = { BTN_0,
* the psx pad.
*/

-static void gc_psx_command(struct gc *gc, int b, unsigned char data[GC_PSX_LENGTH])
+static void gc_psx_command(struct gc *gc, int b, unsigned char data[5])
{
int i, j, cmd, read;
for (i = 0; i < 5; i++)
@@ -302,10 +302,12 @@ static void gc_psx_read_packet(struct gc

for (i =0; i < 5; i++) /* Find the longest pad */
if((gc_status_bit[i] & (gc->pads[GC_PSX] | gc->pads[GC_DDR])) &&
- (GC_PSX_LEN(id[i]) > max_len))
+ (GC_PSX_LEN(id[i]) > max_len) &&
+ /* Too long length is a sign that no joystick is present */
+ (GC_PSX_LEN(id[i]) <= GC_PSX_LENGTH))
max_len = GC_PSX_LEN(id[i]);

- for (i = 0; i < max_len * 2; i++) { /* Read in all the data */
+ for (i = 0; i < max_len; i++) { /* Read in all the data */
gc_psx_command(gc, 0, data2);
for (j = 0; j < 5; j++)
data[j][i] = data2[j];
\
 
 \ /
  Last update: 2005-03-22 14:08    [W:0.044 / U:0.496 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site