hwmon: (corsair-cpro) Use complete_all() instead of complete() in ccp_raw_event()
authorAleksa Savic <savicaleksa83@gmail.com>
Sat, 4 May 2024 09:25:02 +0000 (11:25 +0200)
committerGuenter Roeck <linux@roeck-us.net>
Sat, 4 May 2024 13:36:41 +0000 (06:36 -0700)
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>
drivers/hwmon/corsair-cpro.c

index 8d85f66f81435560cd0de8c0499571f174c526e4..6ab4d2478b1fd6f1533c39a3170ecf26235653bd 100644 (file)
@@ -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;
 }