[PATCH] Input: ims-pcu - fix error unwinding path in application mode

Subsystems: input (keyboard, mouse, joystick, touchscreen) drivers, the rest

STALE4643d

3 messages, 2 authors, 2014-01-04 · open the first message on its own page

[PATCH] Input: ims-pcu - fix error unwinding path in application mode

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2014-01-04 05:01:06

We first create backlight and then input devices so we shoudl destroy them
in opposite order when handling errors.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---

Andrey, this fixup was split off from the patch I sent earlier today.

 drivers/input/misc/ims-pcu.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/input/misc/ims-pcu.c b/drivers/input/misc/ims-pcu.c
index 4244f47..513ecdf 100644
--- a/drivers/input/misc/ims-pcu.c
+++ b/drivers/input/misc/ims-pcu.c
@@ -1929,10 +1929,10 @@ static int ims_pcu_init_application_mode(struct ims_pcu *pcu)
 
 	return 0;
 
-err_destroy_backlight:
-	ims_pcu_destroy_backlight(pcu);
 err_destroy_buttons:
 	ims_pcu_destroy_buttons(pcu);
+err_destroy_backlight:
+	ims_pcu_destroy_backlight(pcu);
 	return error;
 }
 
-- 
1.8.4.2


-- 
Dmitry

Re: [PATCH] Input: ims-pcu - fix error unwinding path in application mode

From: Andrey Smirnov <hidden>
Date: 2014-01-04 05:13:13

Dmitry,

Do you want this patch to go separatly or do you want me to bundle it
with my other changes(probably the first one)?
Also, offtopic, do you want me to backport this changes to 2.6 kernel
that IMS uses or would you do it?

Thanks,

On Fri, Jan 3, 2014 at 9:00 PM, Dmitry Torokhov
[off-list ref] wrote:
quoted hunk
We first create backlight and then input devices so we shoudl destroy them
in opposite order when handling errors.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---

Andrey, this fixup was split off from the patch I sent earlier today.

 drivers/input/misc/ims-pcu.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/input/misc/ims-pcu.c b/drivers/input/misc/ims-pcu.c
index 4244f47..513ecdf 100644
--- a/drivers/input/misc/ims-pcu.c
+++ b/drivers/input/misc/ims-pcu.c
@@ -1929,10 +1929,10 @@ static int ims_pcu_init_application_mode(struct ims_pcu *pcu)

        return 0;

-err_destroy_backlight:
-       ims_pcu_destroy_backlight(pcu);
 err_destroy_buttons:
        ims_pcu_destroy_buttons(pcu);
+err_destroy_backlight:
+       ims_pcu_destroy_backlight(pcu);
        return error;
 }

--
1.8.4.2


--
Dmitry

Re: [PATCH] Input: ims-pcu - fix error unwinding path in application mode

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2014-01-04 05:29:38

On Fri, Jan 03, 2014 at 09:13:10PM -0800, Andrey Smirnov wrote:
Dmitry,

Do you want this patch to go separatly or do you want me to bundle it
with my other changes(probably the first one)?
I'll apply it separately.
Also, offtopic, do you want me to backport this changes to 2.6 kernel
that IMS uses or would you do it?
It should apply as is, no backport needed as far as I can tell.

-- 
Dmitry
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help