2004-10-31 11:44:28

by Michele Debandi

[permalink] [raw]
Subject: [PATCH] cmpci.c fixes for joystick initialization in 2.4.27

Hello,

I hope you are the current mantainer of cmpci module on 2.4 series kernel.

I have an integrated CM8738 sound chip on my Asus P4B533 motherboard.
The lspci -v output is:

02:03.0 Multimedia audio controller: C-Media Electronics Inc CM8738 (rev 10)
Subsystem: Asustek Computer, Inc.: Unknown device 80e2
Flags: bus master, stepping, medium devsel, latency 32, IRQ 21
I/O ports at b800 [size=256]
Capabilities: [c0] Power Management version 2

With the cmpci.c driver the joystick will not work. MSDOS initialization
sets the joystick port at the address 0x201, and windows driver uses also
this port. The cmpci driver initializes instead the port 0x200, and
on my chipset at that address thre is nothing. So I modified the driver
modules to use the port 0x201 but mantaining the 8-port allocation of the
original driver.
This is tested and seems to work on a stantard PC/XT style 2-axis/2-button
joystick.

Below there is the diff file.

Greetings

Mike

--- drivers/sound/cmpci.c.ORIG Tue Oct 26 20:55:08 2004
+++ drivers/sound/cmpci.c Tue Oct 26 21:01:23 2004
@@ -3354,7 +3354,7 @@
#endif
s->iosynth = fmio;
s->iomidi = mpuio;
- s->gameport.io = 0x200;
+ s->gameport.io = 0x201; /*use standard DOS io port */
s->status = 0;
/* range check */
if (speakers < 2)
@@ -3443,7 +3443,8 @@
#endif
/* enable joystick */
if (joystick) {
- if (s->gameport.io && !request_region(s->gameport.io, CM_EXTENT_GAME, "cmpci GAME")) {
+ /* need to use port 0x201, but the extent starts at 0x200??? */
+ if (s->gameport.io && !request_region((s->gameport.io) - 1, CM_EXTENT_GAME, "cmpci GAME")) {
printk(KERN_ERR "cmpci: gameport io ports in use\n");
s->gameport.io = 0;
} else
@@ -3549,8 +3550,13 @@
s->max_channels = 2;
}
/* register gameport */
- if (joystick)
+ if (joystick) {
gameport_register_port(&s->gameport);
+ /* better write some more info */
+ printk(KERN_INFO "gameport%d: CMPCI at %#x", s->gameport.number, s->gameport.io);
+ printk(" size %d", CM_EXTENT_GAME);
+ printk(" speed %d kHz\n", s->gameport.speed);
+ }
/* store it in the driver field */
pci_set_drvdata(pcidev, s);
/* put it into driver list */
@@ -3576,7 +3582,7 @@
free_irq(s->irq, s);
err_irq:
if (s->gameport.io)
- release_region(s->gameport.io, CM_EXTENT_GAME);
+ release_region((s->gameport.io)-1, CM_EXTENT_GAME);
#ifdef CONFIG_SOUND_CMPCI_FM
if (s->iosynth) release_region(s->iosynth, CM_EXTENT_SYNTH);
#endif
@@ -3612,7 +3618,7 @@

if (s->gameport.io) {
gameport_unregister_port(&s->gameport);
- release_region(s->gameport.io, CM_EXTENT_GAME);
+ release_region((s->gameport.io)-1, CM_EXTENT_GAME);
}
release_region(s->iobase, CM_EXTENT_CODEC);
#ifdef CONFIG_SOUND_CMPCI_MIDI




2004-11-27 02:27:14

by Marcelo Tosatti

[permalink] [raw]
Subject: Re: [PATCH] cmpci.c fixes for joystick initialization in 2.4.27



ChenLi,

Have you received this patch?

Can you confirm 0x201 is the standard address for the port?

On Sun, Oct 31, 2004 at 12:01:53PM +0100, Michele Debandi wrote:
> Hello,
>
> I hope you are the current mantainer of cmpci module on 2.4 series kernel.
>
> I have an integrated CM8738 sound chip on my Asus P4B533 motherboard.
> The lspci -v output is:
>
> 02:03.0 Multimedia audio controller: C-Media Electronics Inc CM8738 (rev 10)
> Subsystem: Asustek Computer, Inc.: Unknown device 80e2
> Flags: bus master, stepping, medium devsel, latency 32, IRQ 21
> I/O ports at b800 [size=256]
> Capabilities: [c0] Power Management version 2
>
> With the cmpci.c driver the joystick will not work. MSDOS initialization
> sets the joystick port at the address 0x201, and windows driver uses also
> this port. The cmpci driver initializes instead the port 0x200, and
> on my chipset at that address thre is nothing. So I modified the driver
> modules to use the port 0x201 but mantaining the 8-port allocation of the
> original driver.
> This is tested and seems to work on a stantard PC/XT style 2-axis/2-button
> joystick.
>
> Below there is the diff file.
>
> Greetings
>
> Mike
>
> --- drivers/sound/cmpci.c.ORIG Tue Oct 26 20:55:08 2004
> +++ drivers/sound/cmpci.c Tue Oct 26 21:01:23 2004
> @@ -3354,7 +3354,7 @@
> #endif
> s->iosynth = fmio;
> s->iomidi = mpuio;
> - s->gameport.io = 0x200;
> + s->gameport.io = 0x201; /*use standard DOS io port */
> s->status = 0;
> /* range check */
> if (speakers < 2)
> @@ -3443,7 +3443,8 @@
> #endif
> /* enable joystick */
> if (joystick) {
> - if (s->gameport.io && !request_region(s->gameport.io, CM_EXTENT_GAME, "cmpci GAME")) {
> + /* need to use port 0x201, but the extent starts at 0x200??? */
> + if (s->gameport.io && !request_region((s->gameport.io) - 1, CM_EXTENT_GAME, "cmpci GAME")) {
> printk(KERN_ERR "cmpci: gameport io ports in use\n");
> s->gameport.io = 0;
> } else
> @@ -3549,8 +3550,13 @@
> s->max_channels = 2;
> }
> /* register gameport */
> - if (joystick)
> + if (joystick) {
> gameport_register_port(&s->gameport);
> + /* better write some more info */
> + printk(KERN_INFO "gameport%d: CMPCI at %#x", s->gameport.number, s->gameport.io);
> + printk(" size %d", CM_EXTENT_GAME);
> + printk(" speed %d kHz\n", s->gameport.speed);
> + }
> /* store it in the driver field */
> pci_set_drvdata(pcidev, s);
> /* put it into driver list */
> @@ -3576,7 +3582,7 @@
> free_irq(s->irq, s);
> err_irq:
> if (s->gameport.io)
> - release_region(s->gameport.io, CM_EXTENT_GAME);
> + release_region((s->gameport.io)-1, CM_EXTENT_GAME);
> #ifdef CONFIG_SOUND_CMPCI_FM
> if (s->iosynth) release_region(s->iosynth, CM_EXTENT_SYNTH);
> #endif
> @@ -3612,7 +3618,7 @@
>
> if (s->gameport.io) {
> gameport_unregister_port(&s->gameport);
> - release_region(s->gameport.io, CM_EXTENT_GAME);
> + release_region((s->gameport.io)-1, CM_EXTENT_GAME);
> }
> release_region(s->iobase, CM_EXTENT_CODEC);
> #ifdef CONFIG_SOUND_CMPCI_MIDI