Re: [PATCH] spi: pxa2xx: Remove the shutdown callback

From: Pavel Machek
Date: Sat Nov 03 2018 - 15:13:53 EST


On Mon 2018-09-17 14:11:24, Lubomir Rintel wrote:
> It makes no sense to remove the device on shutdown. And it break things
> when the hardware crucial for shutdown (such as the embedded controller)
> is attached to the SPI bus.

Again, are you sure?

On some hardware (sharp zaurus, phones) it is good idea to shut down
hardware before poweroff, as they don't have embedded controller that
would cut the power.

Will this increase power consumption while "off" on such platforms?
Pavel

> Cc: Daniel Mack <daniel@xxxxxxxxxx>
> Cc: Haojian Zhuang <haojian.zhuang@xxxxxxxxx>
> Cc: Robert Jarzmik <robert.jarzmik@xxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Signed-off-by: Lubomir Rintel <lkundrak@xxxxx>
> ---
> drivers/spi/spi-pxa2xx.c | 9 ---------
> 1 file changed, 9 deletions(-)
>
> diff --git a/drivers/spi/spi-pxa2xx.c b/drivers/spi/spi-pxa2xx.c
> index 14f4ea59caff..b06f37fb29f6 100644
> --- a/drivers/spi/spi-pxa2xx.c
> +++ b/drivers/spi/spi-pxa2xx.c
> @@ -1764,14 +1764,6 @@ static int pxa2xx_spi_remove(struct platform_device *pdev)
> return 0;
> }
>
> -static void pxa2xx_spi_shutdown(struct platform_device *pdev)
> -{
> - int status = 0;
> -
> - if ((status = pxa2xx_spi_remove(pdev)) != 0)
> - dev_err(&pdev->dev, "shutdown failed with %d\n", status);
> -}
> -
> #ifdef CONFIG_PM_SLEEP
> static int pxa2xx_spi_suspend(struct device *dev)
> {
> @@ -1851,7 +1843,6 @@ static struct platform_driver driver = {
> },
> .probe = pxa2xx_spi_probe,
> .remove = pxa2xx_spi_remove,
> - .shutdown = pxa2xx_spi_shutdown,
> };
>
> static int __init pxa2xx_spi_init(void)

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature