diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2019-12-26 15:54:35 +0100 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2020-01-13 13:26:43 +0100 |
| commit | 8f2828d737241be558b9d31c6c0465595a014341 (patch) | |
| tree | 757f635671a0e6caa619b97f89b594d12b4fd9ad /drivers/hid | |
| parent | be281368f29797cf4b318ad890673ce96bb9251e (diff) | |
| download | linux-8f2828d737241be558b9d31c6c0465595a014341.tar.bz2 | |
HID: logitech-hidpp: avoid duplicate error handling code in 'hidpp_probe()'
'hid_hw_stop()' is already in the error handling path when branching to
the 'hid_hw_open_fail' label.
There is no point in calling it twice, so remove one.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid')
| -rw-r--r-- | drivers/hid/hid-logitech-hidpp.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c index bb063e7d48df..70e1cb928bf0 100644 --- a/drivers/hid/hid-logitech-hidpp.c +++ b/drivers/hid/hid-logitech-hidpp.c @@ -3817,7 +3817,6 @@ static int hidpp_probe(struct hid_device *hdev, const struct hid_device_id *id) if (ret < 0) { dev_err(&hdev->dev, "%s:hid_hw_open returned error:%d\n", __func__, ret); - hid_hw_stop(hdev); goto hid_hw_open_fail; } |