commit 696c5c04 (USB: Palmas OTG Transceiver Driver) introduced empty
palmas_vbus_irq/palmas_id_irq with only *palmas_usb* declaration.
Removed these unnecessary functions.

Signed-off-by: Kishon Vijay Abraham I <kis...@ti.com>
---
 drivers/usb/otg/palmas-usb.c |   38 --------------------------------------
 1 file changed, 38 deletions(-)

diff --git a/drivers/usb/otg/palmas-usb.c b/drivers/usb/otg/palmas-usb.c
index f063350..be9d81c 100644
--- a/drivers/usb/otg/palmas-usb.c
+++ b/drivers/usb/otg/palmas-usb.c
@@ -103,15 +103,6 @@ static ssize_t palmas_usb_vbus_show(struct device *dev,
 }
 static DEVICE_ATTR(vbus, 0444, palmas_usb_vbus_show, NULL);
 
-static irqreturn_t palmas_vbus_irq(int irq, void *_palmas_usb)
-{
-       struct palmas_usb *palmas_usb = _palmas_usb;
-
-       /* TODO: Do we need to do any work here? */
-
-       return IRQ_HANDLED;
-}
-
 static irqreturn_t palmas_vbus_wakeup_irq(int irq, void *_palmas_usb)
 {
        struct palmas_usb *palmas_usb = _palmas_usb;
@@ -138,15 +129,6 @@ static irqreturn_t palmas_vbus_wakeup_irq(int irq, void 
*_palmas_usb)
        return IRQ_HANDLED;
 }
 
-static irqreturn_t palmas_id_irq(int irq, void *_palmas_usb)
-{
-       struct palmas_usb *palmas_usb = _palmas_usb;
-
-       /* TODO: Do we need to do any work here? */
-
-       return IRQ_HANDLED;
-}
-
 static irqreturn_t palmas_id_wakeup_irq(int irq, void *_palmas_usb)
 {
        int                     status = 0;
@@ -329,16 +311,6 @@ static int palmas_usb_probe(struct platform_device *pdev)
 
        INIT_WORK(&palmas_usb->set_vbus_work, palmas_set_vbus_work);
 
-       status = devm_request_threaded_irq(palmas_usb->dev, palmas_usb->irq1,
-                               NULL, palmas_id_irq,
-                               IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING,
-                               "palmas_usb", palmas_usb);
-       if (status < 0) {
-               dev_err(&pdev->dev, "can't get IRQ %d, err %d\n",
-                                       palmas_usb->irq1, status);
-               goto fail_irq;
-       }
-
        status = devm_request_threaded_irq(palmas_usb->dev, palmas_usb->irq2,
                        NULL, palmas_id_wakeup_irq,
                        IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING,
@@ -349,16 +321,6 @@ static int palmas_usb_probe(struct platform_device *pdev)
                goto fail_irq;
        }
 
-       status = devm_request_threaded_irq(palmas_usb->dev, palmas_usb->irq3,
-                       NULL, palmas_vbus_irq,
-                       IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING,
-                       "palmas_usb", palmas_usb);
-       if (status < 0) {
-               dev_err(&pdev->dev, "can't get IRQ %d, err %d\n",
-                                       palmas_usb->irq3, status);
-               goto fail_irq;
-       }
-
        status = devm_request_threaded_irq(palmas_usb->dev, palmas_usb->irq4,
                        NULL, palmas_vbus_wakeup_irq,
                        IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING,
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to