diff for duplicates of <87lij7nltv.fsf@ti.com> diff --git a/a/1.txt b/N1/1.txt index 50c3ea2..56552a1 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ Hi Shubhrajyoti, -Shubhrajyoti Datta <omaplinuxkernel-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> writes: +Shubhrajyoti Datta <omaplinuxkernel@gmail.com> writes: > Hi Kevin, > Thanks for the patch , @@ -8,7 +8,7 @@ Shubhrajyoti Datta <omaplinuxkernel-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wri Thanks for the review. -> On Wed, Jun 27, 2012 at 7:15 AM, Kevin Hilman <khilman-l0cyMroinI0@public.gmane.org> wrote: +> On Wed, Jun 27, 2012 at 7:15 AM, Kevin Hilman <khilman@ti.com> wrote: >> In omap_i2c_xfer(), ensure pm_runtime_put() is called, even on >> failure. > So the failure means that the usecount is incremented. However the @@ -53,17 +53,17 @@ Kevin > Agree. > >> causing the usecount to never ->> reach zero after a failure. This keeps the device always runtime PM +>> reach zero after a failure. ?This keeps the device always runtime PM >> enabled which keeps the enclosing power domain active, and prevents >> full-chip retention/off from happening during idle. >> ->> Cc: Shubhrajyoti D <shubhrajyoti-l0cyMroinI0@public.gmane.org> ->> Signed-off-by: Kevin Hilman <khilman-l0cyMroinI0@public.gmane.org> +>> Cc: Shubhrajyoti D <shubhrajyoti@ti.com> +>> Signed-off-by: Kevin Hilman <khilman@ti.com> >> --- >> This patch applies to current i2c-embedded/for-next branch >> ->> drivers/i2c/busses/i2c-omap.c | 2 +- ->> 1 file changed, 1 insertion(+), 1 deletion(-) +>> ?drivers/i2c/busses/i2c-omap.c | ? ?2 +- +>> ?1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c >> index 9895fa7..b105733 100644 @@ -71,21 +71,21 @@ Kevin >> +++ b/drivers/i2c/busses/i2c-omap.c >> @@ -582,7 +582,7 @@ omap_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num) >> ->> r = pm_runtime_get_sync(dev->dev); ->> if (IS_ERR_VALUE(r)) ->> - return r; ->> + goto out; +>> ? ? ? ?r = pm_runtime_get_sync(dev->dev); +>> ? ? ? ?if (IS_ERR_VALUE(r)) +>> - ? ? ? ? ? ? ? return r; +>> + ? ? ? ? ? ? ? goto out; >> ->> r = omap_i2c_wait_for_bb(dev); ->> if (r < 0) +>> ? ? ? ?r = omap_i2c_wait_for_bb(dev); +>> ? ? ? ?if (r < 0) >> -- >> 1.7.9.2 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-omap" in ->> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org ->> More majordomo info at http://vger.kernel.org/majordomo-info.html +>> the body of a message to majordomo at vger.kernel.org +>> More majordomo info at ?http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in -> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org +> the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 28f9062..477d36f 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,20 +1,14 @@ "ref\01340761532-27231-1-git-send-email-khilman@ti.com\0" "ref\0CAM=Q2cuLCT2VzcUgrpyRWiX61t1F+iWchO9iLC=Ka9MFVm4_bQ@mail.gmail.com\0" - "ref\0CAM=Q2cuLCT2VzcUgrpyRWiX61t1F+iWchO9iLC=Ka9MFVm4_bQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org\0" - "From\0Kevin Hilman <khilman-l0cyMroinI0@public.gmane.org>\0" - "Subject\0Re: [PATCH] I2C: OMAP: xfer: fix runtime PM get/put balance on error\0" + "From\0khilman@ti.com (Kevin Hilman)\0" + "Subject\0[PATCH] I2C: OMAP: xfer: fix runtime PM get/put balance on error\0" "Date\0Thu, 28 Jun 2012 16:55:08 -0500\0" - "To\0Shubhrajyoti Datta <omaplinuxkernel-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>\0" - "Cc\0linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" - Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> - linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org - " Shubhrajyoti D <shubhrajyoti-l0cyMroinI0@public.gmane.org>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "Hi Shubhrajyoti,\n" "\n" - "Shubhrajyoti Datta <omaplinuxkernel-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> writes:\n" + "Shubhrajyoti Datta <omaplinuxkernel@gmail.com> writes:\n" "\n" "> Hi Kevin,\n" "> Thanks for the patch ,\n" @@ -22,7 +16,7 @@ "\n" "Thanks for the review.\n" "\n" - "> On Wed, Jun 27, 2012 at 7:15 AM, Kevin Hilman <khilman-l0cyMroinI0@public.gmane.org> wrote:\n" + "> On Wed, Jun 27, 2012 at 7:15 AM, Kevin Hilman <khilman@ti.com> wrote:\n" ">> In omap_i2c_xfer(), ensure pm_runtime_put() is called, even on\n" ">> failure.\n" "> So the failure means that the usecount is incremented. However the\n" @@ -67,17 +61,17 @@ "> Agree.\n" ">\n" ">> causing the usecount to never\n" - ">> reach zero after a failure. \302\240This keeps the device always runtime PM\n" + ">> reach zero after a failure. ?This keeps the device always runtime PM\n" ">> enabled which keeps the enclosing power domain active, and prevents\n" ">> full-chip retention/off from happening during idle.\n" ">>\n" - ">> Cc: Shubhrajyoti D <shubhrajyoti-l0cyMroinI0@public.gmane.org>\n" - ">> Signed-off-by: Kevin Hilman <khilman-l0cyMroinI0@public.gmane.org>\n" + ">> Cc: Shubhrajyoti D <shubhrajyoti@ti.com>\n" + ">> Signed-off-by: Kevin Hilman <khilman@ti.com>\n" ">> ---\n" ">> This patch applies to current i2c-embedded/for-next branch\n" ">>\n" - ">> \302\240drivers/i2c/busses/i2c-omap.c | \302\240 \302\2402 +-\n" - ">> \302\2401 file changed, 1 insertion(+), 1 deletion(-)\n" + ">> ?drivers/i2c/busses/i2c-omap.c | ? ?2 +-\n" + ">> ?1 file changed, 1 insertion(+), 1 deletion(-)\n" ">>\n" ">> diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c\n" ">> index 9895fa7..b105733 100644\n" @@ -85,23 +79,23 @@ ">> +++ b/drivers/i2c/busses/i2c-omap.c\n" ">> @@ -582,7 +582,7 @@ omap_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)\n" ">>\n" - ">> \302\240 \302\240 \302\240 \302\240r = pm_runtime_get_sync(dev->dev);\n" - ">> \302\240 \302\240 \302\240 \302\240if (IS_ERR_VALUE(r))\n" - ">> - \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 return r;\n" - ">> + \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 goto out;\n" + ">> ? ? ? ?r = pm_runtime_get_sync(dev->dev);\n" + ">> ? ? ? ?if (IS_ERR_VALUE(r))\n" + ">> - ? ? ? ? ? ? ? return r;\n" + ">> + ? ? ? ? ? ? ? goto out;\n" ">>\n" - ">> \302\240 \302\240 \302\240 \302\240r = omap_i2c_wait_for_bb(dev);\n" - ">> \302\240 \302\240 \302\240 \302\240if (r < 0)\n" + ">> ? ? ? ?r = omap_i2c_wait_for_bb(dev);\n" + ">> ? ? ? ?if (r < 0)\n" ">> --\n" ">> 1.7.9.2\n" ">>\n" ">> --\n" ">> To unsubscribe from this list: send the line \"unsubscribe linux-omap\" in\n" - ">> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" - ">> More majordomo info at \302\240http://vger.kernel.org/majordomo-info.html\n" + ">> the body of a message to majordomo at vger.kernel.org\n" + ">> More majordomo info at ?http://vger.kernel.org/majordomo-info.html\n" "> --\n" "> To unsubscribe from this list: send the line \"unsubscribe linux-omap\" in\n" - "> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + "> the body of a message to majordomo at vger.kernel.org\n" > More majordomo info at http://vger.kernel.org/majordomo-info.html -bbc42bd22270c6af0f09645fc57c79a2e03827a73c564ab9ce67a03a31d82c09 +ece9b0b927d4d20e210b1bb6b7087563600f9a8c68b9ae098dbfaccfeae0fb16
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.