Skip to content

Commit 65e318e

Browse files
l1kbroonie
authored andcommitted
spi: pxa2xx: Fix runtime PM ref imbalance on probe error
The PXA2xx SPI driver releases a runtime PM ref in the probe error path even though it hasn't acquired a ref earlier. Apparently commit e2b714a ("spi: pxa2xx: Disable runtime PM if controller registration fails") sought to copy-paste the invocation of pm_runtime_disable() from pxa2xx_spi_remove(), but erroneously copied the call to pm_runtime_put_noidle() as well. Drop it. Fixes: e2b714a ("spi: pxa2xx: Disable runtime PM if controller registration fails") Signed-off-by: Lukas Wunner <[email protected]> Reviewed-by: Jarkko Nikula <[email protected]> Reviewed-by: Andy Shevchenko <[email protected]> Cc: [email protected] # v4.17+ Cc: Jarkko Nikula <[email protected]> Link: https://lore.kernel.org/r/58b2ac6942ca1f91aaeeafe512144bc5343e1d84.1590408496.git.lukas@wunner.de Signed-off-by: Mark Brown <[email protected]>
1 parent 32e5b57 commit 65e318e

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

drivers/spi/spi-pxa2xx.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1893,7 +1893,6 @@ static int pxa2xx_spi_probe(struct platform_device *pdev)
18931893
return status;
18941894

18951895
out_error_pm_runtime_enabled:
1896-
pm_runtime_put_noidle(&pdev->dev);
18971896
pm_runtime_disable(&pdev->dev);
18981897

18991898
out_error_clock_enabled:

0 commit comments

Comments
 (0)