diff options
| author | Thierry Reding <[email protected]> | 2012-09-02 22:13:40 +0200 |
|---|---|---|
| committer | Thierry Reding <[email protected]> | 2012-10-05 20:56:43 +0200 |
| commit | c2d476a98f71c55e9acdca1d5a1080a22c0622af (patch) | |
| tree | 550ef5612d413d39f6d5928ea261eab9fd859fac /drivers/pwm/pwm-pxa.c | |
| parent | f6b8a5700057cc1b531c2f9b7806428a6f83b467 (diff) | |
pwm: Check for negative duty-cycle and period
Make sure the duty-cycle and period passed in are not negative. This
should eventually be made implicit by making them unsigned. While at
it, the drivers' .config() implementations can have the equivalent
checks removed.
Signed-off-by: Thierry Reding <[email protected]>
Cc: Shawn Guo <[email protected]>
Cc: Mark Brown <[email protected]>
Cc: Arnd Bergmann <[email protected]>
Cc: Sachin Kamat <[email protected]>
Cc: Axel Lin <[email protected]>
Cc: Kukjin Kim <[email protected]>
Cc: Jingoo Han <[email protected]>
Cc: Jonghwan Choi <[email protected]>
Cc: Sascha Hauer <[email protected]>
Cc: "Philip, Avinash" <[email protected]>
Cc: Vaibhav Bedia <[email protected]>
Acked-by: Jingoo Han <[email protected]>
Diffstat (limited to 'drivers/pwm/pwm-pxa.c')
| -rw-r--r-- | drivers/pwm/pwm-pxa.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/pwm/pwm-pxa.c b/drivers/pwm/pwm-pxa.c index bd5867a1c700..260c3a88564d 100644 --- a/drivers/pwm/pwm-pxa.c +++ b/drivers/pwm/pwm-pxa.c @@ -70,9 +70,6 @@ static int pxa_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm, unsigned long offset; int rc; - if (period_ns == 0 || duty_ns > period_ns) - return -EINVAL; - offset = pwm->hwpwm ? 0x10 : 0; c = clk_get_rate(pc->clk); |