Commit 4c719601 authored by Sergey Shtylyov's avatar Sergey Shtylyov Committed by Greg Kroah-Hartman
Browse files

usb: gadget: udc: bcm63xx: propagate errors from platform_get_irq() 



The driver overrides the error codes returned by platform_get_irq() to
-ENXIO for some strange reason.  Switch to propagating the error codes
upstream.

Acked-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarSergey Shtylyov <s.shtylyov@omp.ru>
Link: https://lore.kernel.org/r/20211214204247.7172-2-s.shtylyov@omp.ru


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 1646566b

drivers/usb/gadget/udc/bcm63xx_udc.c

+6 −2
Original line number Diff line number Diff line
@@ -2321,8 +2321,10 @@ static int bcm63xx_udc_probe(struct platform_device *pdev) 


	/* IRQ resource #0: control interrupt (VBUS, speed, etc.) */

	irq = platform_get_irq(pdev, 0);

	if (irq < 0)

	if (irq < 0) {

		rc = irq;

		goto out_uninit;

	}

	if (devm_request_irq(dev, irq, &bcm63xx_udc_ctrl_isr, 0,

			     dev_name(dev), udc) < 0)

		goto report_request_failure;
@@ -2330,8 +2332,10 @@ static int bcm63xx_udc_probe(struct platform_device *pdev) 
	/* IRQ resources #1-6: data interrupts for IUDMA channels 0-5 */

	for (i = 0; i < BCM63XX_NUM_IUDMA; i++) {

		irq = platform_get_irq(pdev, i + 1);

		if (irq < 0)

		if (irq < 0) {

			rc = irq;

			goto out_uninit;

		}

		if (devm_request_irq(dev, irq, &bcm63xx_udc_data_isr, 0,

				     dev_name(dev), &udc->iudma[i]) < 0)

			goto report_request_failure;