* [PATCH] i2c: qup: Fix pm_runtime_get_sync usage
@ 2014-05-03 1:54 Andy Gross
2014-05-03 6:40 ` Ivan T. Ivanov
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Andy Gross @ 2014-05-03 1:54 UTC (permalink / raw)
To: linux-i2c
Cc: Bjorn Andersson, linux-kernel, linux-arm-kernel, linux-arm-msm,
Andy Gross
This patch corrects the error check on the call to pm_runtime_get_sync.
Signed-off-by: Andy Gross <agross@codeaurora.org>
---
drivers/i2c/busses/i2c-qup.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c
index 1b4cf14..2a5efb5 100644
--- a/drivers/i2c/busses/i2c-qup.c
+++ b/drivers/i2c/busses/i2c-qup.c
@@ -479,7 +479,7 @@ static int qup_i2c_xfer(struct i2c_adapter *adap,
int ret, idx;
ret = pm_runtime_get_sync(qup->dev);
- if (ret)
+ if (ret < 0)
goto out;
writel(1, qup->base + QUP_SW_RESET);
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c: qup: Fix pm_runtime_get_sync usage
2014-05-03 1:54 [PATCH] i2c: qup: Fix pm_runtime_get_sync usage Andy Gross
@ 2014-05-03 6:40 ` Ivan T. Ivanov
2014-05-09 16:51 ` Bjorn Andersson
2014-05-14 16:07 ` Wolfram Sang
2 siblings, 0 replies; 4+ messages in thread
From: Ivan T. Ivanov @ 2014-05-03 6:40 UTC (permalink / raw)
To: Andy Gross
Cc: linux-i2c, Bjorn Andersson, linux-kernel, linux-arm-kernel,
linux-arm-msm
On Fri, 2014-05-02 at 20:54 -0500, Andy Gross wrote:
> This patch corrects the error check on the call to pm_runtime_get_sync.
>
> Signed-off-by: Andy Gross <agross@codeaurora.org>
Reviewed-by: Ivan T. Ivanov <iivanov@mm-sol.com>
> ---
> drivers/i2c/busses/i2c-qup.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c
> index 1b4cf14..2a5efb5 100644
> --- a/drivers/i2c/busses/i2c-qup.c
> +++ b/drivers/i2c/busses/i2c-qup.c
> @@ -479,7 +479,7 @@ static int qup_i2c_xfer(struct i2c_adapter *adap,
> int ret, idx;
>
> ret = pm_runtime_get_sync(qup->dev);
> - if (ret)
> + if (ret < 0)
> goto out;
>
> writel(1, qup->base + QUP_SW_RESET);
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c: qup: Fix pm_runtime_get_sync usage
2014-05-03 1:54 [PATCH] i2c: qup: Fix pm_runtime_get_sync usage Andy Gross
2014-05-03 6:40 ` Ivan T. Ivanov
@ 2014-05-09 16:51 ` Bjorn Andersson
2014-05-14 16:07 ` Wolfram Sang
2 siblings, 0 replies; 4+ messages in thread
From: Bjorn Andersson @ 2014-05-09 16:51 UTC (permalink / raw)
To: Andy Gross, Wolfram Sang
Cc: linux-i2c, Bjorn Andersson, linux-kernel, linux-arm-kernel,
linux-arm-msm
On Fri, May 2, 2014 at 6:54 PM, Andy Gross <agross@codeaurora.org> wrote:
[...]
> @@ -479,7 +479,7 @@ static int qup_i2c_xfer(struct i2c_adapter *adap,
> int ret, idx;
>
> ret = pm_runtime_get_sync(qup->dev);
> - if (ret)
> + if (ret < 0)
> goto out;
Sorry about that, thought I tested that path properly.
Wolfram, please pick this up as soon as possible as it renders the
driver useless if one enables pm runtime.
Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Regards,
Bjorn
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c: qup: Fix pm_runtime_get_sync usage
2014-05-03 1:54 [PATCH] i2c: qup: Fix pm_runtime_get_sync usage Andy Gross
2014-05-03 6:40 ` Ivan T. Ivanov
2014-05-09 16:51 ` Bjorn Andersson
@ 2014-05-14 16:07 ` Wolfram Sang
2 siblings, 0 replies; 4+ messages in thread
From: Wolfram Sang @ 2014-05-14 16:07 UTC (permalink / raw)
To: Andy Gross
Cc: linux-i2c, Bjorn Andersson, linux-kernel, linux-arm-kernel,
linux-arm-msm
[-- Attachment #1: Type: text/plain, Size: 230 bytes --]
On Fri, May 02, 2014 at 08:54:29PM -0500, Andy Gross wrote:
> This patch corrects the error check on the call to pm_runtime_get_sync.
>
> Signed-off-by: Andy Gross <agross@codeaurora.org>
Applied to for-current, thanks!
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-05-14 16:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-03 1:54 [PATCH] i2c: qup: Fix pm_runtime_get_sync usage Andy Gross
2014-05-03 6:40 ` Ivan T. Ivanov
2014-05-09 16:51 ` Bjorn Andersson
2014-05-14 16:07 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).