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

Peter Ujfalusi peter.ujfalusi at nokia.com
Fri Oct 9 15:36:16 CEST 2009


On Friday 09 October 2009 16:10:31 Nurkkala Eero.An (EXT-Offcode/Oulu) wrote:
> On Fri, 2009-10-09 at 14:55 +0200, Ujfalusi Peter (Nokia-D/Tampere)
> 
> 
> During the tpa6130a2_i2c_write, you read the data->power_state, that
> may change it's state meanwhile (preempted). Thus, I suggest you using
> the mutex to cover all i2c writes. (and all tpa read <-> write cycles,
> so that things are consistent?)
> 
> A call to tpa6130a2_power() -> preemted over somewhere here, you may
> have i2c accesses to a chip that's not powered up?

Well it is very unlikely that something like this could ever happen since the 
tpa6130a2_channel_enable and the tpa6130a2_power is called when DAPM is 
enabling/disabling the path. I think in a middle of enable or disable there 
should not be any opposite operation running in DAPM...

But better to be safe: I can add the mutex_lock, unlock pair.

I'll wait for more comments and address this one as well.

-- 
Péter


More information about the Alsa-devel mailing list