diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2019-01-09 03:27:47 -0500 |
---|---|---|
committer | Thierry Reding <thierry.reding@gmail.com> | 2019-01-16 02:45:38 -0500 |
commit | 7ca17b207127240b1e4c8687ba7b9077e655336d (patch) | |
tree | d87db62c5dc432a49cb12e2eb205b804904a66a9 /drivers/pwm/pwm-imx27.c | |
parent | d80f8206905c1a8c3857d90f12bbfd6293b48a4b (diff) |
pwm: imx: Signedness bug in imx_pwm_get_state()
"ret" only holds zero and negative error codes. It needs to be signed
for the error handling to work.
Fixes: 9f4c8f9607c3 ("pwm: imx: Add ipg clock operation")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
Diffstat (limited to 'drivers/pwm/pwm-imx27.c')
-rw-r--r-- | drivers/pwm/pwm-imx27.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pwm/pwm-imx27.c b/drivers/pwm/pwm-imx27.c index 8b8b1c6b7f29..55666cca4cee 100644 --- a/drivers/pwm/pwm-imx27.c +++ b/drivers/pwm/pwm-imx27.c | |||
@@ -115,8 +115,9 @@ static void pwm_imx27_get_state(struct pwm_chip *chip, | |||
115 | struct pwm_device *pwm, struct pwm_state *state) | 115 | struct pwm_device *pwm, struct pwm_state *state) |
116 | { | 116 | { |
117 | struct pwm_imx27_chip *imx = to_pwm_imx27_chip(chip); | 117 | struct pwm_imx27_chip *imx = to_pwm_imx27_chip(chip); |
118 | u32 period, prescaler, pwm_clk, ret, val; | 118 | u32 period, prescaler, pwm_clk, val; |
119 | u64 tmp; | 119 | u64 tmp; |
120 | int ret; | ||
120 | 121 | ||
121 | ret = pwm_imx27_clk_prepare_enable(chip); | 122 | ret = pwm_imx27_clk_prepare_enable(chip); |
122 | if (ret < 0) | 123 | if (ret < 0) |