pwm: samsung: Simplify code to determine the pwmchip's parent device
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Wed, 14 Feb 2024 09:32:23 +0000 (10:32 +0100)
committerUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Mon, 19 Feb 2024 10:04:14 +0000 (11:04 +0100)
There is already a pointer to the pwmchip, make use of it directly
instead of using the struct samsung_pwm_chip *our_chip just obtained
from it. This also has the advantage of not using struct
samsung_pwm_chip::chip any more which will be dropped soon.

Link: https://lore.kernel.org/r/4e8400a42979b6be0b80dff5ae964d8e9d6232eb.1707900770.git.u.kleine-koenig@pengutronix.de
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
drivers/pwm/pwm-samsung.c

index 6e77302f73687f5ac4e63b1a00663b639772264b..0f9a2a5e3ebcbd5942605ab8b93c5a6c8b869856 100644 (file)
@@ -326,12 +326,12 @@ static int __pwm_samsung_config(struct pwm_chip *chip, struct pwm_device *pwm,
 
                period = NSEC_PER_SEC / period_ns;
 
-               dev_dbg(our_chip->chip.dev, "duty_ns=%d, period_ns=%d (%u)\n",
+               dev_dbg(chip->dev, "duty_ns=%d, period_ns=%d (%u)\n",
                                                duty_ns, period_ns, period);
 
                tin_rate = pwm_samsung_calc_tin(our_chip, pwm->hwpwm, period);
 
-               dev_dbg(our_chip->chip.dev, "tin_rate=%lu\n", tin_rate);
+               dev_dbg(chip->dev, "tin_rate=%lu\n", tin_rate);
 
                tin_ns = NSEC_PER_SEC / tin_rate;
                tcnt = period_ns / tin_ns;
@@ -355,8 +355,7 @@ static int __pwm_samsung_config(struct pwm_chip *chip, struct pwm_device *pwm,
        /* -1UL will give 100% duty. */
        --tcmp;
 
-       dev_dbg(our_chip->chip.dev,
-                               "tin_ns=%u, tcmp=%u/%u\n", tin_ns, tcmp, tcnt);
+       dev_dbg(chip->dev, "tin_ns=%u, tcmp=%u/%u\n", tin_ns, tcmp, tcnt);
 
        /* Update PWM registers. */
        writel(tcnt, our_chip->base + REG_TCNTB(pwm->hwpwm));
@@ -368,7 +367,7 @@ static int __pwm_samsung_config(struct pwm_chip *chip, struct pwm_device *pwm,
         * shortly afer this update (before it autoreloaded the new values).
         */
        if (oldtcmp == (u32) -1) {
-               dev_dbg(our_chip->chip.dev, "Forcing manual update");
+               dev_dbg(chip->dev, "Forcing manual update");
                pwm_samsung_manual_update(our_chip, pwm);
        }