[RESUBMIT] [PATCH] [BUGFIX] sound/oss/es1371.c: Don't print joystickaddress before it's set.

[Bodo Eggert]

Resubmit because of no feedback nor inclusion in the latest changelogs.
I'm not sure wether this patch qualifies for the patch monkey, so I still
omit it.

Changed to apply with -p1 instead of -p0 after reading a unrelated hint on 
LKML (maybe this should be mentioned in the SubmittingPatches?) and added 
more explanation why I feel this patch is needed.



This patch was created against Version: 2.6.10-ac9

The old code printed the joystick address before it was set, possibly 
before the field was initialized.

This caused me to search for the reason why the f...ine joystic port
didn't work (it did, but it was hidden) instead of simply loading the
module for the attached device.

Old output was: (from memory)
es1371: found es1371 rev 2 at io 0xec00 irq 5 joystick 0x0

New output is:
es1371: found es1371 rev 2 at io 0xec00 irq 5
es1371: es1371 joystick at 0x218

Signed-off-by: Bodo Eggert <7eggert@xxxxxx>

--- sound/oss/es1371.c.ori	2005-01-22 17:38:10.000000000 +0100
+++ sound/oss/es1371.c	2005-02-13 16:41:19.196704432 +0100
@@ -105,6 +105,8 @@
  *                       Fix SETTRIGGER non OSS API conformity
  *    14.07.2001   0.31  Add list of laptops needing amplifier control
  *    03.01.2003   0.32  open_mode fixes from Georg Acher <acher@xxxxxxxxx>
+ *    22.01.2004   0.33  fix output of joystick address
+ *                       by Bodo Eggert <7eggert@xxxxxx>
  */
 
 /*****************************************************************************/
@@ -2849,8 +2851,8 @@ static int __devinit es1371_probe(struct
 		printk(KERN_ERR PFX "irq %u in use\n", s->irq);
 		goto err_irq;
 	}
-	printk(KERN_INFO PFX "found es1371 rev %d at io %#lx irq %u joystick %#x\n",
-	       s->rev, s->io, s->irq, s->gameport.io);
+	printk(KERN_INFO PFX "found es1371 rev %d at io %#lx irq %u\n",
+	       s->rev, s->io, s->irq);
 	/* register devices */
 	if ((res=(s->dev_audio = register_sound_dsp(&es1371_audio_fops,-1)))<0)
 		goto err_dev1;
@@ -2886,6 +2888,8 @@ static int __devinit es1371_probe(struct
 		if (request_region(i, JOY_EXTENT, "es1371")) {
 			s->ctrl |= CTRL_JYSTK_EN | (((i >> 3) & CTRL_JOY_MASK) << CTRL_JOY_SHIFT);
 			s->gameport.io = i;
+			printk(KERN_INFO PFX "es1371 joystick at %#x\n",
+				s->gameport.io);
 			break;
 		}
 	}
-- 
A bone to the dog is not charity. Charity is the bone shared with the dog, when you are
just as hungry as the dog.	-- Jack London
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/