Re: [PATCH 1/5] HDA : patch_sigmatel.c : cleanups

[Maxim Levitsky <maximlevitsky@gmail.com>]

On Friday 31 August 2007 13:56:19 Takashi Iwai wrote:
> At Tue, 28 Aug 2007 12:46:01 +0200,
> I wrote:
> > 
> > At Tue, 28 Aug 2007 05:07:48 +0300,
> > Maxim Levitsky wrote:
> > > 
> > > From c9392b0293f7d39e40e4a5e07e7b140d5a385079 Mon Sep 17 00:00:00 2001
> > > From: Maxim Levitsky <maximlevitsky@gmail.com>
> > > Date: Mon, 27 Aug 2007 22:49:49 +0300
> > > Subject: [PATCH] HDA : patch_sigmatel.c : cleanups
> > 
> > OK, it's a nice clean-up.  Can be merged as is.
> 
> Now merged to ALSA HG tree.
> 
> If the fixes for other patches are ready, please let me know.
> I'll merge them to HG tree, too.
> 
> 
> Thanks,
> 
> Takashi
> 


Hi,

I run into unexpected problems:
latest hg adds a power saving mode to hda codec, and while the idea is nice, it plays very bad with my patches.

1) The analog loop-back becomes a nightmare, since power-saving code turns the codec off, but for loopback to function it has to be powered.
I tried to add a snd_hda_power_up/down calls to .put function, and while it did stop the unwanted powerdown of codec, on resume from ram , analog loopback vanishes.
any access to device (arecord/aplay or any mixer change make analog loop work again).
Can you tell me how I properly tell the core that user _uses_ the device, and device should be powered.

2) on resume all playback apps hang, and recording (arecord) exits with message "Resource temporary unavailable"
restarting apps help.

Looking for your comments,
	Best regards,
		Maxim Levitsky