pwm: lpc18xx-sct: Drop hardly used member from driver private data
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Wed, 14 Feb 2024 09:31:47 +0000 (10:31 +0100)
committerUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Mon, 19 Feb 2024 10:04:10 +0000 (11:04 +0100)
The device pointer is only used once in lpc18xx_pwm_request(). There the
pwm_chip is available, too, which has a pointer to the same structure.
Replace the usage accordingly and drop the then write-only member
variable.

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

index fe891fa71a1ddc7fd50ac7cd444cce53aa35ebff..754ea2c3f49bed0b38d1a22804337db20feade01 100644 (file)
@@ -92,7 +92,6 @@ struct lpc18xx_pwm_data {
 };
 
 struct lpc18xx_pwm_chip {
-       struct device *dev;
        struct pwm_chip chip;
        void __iomem *base;
        struct clk *pwm_clk;
@@ -289,7 +288,7 @@ static int lpc18xx_pwm_request(struct pwm_chip *chip, struct pwm_device *pwm)
                                    LPC18XX_PWM_EVENT_MAX);
 
        if (event >= LPC18XX_PWM_EVENT_MAX) {
-               dev_err(lpc18xx_pwm->dev,
+               dev_err(chip->dev,
                        "maximum number of simultaneous channels reached\n");
                return -EBUSY;
        }
@@ -358,8 +357,6 @@ static int lpc18xx_pwm_probe(struct platform_device *pdev)
        if (!lpc18xx_pwm)
                return -ENOMEM;
 
-       lpc18xx_pwm->dev = &pdev->dev;
-
        lpc18xx_pwm->base = devm_platform_ioremap_resource(pdev, 0);
        if (IS_ERR(lpc18xx_pwm->base))
                return PTR_ERR(lpc18xx_pwm->base);