[PATCH V2 3/3] spi: spi-pl022: Minor simplification for runtime pm
Ulf Hansson
ulf.hansson at linaro.org
Thu Oct 4 05:07:03 EDT 2012
Hi,
Mark, I am not sure this particular patch is actually wanted. Realized
that when reading up on the driver/base/* patches for PM changes this
summer. Especially how device probe/suspend/shutdown etc. has been
changed for runtime PM point of view.
Maybe you can comment?
Kind regards
Ulf Hansson
On 4 October 2012 10:04, Ulf Hansson <ulf.hansson at stericsson.com> wrote:
> From: Ulf Hansson <ulf.hansson at linaro.org>
>
> In probe pm_runtime_put_autosuspend has the same effect as doing
> pm_runtime_put. This due to upper layer in driver core is preventing
> the device from being runtime suspended by a pm_runtime_get*.
>
> Signed-off-by: Ulf Hansson <ulf.hansson at linaro.org>
> Reviewed-by: Linus Walleij <linus.walleij at linaro.org>
> ---
> drivers/spi/spi-pl022.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c
> index a1db91a..51b7a95 100644
> --- a/drivers/spi/spi-pl022.c
> +++ b/drivers/spi/spi-pl022.c
> @@ -2246,10 +2246,9 @@ pl022_probe(struct amba_device *adev, const struct amba_id *id)
> pm_runtime_set_autosuspend_delay(dev,
> platform_info->autosuspend_delay);
> pm_runtime_use_autosuspend(dev);
> - pm_runtime_put_autosuspend(dev);
> - } else {
> - pm_runtime_put(dev);
> }
> + pm_runtime_put(dev);
> +
> return 0;
>
> err_spi_register:
> --
> 1.7.10
>
More information about the linux-arm-kernel
mailing list