2013-03-10 12:15:11

by Alexandru Gheorghiu

Subject: [PATCH] Drivers: staging: media: davinci_vpfe: Use resource_size function

Use resource_size function on resource object instead of explicit
computation.

Signed-off-by: Alexandru Gheorghiu <[email protected]>
---
drivers/staging/media/davinci_vpfe/dm365_ipipeif.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c b/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
index c8cae51..b2f4ef8 100644
--- a/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
+++ b/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
@@ -1065,7 +1065,6 @@ vpfe_ipipeif_cleanup(struct vpfe_ipipeif_device *ipipeif,
iounmap(ipipeif->ipipeif_base_addr);
res = platform_get_resource(pdev, IORESOURCE_MEM, 3);
if (res)
- release_mem_region(res->start,
- res->end - res->start + 1);
+ release_mem_region(res->start, resource_size(res));

}
--
1.7.10.4

2013-03-11 04:44:51

by Prabhakar

Subject: Re: [PATCH] Drivers: staging: media: davinci_vpfe: Use resource_size function

Hi Alexandru,

Thanks for the patch!

On Sun, Mar 10, 2013 at 5:44 PM, Alexandru Gheorghiu
<[email protected]> wrote:
> Use resource_size function on resource object instead of explicit
> computation.
>
> Signed-off-by: Alexandru Gheorghiu <[email protected]>

Acked-by: Lad, Prabhakar <[email protected]>

Regards,
--Prabhakar Lad

> ---
> drivers/staging/media/davinci_vpfe/dm365_ipipeif.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c b/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
> index c8cae51..b2f4ef8 100644
> --- a/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
> +++ b/drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
> @@ -1065,7 +1065,6 @@ vpfe_ipipeif_cleanup(struct vpfe_ipipeif_device *ipipeif,
> iounmap(ipipeif->ipipeif_base_addr);
> res = platform_get_resource(pdev, IORESOURCE_MEM, 3);
> if (res)
> - release_mem_region(res->start,
> - res->end - res->start + 1);
> + release_mem_region(res->start, resource_size(res));
>
> }
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html