From b2f388aca257d8e1de55090280cfce6ba3e1ea8c Mon Sep 17 00:00:00 2001 From: Jyri Sarha Date: Wed, 3 Aug 2016 16:28:26 +0530 Subject: [PATCH] pwm: pwm-tipwmss: Remove all pm_runtime gets and puts from the driver Remove all pm_runtime gets and puts, and dummy pm_ops, from the pwm-tipwmss driver as there is no direct hardware access. The runtime PM needs to be enabled, so that the runtime PM framework takes care of enabling/disabling of PWMSS clock when submodules of PWMSS (ECAP or EHRPWM) call pm_runtime APIs. With this change PWMSS clock goes to idle when none of the submodules are in use. Signed-off-by: Jyri Sarha Tested-by: Dave Gerlach Signed-off-by: Vignesh R Signed-off-by: Sekhar Nori --- drivers/pwm/pwm-tipwmss.c | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/drivers/pwm/pwm-tipwmss.c b/drivers/pwm/pwm-tipwmss.c index 829f4991c96f5..7fa85a1604da9 100644 --- a/drivers/pwm/pwm-tipwmss.c +++ b/drivers/pwm/pwm-tipwmss.c @@ -34,7 +34,6 @@ static int pwmss_probe(struct platform_device *pdev) struct device_node *node = pdev->dev.of_node; pm_runtime_enable(&pdev->dev); - pm_runtime_get_sync(&pdev->dev); /* Populate all the child nodes here... */ ret = of_platform_populate(node, NULL, NULL, &pdev->dev); @@ -46,31 +45,13 @@ static int pwmss_probe(struct platform_device *pdev) static int pwmss_remove(struct platform_device *pdev) { - pm_runtime_put_sync(&pdev->dev); pm_runtime_disable(&pdev->dev); return 0; } -#ifdef CONFIG_PM_SLEEP -static int pwmss_suspend(struct device *dev) -{ - pm_runtime_put_sync(dev); - return 0; -} - -static int pwmss_resume(struct device *dev) -{ - pm_runtime_get_sync(dev); - return 0; -} -#endif - -static SIMPLE_DEV_PM_OPS(pwmss_pm_ops, pwmss_suspend, pwmss_resume); - static struct platform_driver pwmss_driver = { .driver = { .name = "pwmss", - .pm = &pwmss_pm_ops, .of_match_table = pwmss_of_match, }, .probe = pwmss_probe,