Re: [PATCH 5/6] Input: elantech - clean up elantech_init

From: Wanlong Gao
Date: Thu Aug 18 2011 - 03:59:59 EST


On 08/18/2011 09:57 AM, JJ Ding wrote:
if (elantech_set_absolute_mode(psmouse)) {
pr_err("failed to put touchpad into absolute mode.\n");
goto init_fail;
Hi JJ,
--
diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c
index e13a719..675c236 100644
--- a/drivers/input/mouse/elantech.c
+++ b/drivers/input/mouse/elantech.c
@@ -1069,7 +1069,7 @@ int elantech_init(struct psmouse *psmouse)

return 0;

- init_fail:
+init_fail:
kfree(etd);
return -1;
}
--

also has space before *init_fail*, you may do this in your next version.

Thanks
-Wanlong Gao
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/