diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-12-17 11:58:18 -0500 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2013-12-18 11:47:39 -0500 |
commit | 3c4396b434613f653bf95f741a6000e671d1e011 (patch) | |
tree | b719ce87281e6e002b137f018701e63e93a9cf00 /drivers/input/touchscreen | |
parent | c15bdfd5b9831e4cab8cfc118243956e267dd30e (diff) |
Input: zforce - fix error return code in zforce_start()
The error code was not set if unable to set config, so the error
condition wasn't reflected in the return value. Fix to return a
negative error code from the error handling case instead of 0.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Heiko Stuebner <heiko@sntech.de>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'drivers/input/touchscreen')
-rw-r--r-- | drivers/input/touchscreen/zforce_ts.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/input/touchscreen/zforce_ts.c b/drivers/input/touchscreen/zforce_ts.c index 75762d6ff3ba..4ffe4cc3b234 100644 --- a/drivers/input/touchscreen/zforce_ts.c +++ b/drivers/input/touchscreen/zforce_ts.c | |||
@@ -279,7 +279,8 @@ static int zforce_start(struct zforce_ts *ts) | |||
279 | goto error; | 279 | goto error; |
280 | } | 280 | } |
281 | 281 | ||
282 | if (zforce_setconfig(ts, SETCONFIG_DUALTOUCH)) { | 282 | ret = zforce_setconfig(ts, SETCONFIG_DUALTOUCH); |
283 | if (ret) { | ||
283 | dev_err(&client->dev, "Unable to set config\n"); | 284 | dev_err(&client->dev, "Unable to set config\n"); |
284 | goto error; | 285 | goto error; |
285 | } | 286 | } |