lkml.org 
[lkml]   [2003]   [Feb]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH]: M5451 (OSS trident.c) did not come out of reset
Last time I booted 2.5, I noticed that my sound card no longer
works. The card is:

00:06.0 Multimedia audio controller: Acer Laboratories Inc. [ALi]
M5451 PCI AC-Link Controller Audio Device (rev 01)

And the computer is a thinkpad R30. It turns out that this patch, from
Alan Cox on 01/11/2002, broke it for me, by failing ali_reset_5451 if
the card doesn't come out of reset:

# --------------------------------------------
# 02/11/01 alan@lxorguk.ukuu.org.uk 1.786.161.45
# [PATCH] some trident needs longer delays to power up codecs
# --------------------------------------------

The 2.4 behaviour is to continue as usual even if the card doesn't
come out of reset, because it's a non fatal error on at least some
cards. This patch reverts the behaviour to the 2.4 behaviour, which
works for me. If anyone knows how to tell for a given card whether
this is a fatal error or not, please let me know and I'll update the
patch.

# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
# ChangeSet 1.1046 -> 1.1047
# sound/oss/trident.c 1.30 -> 1.31
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 03/02/18 mulix@alhambra.mulix.org 1.1047
# The M5451 can sometimes not come out of reset.
# This is non fatal and it continues to work fine, so print a nasty message
# but don't fail the driver initialization.
# --------------------------------------------
#
diff -Nru a/sound/oss/trident.c b/sound/oss/trident.c
--- a/sound/oss/trident.c Tue Feb 18 10:10:50 2003
+++ b/sound/oss/trident.c Tue Feb 18 10:10:50 2003
@@ -3933,8 +3933,10 @@
udelay(5000);
}

- printk(KERN_ERR "ALi 5451 did not come out of reset.\n");
- return 1;
+ /* This is non fatal if you have a non PM capable codec.. */
+ printk(KERN_ERR "ALi 5451 did not come out of reset "
+ "- continuing anyway.\n");
+ return 0;
}

/* AC97 codec initialisation. */






--
Muli Ben-Yehuda
http://www.mulix.org
http://syscalltrack.sf.net

-
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/

\
 
 \ /
  Last update: 2005-03-22 13:33    [W:0.036 / U:0.924 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site