usb: core: Don't force USB generic_subclass drivers to define probe()
authorDouglas Anderson <dianders@chromium.org>
Fri, 1 Dec 2023 18:29:50 +0000 (10:29 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 4 Dec 2023 13:28:19 +0000 (22:28 +0900)
There's no real reason that subclassed USB drivers _need_ to define
probe() since they might want to subclass for some other reason. Make
it optional to define probe() if we're a generic_subclass.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Grant Grundler <grundler@chromium.org>
Reviewed-by: Alan Stern <stern@rowland.harvard.edu>
Link: https://lore.kernel.org/r/20231201102946.v2.1.I7ea0dd55ee2acdb48b0e6d28c1a704ab2c29206f@changeid
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/core/driver.c

index f58a0299fb3bd30c6cdcc7dc9c47b24f6be8d680..1dc0c041304383c7c639f03909dc3a0142d89b66 100644 (file)
@@ -290,7 +290,10 @@ static int usb_probe_device(struct device *dev)
         * specialised device drivers prior to setting the
         * use_generic_driver bit.
         */
-       error = udriver->probe(udev);
+       if (udriver->probe)
+               error = udriver->probe(udev);
+       else if (!udriver->generic_subclass)
+               error = -EINVAL;
        if (error == -ENODEV && udriver != &usb_generic_driver &&
            (udriver->id_table || udriver->match)) {
                udev->use_generic_driver = 1;