]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
pwm: Simplify pwm_capture()
authorUwe Kleine-König <u.kleine-koenig@baylibre.com>
Mon, 29 Jul 2024 14:34:17 +0000 (16:34 +0200)
committerUwe Kleine-König <ukleinek@kernel.org>
Thu, 5 Sep 2024 09:14:14 +0000 (11:14 +0200)
When pwm_capture() is called, pwm is valid, so the checks for pwm and
pwm->chip->ops being NULL can be dropped.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
Link: https://lore.kernel.org/r/ee7b3322c7b3e28defdfb886a70b8ba40d298416.1722261050.git.u.kleine-koenig@baylibre.com
Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>
drivers/pwm/core.c

index 56d91c11f0d4d57bfe4ef795f7672f5ed12cd8ac..6e752e148b98ccad653798c817f973ce532df066 100644 (file)
@@ -328,15 +328,15 @@ EXPORT_SYMBOL_GPL(pwm_adjust_config);
 static int pwm_capture(struct pwm_device *pwm, struct pwm_capture *result,
                       unsigned long timeout)
 {
-       if (!pwm || !pwm->chip->ops)
-               return -EINVAL;
+       struct pwm_chip *chip = pwm->chip;
+       const struct pwm_ops *ops = chip->ops;
 
-       if (!pwm->chip->ops->capture)
+       if (!ops->capture)
                return -ENOSYS;
 
        guard(mutex)(&pwm_lock);
 
-       return pwm->chip->ops->capture(pwm->chip, pwm, result, timeout);
+       return ops->capture(chip, pwm, result, timeout);
 }
 
 static struct pwm_chip *pwmchip_find_by_name(const char *name)