usb: udc: pxa: fix error handling

As Dan Carpenter points out, my recent change makes subtle
changes to the error handling that were not intended.

Move the warning print up so it does not get skipped in
case of an error, but handle -EPROBE_DEFER properly now.

Change the devm_gpiod_get() to the _optional variant, which
is in line with the intended behavior and the DT binding,
though this did not work previously.

Reported-by: Dan Carpenter <error27@gmail.com>
Link: https://lore.kernel.org/linux-usb/ag6-xhfFjb5NpXQz@stanley.mountain/
Fixes: 25bd55f46032 ("usb: udc: pxa: remove unused platform_data")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://patch.msgid.link/20260526104810.3906090-1-arnd@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/gadget/udc/pxa27x_udc.c b/drivers/usb/gadget/udc/pxa27x_udc.c
index 640f81988c041de41a0fd9c8483bbbdc156faeb0..df5cca43afbefb8d2ba0e1b6e4ba10a5d3b1d974 100644 (file)
--- a/drivers/usb/gadget/udc/pxa27x_udc.c
+++ b/drivers/usb/gadget/udc/pxa27x_udc.c
@@ -2374,9 +2374,10 @@ static int pxa_udc_probe(struct platform_device *pdev)
        struct pxa_udc *udc = &memory;
        int retval = 0;
 
-       udc->gpiod = devm_gpiod_get(&pdev->dev, NULL, GPIOD_ASIS);
+       udc->gpiod = devm_gpiod_get_optional(&pdev->dev, NULL, GPIOD_ASIS);
        if (IS_ERR(udc->gpiod))
-               return PTR_ERR(udc->gpiod);
+               return dev_err_probe(&pdev->dev, PTR_ERR(udc->gpiod),
+                                    "Couldn't find or request D+ gpio\n");
 
        udc->regs = devm_platform_ioremap_resource(pdev, 0);
        if (IS_ERR(udc->regs))
@@ -2395,11 +2396,6 @@ static int pxa_udc_probe(struct platform_device *pdev)
                udc->transceiver = usb_get_phy(USB_PHY_TYPE_USB2);
        }
 
-       if (IS_ERR(udc->gpiod)) {
-               dev_err(&pdev->dev, "Couldn't find or request D+ gpio : %ld\n",
-                       PTR_ERR(udc->gpiod));
-               return PTR_ERR(udc->gpiod);
-       }
        if (udc->gpiod)
                gpiod_direction_output(udc->gpiod, 0);