diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2024-02-14 10:31:41 +0100 |
|---|---|---|
| committer | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2024-02-19 11:04:10 +0100 |
| commit | 1b6691680bcc845276b955bc078e953c86c6ecd3 (patch) | |
| tree | 405f4ec2888282826028f06576aca18ba4542131 /drivers/pwm | |
| parent | 07b61160e45d3f6694708e2f9891845713043625 (diff) | |
| download | linux-1b6691680bcc845276b955bc078e953c86c6ecd3.tar.gz | |
pwm: iqs620a: Make use of devm_pwmchip_alloc() function
This prepares the pwm-iqs620a driver to further changes of the pwm core
outlined in the commit introducing devm_pwmchip_alloc(). There is no
intended semantical change and the driver should behave as before.
Link: https://lore.kernel.org/r/85cb0e73baadd76ede364abc4b57ee72000880e5.1707900770.git.u.kleine-koenig@pengutronix.de
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Diffstat (limited to 'drivers/pwm')
| -rw-r--r-- | drivers/pwm/pwm-iqs620a.c | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/drivers/pwm/pwm-iqs620a.c b/drivers/pwm/pwm-iqs620a.c index faf52d3c2720b..13e5e138c8e9c 100644 --- a/drivers/pwm/pwm-iqs620a.c +++ b/drivers/pwm/pwm-iqs620a.c @@ -35,7 +35,6 @@ struct iqs620_pwm_private { struct iqs62x_core *iqs62x; struct device *dev; - struct pwm_chip chip; struct notifier_block notifier; struct mutex lock; unsigned int duty_scale; @@ -43,7 +42,7 @@ struct iqs620_pwm_private { static inline struct iqs620_pwm_private *iqs620_pwm_from_chip(struct pwm_chip *chip) { - return container_of(chip, struct iqs620_pwm_private, chip); + return pwmchip_get_drvdata(chip); } static int iqs620_pwm_init(struct iqs620_pwm_private *iqs620_pwm, @@ -189,14 +188,16 @@ static void iqs620_pwm_notifier_unregister(void *context) static int iqs620_pwm_probe(struct platform_device *pdev) { struct iqs62x_core *iqs62x = dev_get_drvdata(pdev->dev.parent); + struct pwm_chip *chip; struct iqs620_pwm_private *iqs620_pwm; unsigned int val; int ret; - iqs620_pwm = devm_kzalloc(&pdev->dev, sizeof(*iqs620_pwm), GFP_KERNEL); - if (!iqs620_pwm) - return -ENOMEM; + chip = devm_pwmchip_alloc(&pdev->dev, 1, sizeof(*iqs620_pwm)); + if (IS_ERR(chip)) + return PTR_ERR(chip); + iqs620_pwm = iqs620_pwm_from_chip(chip); iqs620_pwm->dev = &pdev->dev; iqs620_pwm->iqs62x = iqs62x; @@ -212,9 +213,7 @@ static int iqs620_pwm_probe(struct platform_device *pdev) iqs620_pwm->duty_scale = val + 1; } - iqs620_pwm->chip.dev = &pdev->dev; - iqs620_pwm->chip.ops = &iqs620_pwm_ops; - iqs620_pwm->chip.npwm = 1; + chip->ops = &iqs620_pwm_ops; mutex_init(&iqs620_pwm->lock); @@ -232,7 +231,7 @@ static int iqs620_pwm_probe(struct platform_device *pdev) if (ret) return ret; - ret = devm_pwmchip_add(&pdev->dev, &iqs620_pwm->chip); + ret = devm_pwmchip_add(&pdev->dev, chip); if (ret) dev_err(&pdev->dev, "Failed to add device: %d\n", ret); |