[PATCH] uio: pm_runtime_disable is needed if failed

[PATCH] uio: pm_runtime_disable is needed if failed

[Kuninori Morimoto]

Signed-off-by: Kuninori Morimoto <morimoto.kuninori@renesas.com>
---
>> Magnus

Please check this patch.
I think pm_runtime_disable is needed here.

 drivers/uio/uio_pdrv_genirq.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/uio/uio_pdrv_genirq.c b/drivers/uio/uio_pdrv_genirq.c
index 02347c5..aa53db9 100644
--- a/drivers/uio/uio_pdrv_genirq.c
+++ b/drivers/uio/uio_pdrv_genirq.c
@@ -178,6 +178,7 @@ static int uio_pdrv_genirq_probe(struct platform_device *pdev)
 	return 0;
  bad1:
 	kfree(priv);
+	pm_runtime_disable(&pdev->dev);
  bad0:
 	return ret;
 }
-- 
1.6.3.3

Re: [PATCH] uio: pm_runtime_disable is needed if failed

[Magnus Damm]

On Mon, Nov 9, 2009 at 11:42 AM, Kuninori Morimoto
<morimoto.kuninori@renesas.com> wrote:
>
> Signed-off-by: Kuninori Morimoto <morimoto.kuninori@renesas.com>
> ---
>>> Magnus
>
> Please check this patch.
> I think pm_runtime_disable is needed here.

Looks correct!

/ magnus

Re: [PATCH] uio: pm_runtime_disable is needed if failed

[Paul Mundt]

On Mon, Nov 09, 2009 at 11:42:47AM +0900, Kuninori Morimoto wrote:
> 
> Signed-off-by: Kuninori Morimoto <morimoto.kuninori@renesas.com>
> ---
> >> Magnus
> 
> Please check this patch.
> I think pm_runtime_disable is needed here.
> 
>  drivers/uio/uio_pdrv_genirq.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
Unfortunately this missed my pull request by about an hour, so I'll queue
it up once Linus pulls the already pending stuff. I'll merge this for
2.6.32-rc8.