pwm: pwm-tipwmss: Remove all runtime PM gets/puts (549c50af) · Commits · 戴 / test

Remove all pm_runtime_get_sync() and pm_runtime_put_sync() call as well as the dummy pm_ops from the pwm-tipwmss driver. No registers are being modified. The runtime PM still needs to be enabled, so that the runtime PM framework can take care of enabling/disabling the PWMSS clock when submodules of PWMSS (ECAP or EHRPWM) call runtime PM APIs. With this change PWMSS clock goes to idle when none of the submodules are in use. Signed-off-by:

Jyri Sarha <jsarha@ti.com> Signed-off-by:

Vignesh R <vigneshr@ti.com> Signed-off-by:

Thierry Reding <thierry.reding@gmail.com>

drivers/pwm/pwm-tipwmss.c

+0 −19

Original line number	Diff line number	Diff line
		@@ -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,

Admin message