hwmon: (corsair-psu) Close HID device on probe errors

corsairpsu_probe() opens the HID device before sending the device init
and firmware-info commands. If either command fails, the error path jumps
directly to fail_and_stop and skips hid_hw_close().

Use the existing fail_and_close label for those post-open failures so the
open count and low-level close callback are balanced before hid_hw_stop().

Fixes: d115b51e0e56 ("hwmon: add Corsair PSU HID controller driver")
Cc: stable@vger.kernel.org
Signed-off-by: Myeonghun Pak <mhun512@gmail.com>
Reviewed-by: Wilken Gottwalt <wilken.gottwalt@posteo.net>
Link: https://lore.kernel.org/r/20260424135107.13720-1-mhun512@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>

diff --git a/drivers/hwmon/corsair-psu.c b/drivers/hwmon/corsair-psu.c
index dddbd2463f8da70e2fe4292d5aeb41e9c51b7c84..76f3e1da68d09e94811b62eb9887fcd21ee2cd5a 100644 (file)
--- a/drivers/hwmon/corsair-psu.c
+++ b/drivers/hwmon/corsair-psu.c
@@ -796,13 +796,13 @@ static int corsairpsu_probe(struct hid_device *hdev, const struct hid_device_id
        ret = corsairpsu_init(priv);
        if (ret < 0) {
                dev_err(&hdev->dev, "unable to initialize device (%d)\n", ret);
-               goto fail_and_stop;
+               goto fail_and_close;
        }
 
        ret = corsairpsu_fwinfo(priv);
        if (ret < 0) {
                dev_err(&hdev->dev, "unable to query firmware (%d)\n", ret);
-               goto fail_and_stop;
+               goto fail_and_close;
        }
 
        corsairpsu_get_criticals(priv);