[alsa-devel] [PATCH 03/15] ASoC: adav80x: Update locking around use of DAPM pin API

Mark Brown broonie at kernel.org
Tue Feb 18 01:33:14 CET 2014


On Mon, Feb 17, 2014 at 04:51:31PM +0000, Charles Keepax wrote:

> +		mutex_lock(&codec->dapm.card->dapm_mutex);
> +

I think this needs a helper that just takes the CODEC as an argument and
does the lookup - a static inline in the header would be fine.  The big
long dereference is ugly for something people should be doing and will
be annoying if we want to refactor in the future.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20140218/94d759f1/attachment.sig>


More information about the Alsa-devel mailing list