Re: [PATCH] mmci-omap: free irq resource

[Andrew Morton <akpm@linux-foundation.org>]

On Sun, 1 Nov 2009 12:59:05 +0100
Ladislav.Michl@seznam.cz wrote:

> Free IRQ on remove.
> 
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Tony Lindgren <tony@atomide.com>
> 
> diff --git a/drivers/mmc/host/omap.c b/drivers/mmc/host/omap.c
> index 5d773b8..5f970e2 100644
> --- a/drivers/mmc/host/omap.c
> +++ b/drivers/mmc/host/omap.c
> @@ -1529,6 +1529,7 @@ static int mmc_omap_remove(struct platform_device *pdev)
>  		host->pdata->cleanup(&pdev->dev);
>  
>  	mmc_omap_fclk_enable(host, 0);
> +	free_irq(host->irq, host);
>  	clk_put(host->fclk);
>  	clk_disable(host->iclk);
>  	clk_put(host->iclk);

This is a poor changelog.

The reader doesn't know what the implication of the bug is.  I _assume_
that it means that the driver can only be loaded a single time.  That
on a second modprobe, it fails to allocate the interrupt and fails.

In which case this is a fairly serious bug and perhaps the patch should
be backported into the -stable tree.

Or I could be all wrong about all of that.  This is why it's better if
the patch submitter *explains* these things, rather than leaving others
to guess.