[alsa-devel] [PATCH - try2] ASoC: TPA6130A2 amplifier driver

Mark Brown broonie at opensource.wolfsonmicro.com
Fri Oct 9 20:33:46 CEST 2009

On Fri, Oct 09, 2009 at 08:24:49PM +0200, ext-Eero.Nurkkala at nokia.com wrote:

> I guess there's (or may be) an exception though. I think I've seen some
> strangeness with the close_delayed_work() -> and simultaneous mixer tweaking:

As I said when you posted previously that's a problem anyway and
close_delayed_work() needs to be fixed.  You had a half-posted patch,
did you manage to test it properly (even in normal operation)?  I'd been
expecting a proper posting of it to appear at some point.

> So there can and does run two dapm_power_widgets() when stars
> point to the right direction. That's when delayed work is at
> damp_power_widgets, and then preempted to
> the userspace thread.  May that isn't related to this TPA though, but
> I think I'll silently review the code so that the "stars are forced to
> point to the right direction".

This is completely unrelated to driver code, the core's data is much
more of a problem than the register cache which will generally fix
itself the next time DAPM does anything.

More information about the Alsa-devel mailing list