[alsa-devel] [PATCH v2 4/5] ASoC: tpa6130a2: Add DAPM support

[Helen Koike]

On 20-06-2016 16:44, Lars-Peter Clausen wrote:
>> +	/* before widget power up */
>> +	if (SND_SOC_DAPM_EVENT_ON(event)) {
>> +		/* Turn on the chip */
>> +		tpa6130a2_power(data, true);
>> +		/* Sync the registers */
>> +		ret = regcache_sync(data->regmap);
>> +		if (ret < 0) {
>> +			dev_err(c->dev, "Failed to initialize chip\n");
>> +			tpa6130a2_power(data, false);
>> +			return ret;
>> +		}
>> +	/* after widget power down */
>> +	} else
>> +		tpa6130a2_power(data, false);
>
> checkpatch.pl should complain about this. Kernel code style is if one branch
> has branches the other has to have it as well.
>

Weird, checkpatch.pl doesn't complain. I'll add the braces in v3, thank 
you for reviewing