From: Jani Nikula <[email protected]>

Separate gpio_request() and request_irq() error handling.

Signed-off-by: Jani Nikula <[email protected]>
---
 drivers/video/omap2/displays/panel-taal.c |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/video/omap2/displays/panel-taal.c 
b/drivers/video/omap2/displays/panel-taal.c
index 3729ec2..98f882a 100644
--- a/drivers/video/omap2/displays/panel-taal.c
+++ b/drivers/video/omap2/displays/panel-taal.c
@@ -601,7 +601,7 @@ static int taal_probe(struct omap_dss_device *dssdev)
                if (r) {
                        dev_err(&dssdev->dev, "IRQ request failed\n");
                        gpio_free(gpio);
-                       goto err3;
+                       goto err4;
                }
 
                init_completion(&td->te_completion);
@@ -612,18 +612,18 @@ static int taal_probe(struct omap_dss_device *dssdev)
        r = sysfs_create_group(&dssdev->dev.kobj, &taal_attr_group);
        if (r) {
                dev_err(&dssdev->dev, "failed to create sysfs files\n");
-               goto err4;
+               goto err5;
        }
 
        taal_hw_reset(dssdev);
 
        return 0;
+err5:
+       if (td->use_ext_te)
+               free_irq(gpio_to_irq(dssdev->phy.dsi.ext_te_gpio), dssdev);
 err4:
-       if (td->use_ext_te) {
-               int gpio = dssdev->phy.dsi.ext_te_gpio;
-               free_irq(gpio_to_irq(gpio), dssdev);
-               gpio_free(gpio);
-       }
+       if (td->use_ext_te)
+               gpio_free(dssdev->phy.dsi.ext_te_gpio);
 err3:
        backlight_device_unregister(bldev);
 err2:
-- 
1.6.5.2

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to