[PATCH] emu10k1: Fix outl() in snd_emu10k1_resume_regs()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,


I've given a try to S3/S4 states on my box at home and it turned out
that I had to reenable the Analog/Digital output switch to get sound
again.
A quick look at the resume code showed that it was trying to restore the
A_IOCFG and HCFG ports content with arguments to outl() reversed.
My trivial patch fixes this issue.


Regards,
Arnaud

The emu10k1 driver saves the A_IOCFG and HCFG register on suspend and restores
it on resumes. Unfortunately, this doesn't work as the arguments to outl() are
reversed.

Signed-off-by: Arnaud Patard <arnaud.patard@xxxxxxxxxxx>

---
 sound/pci/emu10k1/emu10k1_main.c |    4 	2 +	2 -	0 !
 1 file changed, 2 insertions(+), 2 deletions(-)

Index: linux-2.6.18-rc7/sound/pci/emu10k1/emu10k1_main.c
===================================================================
--- linux-2.6.18-rc7.orig/sound/pci/emu10k1/emu10k1_main.c	2006-10-01 19:49:28.000000000 +0200
+++ linux-2.6.18-rc7/sound/pci/emu10k1/emu10k1_main.c	2006-10-01 20:21:08.000000000 +0200
@@ -1460,8 +1460,8 @@ void snd_emu10k1_resume_regs(struct snd_
 
 	/* resore for spdif */
 	if (emu->audigy)
-		outl(emu->port + A_IOCFG, emu->saved_a_iocfg);
-	outl(emu->port + HCFG, emu->saved_hcfg);
+		outl(emu->saved_a_iocfg, emu->port + A_IOCFG);
+	outl(emu->saved_hcfg, emu->port + HCFG);
 
 	val = emu->saved_ptr;
 	for (reg = saved_regs; *reg != 0xff; reg++)
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux