Patchwork Fix issue with Winbond W83627EHG MIDI driver

login
register
about
Submitter Alexandru Gagniuc
Date 2011-02-05 19:54:49
Message ID <4D4DAB09.3080606@gmail.com>
Download mbox | patch
Permalink /patch/2614/
State Superseded
Headers show

Comments

Alexandru Gagniuc - 2011-02-05 19:54:49
Sorry about forgetting to attach the patch.

Alex

On 02/05/2011 09:50 PM, Alex G. wrote:
> (See Patch)
> 
> This fixes the
> "ERROR: device PNP: 002e.207 index 98 has no mask."
> that I have been getting on the W83627EHG.
> 
> Alex
> 
>
Jonathan A. Kollasch - 2011-02-08 01:41:43
On Sat, Feb 05, 2011 at 09:54:49PM +0200, Alex G. wrote:
> Sorry about forgetting to attach the patch.
> 
> Alex
> 
> On 02/05/2011 09:50 PM, Alex G. wrote:
> > (See Patch)
> > 
> > This fixes the
> > "ERROR: device PNP: 002e.207 index 98 has no mask."
> > that I have been getting on the W83627EHG.
> > 
> > Alex
> > 
> > 
> 

> Fixes a problem with the W8327EHG driver, where the midi device
> could get initialised with no mask.
> 
> Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
> 
> Index: src/superio/winbond/w83627ehg/superio.c
> ===================================================================
> --- src/superio/winbond/w83627ehg/superio.c	(revision 6323)
> +++ src/superio/winbond/w83627ehg/superio.c	(working copy)
> @@ -189,7 +189,7 @@
>  	{ &ops, W83627EHG_HWM,  PNP_IO0 | PNP_IRQ0, {0x07fe, 0}, },
>  
>  	{ &ops, W83627EHG_GAME, PNP_IO0, {0x07ff, 0}, },
> -	{ &ops, W83627EHG_MIDI, PNP_IO1 | PNP_IRQ0, {0x07fe, 4}, },
> +	{ &ops, W83627EHG_MIDI, PNP_IO0 | PNP_IRQ0, {0x07fe, 4}, },
>  	{ &ops, W83627EHG_GPIO1, },
>  	{ &ops, W83627EHG_GPIO2, },
>  	{ &ops, W83627EHG_GPIO3, },

I don't think this is the right fix.
The datasheet says the MIDI BAR is at 62h, 63h.
Hence the mask needs to be in the second position, not the first.

Maybe try something like: 
{ &ops, W83627EHG_MIDI, PNP_IO1 | PNP_IRQ0, {0, 0}, {0x07fe, 4}, }

	Jonathan Kollasch

Patch

Fixes a problem with the W8327EHG driver, where the midi device
could get initialised with no mask.

Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>

Index: src/superio/winbond/w83627ehg/superio.c
===================================================================
--- src/superio/winbond/w83627ehg/superio.c	(revision 6323)
+++ src/superio/winbond/w83627ehg/superio.c	(working copy)
@@ -189,7 +189,7 @@ 
 	{ &ops, W83627EHG_HWM,  PNP_IO0 | PNP_IRQ0, {0x07fe, 0}, },
 
 	{ &ops, W83627EHG_GAME, PNP_IO0, {0x07ff, 0}, },
-	{ &ops, W83627EHG_MIDI, PNP_IO1 | PNP_IRQ0, {0x07fe, 4}, },
+	{ &ops, W83627EHG_MIDI, PNP_IO0 | PNP_IRQ0, {0x07fe, 4}, },
 	{ &ops, W83627EHG_GPIO1, },
 	{ &ops, W83627EHG_GPIO2, },
 	{ &ops, W83627EHG_GPIO3, },