diff options
author | Felix Rueegg <felix.rueegg@gmail.com> | 2013-10-08 13:33:47 -0400 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2013-10-30 09:34:28 -0400 |
commit | 556483e2ad4961bd2770e3b65f3a0eb2825d5791 (patch) | |
tree | e5b1531fecafd62e109739aba4671395b2765f08 /drivers | |
parent | d4b1bba76171cb783e32441b28462fe841073ed8 (diff) |
HID: remove self-assignment from hid_input_report
The ternary expression will always result in a self-assignment, which is
pointless.
Signed-off-by: Felix Rueegg <felix.rueegg@gmail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/hid/hid-core.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index b8470b1a10fe..868ebaa39413 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core.c | |||
@@ -1417,10 +1417,8 @@ int hid_input_report(struct hid_device *hid, int type, u8 *data, int size, int i | |||
1417 | 1417 | ||
1418 | if (hdrv && hdrv->raw_event && hid_match_report(hid, report)) { | 1418 | if (hdrv && hdrv->raw_event && hid_match_report(hid, report)) { |
1419 | ret = hdrv->raw_event(hid, report, data, size); | 1419 | ret = hdrv->raw_event(hid, report, data, size); |
1420 | if (ret < 0) { | 1420 | if (ret < 0) |
1421 | ret = ret < 0 ? ret : 0; | ||
1422 | goto unlock; | 1421 | goto unlock; |
1423 | } | ||
1424 | } | 1422 | } |
1425 | 1423 | ||
1426 | ret = hid_report_raw_event(hid, type, data, size, interrupt); | 1424 | ret = hid_report_raw_event(hid, type, data, size, interrupt); |