From: Chris Ball <cjb@laptop.org>
To: Alexandru Gheorghiu <gheorghiuandru@gmail.com>
Cc: Tony Prisk <linux@prisktech.co.nz>, linux-mmc@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] drivers: mmc: host: Use resource_size()
References: <1363043094-11352-1-git-send-email-gheorghiuandru@gmail.com>
Date: Fri, 22 Mar 2013 12:39:06 -0400
In-Reply-To: <1363043094-11352-1-git-send-email-gheorghiuandru@gmail.com>
	(Alexandru Gheorghiu's message of "Tue, 12 Mar 2013 01:04:54 +0200")
Message-ID: <87hak3o0d1.fsf@octavius.laptop.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1182
Lines: 35

Hi,

On Mon, Mar 11 2013, Alexandru Gheorghiu wrote:
> Used resource_size function instead of explicit computation.
> Patch found using coccinelle.
>
> Signed-off-by: Alexandru Gheorghiu <gheorghiuandru@gmail.com>
> ---
>  drivers/mmc/host/wmt-sdmmc.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
> index 154f0e8..541e097 100644
> --- a/drivers/mmc/host/wmt-sdmmc.c
> +++ b/drivers/mmc/host/wmt-sdmmc.c
> @@ -925,7 +925,7 @@ static int wmt_mci_remove(struct platform_device *pdev)
>  	clk_put(priv->clk_sdmmc);
>  
>  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	release_mem_region(res->start, res->end - res->start + 1);
> +	release_mem_region(res->start, resource_size(res));
>  
>  	mmc_free_host(mmc);

Thanks, pushed to mmc-next for 3.10.

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/