20 Feb
2013
20 Feb
'13
5:52 p.m.
On Mon, Feb 18, 2013 at 05:02:13PM +0530, Sachin Kamat wrote:
- if (val >= 1) {
if (w->reg == M98090_REG_MIC1_INPUT_LEVEL) {
- if (val >= 1){
Your coding style fixes shouldn't introduce new coding style issues and...
if (w->reg == M98090_REG_MIC1_INPUT_LEVEL)
...in cases where you're nesting with other if statements it's often clearer to have the braces consistently over all levels of the conditional.
For the isolated if statements this is fine though.