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
-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
