public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* PATCH: Dell laptop lockup fix for ALSA
@ 2004-06-21 12:16 Alan Cox
  2004-06-21 14:24 ` Takashi Iwai
  0 siblings, 1 reply; 2+ messages in thread
From: Alan Cox @ 2004-06-21 12:16 UTC (permalink / raw)
  To: perex, akpm, linux-kernel; +Cc: trivial

OSS avoids the Dell lockup by not hitting the problem register (which 
apparently breaks resume on a Sony laptop). ALSA keeps a flag and uses
pci subvendor info to clear it for problem Dell laptops. Unfortunately
there is at least one other Dell laptop which is affected. This adds
its sub id's

[Patch from Dan Williams @ Red Hat slightly reformatted by me]


--- sound/pci/nm256/nm256.c	2004-06-14 14:11:34.000000000 +0100
+++ /tmp/nm256.c	2004-06-21 12:08:49.099484448 +0100
@@ -1509,6 +1509,10 @@
 		/* this workaround will cause lock-up after suspend/resume on Sony PCG-F305 */
 		chip->latitude_workaround = 0;
 	}
+	if (subsystem_vendor == 0x1028 && subsystem_device == 0x0080) {
+		/* this workaround will cause lock-up after suspend/resume on Sony PCG-F305 */
+		chip->latitude_workaround = 0;
+	}
 
 	snd_nm256_init_chip(chip);
 



      Developer's Certificate of Origin 1.0
 
        By making a contribution to this project, I certify that:
 
        (a) The contribution was created in whole or in part by me and I
            have the right to submit it under the open source license
            indicated in the file; or
 
        (b) The contribution is based upon previous work that, to the best
            of my knowledge, is covered under an appropriate open source
            license and I have the right under that license to submit that
            work with modifications, whether created in whole or in part
            by me, under the same open source license (unless I am
            permitted to submit under a different license), as indicated
            in the file; or
 
        (c) The contribution was provided directly to me by some other
            person who certified (a), (b) or (c) and I have not modified
            it.


Signed-off-by: Alan Cox <alan@redhat.com>


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: PATCH: Dell laptop lockup fix for ALSA
  2004-06-21 12:16 PATCH: Dell laptop lockup fix for ALSA Alan Cox
@ 2004-06-21 14:24 ` Takashi Iwai
  0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2004-06-21 14:24 UTC (permalink / raw)
  To: Alan Cox; +Cc: perex, akpm, linux-kernel, trivial

At Mon, 21 Jun 2004 08:16:47 -0400,
Alan Cox wrote:
> 
> OSS avoids the Dell lockup by not hitting the problem register (which 
> apparently breaks resume on a Sony laptop). ALSA keeps a flag and uses
> pci subvendor info to clear it for problem Dell laptops. Unfortunately
> there is at least one other Dell laptop which is affected. This adds
> its sub id's

Could you fix the comment, too?  Surely it's no Sony laptop ;)


--
Takashi Iwai <tiwai@suse.de>		ALSA Developer - www.alsa-project.org


> 
> [Patch from Dan Williams @ Red Hat slightly reformatted by me]
> 
> 
> --- sound/pci/nm256/nm256.c	2004-06-14 14:11:34.000000000 +0100
> +++ /tmp/nm256.c	2004-06-21 12:08:49.099484448 +0100
> @@ -1509,6 +1509,10 @@
>  		/* this workaround will cause lock-up after suspend/resume on Sony PCG-F305 */
>  		chip->latitude_workaround = 0;
>  	}
> +	if (subsystem_vendor == 0x1028 && subsystem_device == 0x0080) {
> +		/* this workaround will cause lock-up after suspend/resume on Sony PCG-F305 */
> +		chip->latitude_workaround = 0;
> +	}
>  
>  	snd_nm256_init_chip(chip);
>  
> 
> 
> 
>       Developer's Certificate of Origin 1.0
>  
>         By making a contribution to this project, I certify that:
>  
>         (a) The contribution was created in whole or in part by me and I
>             have the right to submit it under the open source license
>             indicated in the file; or
>  
>         (b) The contribution is based upon previous work that, to the best
>             of my knowledge, is covered under an appropriate open source
>             license and I have the right under that license to submit that
>             work with modifications, whether created in whole or in part
>             by me, under the same open source license (unless I am
>             permitted to submit under a different license), as indicated
>             in the file; or
>  
>         (c) The contribution was provided directly to me by some other
>             person who certified (a), (b) or (c) and I have not modified
>             it.
> 
> 
> Signed-off-by: Alan Cox <alan@redhat.com>
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2004-06-21 14:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-06-21 12:16 PATCH: Dell laptop lockup fix for ALSA Alan Cox
2004-06-21 14:24 ` Takashi Iwai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox