[alsa-devel] [PATCH 2/2] ASoC: dapm: Fix SUSPEND -> OFF bias sequence

Mark Brown broonie at kernel.org
Mon May 5 21:41:29 CEST 2014


On Sun, May 04, 2014 at 07:17:06PM +0200, Lars-Peter Clausen wrote:

> -	/* Prepare for a STADDBY->ON or ON->STANDBY transition */
> -	if (d->bias_level != d->target_bias_level) {
> +	/* Prepare for a transition to ON or away from ON */
> +	if ((d->target_bias_level == SND_SOC_BIAS_ON &&
> +		 d->bias_level != SND_SOC_BIAS_ON) ||
> +		(d->target_bias_level != SND_SOC_BIAS_ON &&
> +		 d->bias_level == SND_SOC_BIAS_ON)) {

Can you fix the indentation so that the two sides of the or are lined
up?  It's a bit of a fiddly test and having the two cases not lined up
makes it harder to read.
-------------- 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/20140505/b03e7743/attachment.sig>


More information about the Alsa-devel mailing list