pwm: meson: Drop useless check for channel data being NULL
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Mon, 8 Nov 2021 13:46:25 +0000 (14:46 +0100)
committerThierry Reding <thierry.reding@gmail.com>
Tue, 1 Feb 2022 07:34:04 +0000 (08:34 +0100)
In meson_pwm_free() the function pwm_get_chip_data() always returns a
non-NULL pointer because it's only called when the request callback
succeeded and this callback calls pwm_set_chip_data() in this case.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
drivers/pwm/pwm-meson.c

index be3c806..1fbe54a 100644 (file)
@@ -148,8 +148,7 @@ static void meson_pwm_free(struct pwm_chip *chip, struct pwm_device *pwm)
 {
        struct meson_pwm_channel *channel = pwm_get_chip_data(pwm);
 
-       if (channel)
-               clk_disable_unprepare(channel->clk);
+       clk_disable_unprepare(channel->clk);
 }
 
 static int meson_pwm_calc(struct meson_pwm *meson, struct pwm_device *pwm,