lkml.org 
[lkml]   [2000]   [Dec]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] i810 audio fixes for 2.4.0-test13-pre5
Date
From
> This patch addresses three problems in the i810-audio driver for
> 2.4.0-test13-pre5. I will be happy to split it if someone doesn't like
> part of it. (I see pre6 just popped out, there are no changes to this
> driver in pre6.)

> 1) "DMA overrun on send" - this contains a patch from Tjeerd Mulder that
> prevents almost all of this. I still get an occasional one during
> heavy video activity, the driver was unusable before. (This is a smaller
> patch than the previous flamed patch, it does no format conversion.)

Yeah I have Tjeerd's patches and no quibbles with them

> 3) Add a module parameter to supress powercycling the DACs on rate change -
> This causes a big pop on the outputs at least for the CS4299 codec in
> the emachines etower 700, probably others. I honestly can't find a
> reason to power cycle the DACs. There's nothing in the AC97 spec
> that suggests it should be done. The code is common to OSS and ALSA.
> I left the old behavior as the default. Maybe later the default should
> change if it turns out that everyone wants to force the parameter to
> zero.

What I would love someone to do (hint ;)) is to move the

set_dac_speed
set_adc_speed
powerup_amp
powerdown_amp

type stuff into the ac97_codec - actually have a codec_ops in the AC97
for each type.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

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