Messages in this thread | | | Date | Fri, 24 Mar 2017 15:28:54 +0100 | From | Andrew Lunn <> | Subject | Re: [PATCH v3 1/4] gpio: mvebu: Add limited PWM support |
| |
> + if (val < state->duty_cycle) > + state->period = 1; > + else { > + val -= state->duty_cycle; > + if (val > UINT_MAX) > + state->period = UINT_MAX; > + else if (val) > + state->period = val; > + else > + state->period = 1; > + }
Hi Ralph
The coding style guide says that if one side of an if else clause has {}, the other side must as well, even if it is a single statement.
Andrew
| |