Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756806Ab3CSGOe (ORCPT ); Tue, 19 Mar 2013 02:14:34 -0400 Received: from mail-wg0-f43.google.com ([74.125.82.43]:56292 "EHLO mail-wg0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753443Ab3CSGOd (ORCPT ); Tue, 19 Mar 2013 02:14:33 -0400 MIME-Version: 1.0 In-Reply-To: <1363633580-2920-1-git-send-email-silviupopescu1990@gmail.com> References: <1363633580-2920-1-git-send-email-silviupopescu1990@gmail.com> From: Prabhakar Lad Date: Tue, 19 Mar 2013 11:44:11 +0530 Message-ID: Subject: Re: [PATCH v2] drivers: staging: davinci_vpfe: use resource_size() To: Silviu-Mihai Popescu Cc: linux-media@vger.kernel.org, mchehab@redhat.com, gregkh@linuxfoundation.org, prabhakar.lad@ti.com, sakari.ailus@iki.fi, laurent.pinchart@ideasonboard.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, dlos Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3476 Lines: 77 Hi Wei, Thanks for the patch! I'll queue it up for v3.10 On Tue, Mar 19, 2013 at 12:36 AM, Silviu-Mihai Popescu wrote: > This uses the resource_size() function instead of explicit computation. > > Signed-off-by: Silviu-Mihai Popescu Acked-by: Lad, Prabhakar Regards, --Prabhakar Lad > --- > drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 2 +- > drivers/staging/media/davinci_vpfe/dm365_isif.c | 4 ++-- > drivers/staging/media/davinci_vpfe/dm365_resizer.c | 2 +- > 3 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/media/davinci_vpfe/dm365_ipipe.c b/drivers/staging/media/davinci_vpfe/dm365_ipipe.c > index 9285353..05673ed 100644 > --- a/drivers/staging/media/davinci_vpfe/dm365_ipipe.c > +++ b/drivers/staging/media/davinci_vpfe/dm365_ipipe.c > @@ -1859,5 +1859,5 @@ void vpfe_ipipe_cleanup(struct vpfe_ipipe_device *ipipe, > iounmap(ipipe->isp5_base_addr); > res = platform_get_resource(pdev, IORESOURCE_MEM, 4); > if (res) > - release_mem_region(res->start, res->end - res->start + 1); > + release_mem_region(res->start, resource_size(res)); > } > diff --git a/drivers/staging/media/davinci_vpfe/dm365_isif.c b/drivers/staging/media/davinci_vpfe/dm365_isif.c > index ebeea72..c4a5660 100644 > --- a/drivers/staging/media/davinci_vpfe/dm365_isif.c > +++ b/drivers/staging/media/davinci_vpfe/dm365_isif.c > @@ -1953,7 +1953,7 @@ static void isif_remove(struct vpfe_isif_device *isif, > res = platform_get_resource(pdev, IORESOURCE_MEM, i); > if (res) > release_mem_region(res->start, > - res->end - res->start + 1); > + resource_size(res)); > i++; > } > } > @@ -2003,7 +2003,7 @@ int vpfe_isif_init(struct vpfe_isif_device *isif, struct platform_device *pdev) > status = -ENOENT; > goto fail_nobase_res; > } > - res_len = res->end - res->start + 1; > + res_len = resource_size(res); > res = request_mem_region(res->start, res_len, res->name); > if (!res) { > status = -EBUSY; > diff --git a/drivers/staging/media/davinci_vpfe/dm365_resizer.c b/drivers/staging/media/davinci_vpfe/dm365_resizer.c > index 9cb0262..126f84c 100644 > --- a/drivers/staging/media/davinci_vpfe/dm365_resizer.c > +++ b/drivers/staging/media/davinci_vpfe/dm365_resizer.c > @@ -1995,5 +1995,5 @@ vpfe_resizer_cleanup(struct vpfe_resizer_device *vpfe_rsz, > res = platform_get_resource(pdev, IORESOURCE_MEM, 5); > if (res) > release_mem_region(res->start, > - res->end - res->start + 1); > + resource_size(res)); > } > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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/