Message ID | YNx1y8PlSLehZVIY@mwanda |
---|---|
State | Not Applicable |
Headers | show |
Series | pwm: ep93xx: Fix uninitialized variable bug in ep93xx_pwm_apply() | expand |
Hello, this problem was found already earlier by Colin King: https://lore.kernel.org/r/20210629172253.43131-1-colin.king@canonical.com I'm fine with either change. Best regards Uwe
Hello, On Wed, Jun 30, 2021 at 05:36:00PM +0200, Uwe Kleine-König wrote: > this problem was found already earlier by Colin King: > > https://lore.kernel.org/r/20210629172253.43131-1-colin.king@canonical.com > > I'm fine with either change. FTR: Thierry applied Colin's patch, so I'm discarding this one from patchwork as not applicable. Best regards Uwe
diff --git a/drivers/pwm/pwm-ep93xx.c b/drivers/pwm/pwm-ep93xx.c index 70fa2957f9d3..ffa79248c1e1 100644 --- a/drivers/pwm/pwm-ep93xx.c +++ b/drivers/pwm/pwm-ep93xx.c @@ -61,7 +61,7 @@ static void ep93xx_pwm_free(struct pwm_chip *chip, struct pwm_device *pwm) static int ep93xx_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm, const struct pwm_state *state) { - int ret; + int ret = 0; struct ep93xx_pwm *ep93xx_pwm = to_ep93xx_pwm(chip); bool enabled = state->enabled;
Smatch found a potential uninitialized variable in ep93xx_pwm_apply(): drivers/pwm/pwm-ep93xx.c:147 ep93xx_pwm_apply() error: uninitialized symbol 'ret'. Initialize "ret" to zero at the start to solve this issue. Fixes: f6ef94edf0f6 ("pwm: ep93xx: Unfold legacy callbacks into ep93xx_pwm_apply()") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- drivers/pwm/pwm-ep93xx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)