Re: debugging nm256: some results, many questions

[Florian Schlichting <Florian.Schlichting@gmx.de>]

[-- Attachment #1: Type: text/plain, Size: 2191 bytes --]

On Mon, Mar 13, 2006 at 04:50:31PM +0100, Takashi Iwai wrote:
> How about the patch below?  (It's only for CVS version)

very good. It makes the larger changes that I didn't dare to think of
for lack of insight. 

There's just another tiny cosmetical change, to the comments:

--- nm256.c.orig	2006-03-15 00:50:55.000000000 +0800
+++ nm256.c	2006-03-15 00:59:28.000000000 +0800
@@ -1157,7 +1157,8 @@
 
 /*
  * some nm256 easily crash when reading from mixer registers
- * thus we're caching both read and written values for reading
+ * thus we're treating it as a write-only mixer and cache the 
+ * written values
  */
 static unsigned short
 snd_nm256_ac97_read(struct snd_ac97 *ac97, unsigned short reg)
@@ -1200,9 +1201,6 @@
 }
 
 /* 
- * below copied from OSS driver
- * this list could perhaps be fused with mixer_regs[] ??
- * 
  * Initial register values to be written to the AC97 mixer.
  * While most of these are identical to the reset values, we do this
  * so that we have most of the register contents cached--this avoids



NB: while trying to compile from CVS with ./cvscompile [...] 
-----------
patching file usbaudio.c
Hunk #2 FAILED at 70.
-----------
Running the command again it aparently ignored this problem

Also, there are some warnings during compile and a problem with depmod 
loops at install
----------
  Building modules, stage 2.
  MODPOST
*** Warning: "snd_minor_info_oss_init" [/usr/src/cvs/alsa/alsa-driver/acore/snd.ko] undefined!
*** Warning: "snd_lookup_oss_minor_data" [/usr/src/cvs/alsa/alsa-driver/acore/snd.ko] undefined!
*** Warning: "snd_info_minor_unregister" [/usr/src/cvs/alsa/alsa-driver/acore/snd.ko] undefined!
*** Warning: "snd_minor_info_oss_done" [/usr/src/cvs/alsa/alsa-driver/acore/snd.ko] undefined!
*** Warning: "snd_info_minor_register" [/usr/src/cvs/alsa/alsa-driver/acore/snd.ko] undefined!

[...]

WARNING: Loop detected:
/lib/modules/2.6.15-max/kernel/sound/acore/snd.ko which needs snd.ko
again!
----------
(and the rest of the ALSA modules is ignored), so although I could 
build things, it wouldn't "make install" and I can't make a final smoke 
test :-(

Florian

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]