diff options
| author | Aleksa Savic <savicaleksa83@gmail.com> | 2024-05-04 11:25:02 +0200 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2024-05-04 06:36:41 -0700 |
| commit | 3a034a7b0715eb51124a5263890b1ed39978ed3a (patch) | |
| tree | 80950b6bea0caf41e2ac0d63659947994febfc7a /drivers | |
| parent | e0cd85dc666cb08e1bd313d560cb4eff4d04219e (diff) | |
| download | linux-3a034a7b0715eb51124a5263890b1ed39978ed3a.tar.gz | |
hwmon: (corsair-cpro) Use complete_all() instead of complete() in ccp_raw_event()
In ccp_raw_event(), the ccp->wait_input_report completion is
completed once. Since we're waiting for exactly one report in
send_usb_cmd(), use complete_all() instead of complete()
to mark the completion as spent.
Fixes: 40c3a4454225 ("hwmon: add Corsair Commander Pro driver")
Signed-off-by: Aleksa Savic <savicaleksa83@gmail.com>
Acked-by: Marius Zachmann <mail@mariuszachmann.de>
Link: https://lore.kernel.org/r/20240504092504.24158-3-savicaleksa83@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/hwmon/corsair-cpro.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/corsair-cpro.c b/drivers/hwmon/corsair-cpro.c index 8d85f66f814355..6ab4d2478b1fd6 100644 --- a/drivers/hwmon/corsair-cpro.c +++ b/drivers/hwmon/corsair-cpro.c @@ -140,7 +140,7 @@ static int ccp_raw_event(struct hid_device *hdev, struct hid_report *report, u8 return 0; memcpy(ccp->buffer, data, min(IN_BUFFER_SIZE, size)); - complete(&ccp->wait_input_report); + complete_all(&ccp->wait_input_report); return 0; } |