]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
power_supply: 88pm860x_charger: Do not call free_irq() twice
authorTakeshi Yoshimura <yos@sslab.ics.keio.ac.jp>
Fri, 29 May 2015 13:16:43 +0000 (13:16 +0000)
committerSebastian Reichel <sre@kernel.org>
Fri, 29 May 2015 14:06:10 +0000 (16:06 +0200)
My static checker detected double free_irq() in pm860x_charger_remove().
Unloading this module always causes a warning. This patch removes the
first redundant free_irq() call.

Signed-off-by: Takeshi Yoshimura <yos@sslab.ics.keio.ac.jp>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
drivers/power/88pm860x_charger.c

index 0e448c68c02bb9e4e6cba0a428e59b29e5231e68..297e72dc70e603aede8caa31d6f8d3cec22e8a61 100644 (file)
@@ -742,7 +742,6 @@ static int pm860x_charger_remove(struct platform_device *pdev)
        int i;
 
        power_supply_unregister(info->usb);
-       free_irq(info->irq[0], info);
        for (i = 0; i < info->irq_nums; i++)
                free_irq(info->irq[i], info);
        return 0;