All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] au1xmmc.c: use resource_size()
@ 2009-12-14 19:28 ` H Hartley Sweeten
  0 siblings, 0 replies; 6+ messages in thread
From: H Hartley Sweeten @ 2009-12-14 19:28 UTC (permalink / raw)
  To: kernel list, linux-mmc

Use resource_size().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>

---

diff --git a/drivers/mmc/host/au1xmmc.c b/drivers/mmc/host/au1xmmc.c
index d3f5561..2d2119b 100644
--- a/drivers/mmc/host/au1xmmc.c
+++ b/drivers/mmc/host/au1xmmc.c
@@ -963,7 +963,7 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
 		goto out1;
 	}
 
-	host->ioarea = request_mem_region(r->start, r->end - r->start + 1,
+	host->ioarea = request_mem_region(r->start, resource_size(r),
 					   pdev->name);
 	if (!host->ioarea) {
 		dev_err(&pdev->dev, "mmio already in use\n"); 

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] au1xmmc.c: use resource_size()
@ 2009-12-14 19:28 ` H Hartley Sweeten
  0 siblings, 0 replies; 6+ messages in thread
From: H Hartley Sweeten @ 2009-12-14 19:28 UTC (permalink / raw)
  To: kernel list, linux-mmc

Use resource_size().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>

---

diff --git a/drivers/mmc/host/au1xmmc.c b/drivers/mmc/host/au1xmmc.c
index d3f5561..2d2119b 100644
--- a/drivers/mmc/host/au1xmmc.c
+++ b/drivers/mmc/host/au1xmmc.c
@@ -963,7 +963,7 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
 		goto out1;
 	}
 
-	host->ioarea = request_mem_region(r->start, r->end - r->start + 1,
+	host->ioarea = request_mem_region(r->start, resource_size(r),
 					   pdev->name);
 	if (!host->ioarea) {
 		dev_err(&pdev->dev, "mmio already in use\n"); 

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH RESEND] au1xmmc.c: use resource_size()
  2009-12-14 19:28 ` H Hartley Sweeten
  (?)
@ 2010-08-27 19:11 ` Chris Ball
  2010-08-27 19:26   ` Matt Fleming
  -1 siblings, 1 reply; 6+ messages in thread
From: Chris Ball @ 2010-08-27 19:11 UTC (permalink / raw)
  To: H Hartley Sweeten; +Cc: akpm, linux-mmc

From: H Hartley Sweeten <hsweeten@visionengravers.com>
Date: Mon, 14 Dec 2009 14:28:06 -0500

Use resource_size().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
[cjb: rebased patch against Linus]
Signed-off-by: Chris Ball <cjb@laptop.org>
---
 drivers/mmc/host/au1xmmc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/mmc/host/au1xmmc.c b/drivers/mmc/host/au1xmmc.c
index c8da5d3..e14b866 100644
--- a/drivers/mmc/host/au1xmmc.c
+++ b/drivers/mmc/host/au1xmmc.c
@@ -964,7 +964,7 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
 		goto out1;
 	}
 
-	host->ioarea = request_mem_region(r->start, r->end - r->start + 1,
+	host->ioarea = request_mem_region(r->start, resource_size(r),
 					   pdev->name);
 	if (!host->ioarea) {
 		dev_err(&pdev->dev, "mmio already in use\n");
-- 
1.7.0.1

-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] au1xmmc.c: use resource_size()
  2010-08-27 19:11 ` [PATCH RESEND] " Chris Ball
@ 2010-08-27 19:26   ` Matt Fleming
  2010-08-27 22:38     ` Manuel Lauss
  0 siblings, 1 reply; 6+ messages in thread
From: Matt Fleming @ 2010-08-27 19:26 UTC (permalink / raw)
  To: Chris Ball; +Cc: H Hartley Sweeten, akpm, linux-mmc, Manuel Lauss

[Adding maintainer to CC]

On Fri, Aug 27, 2010 at 08:11:34PM +0100, Chris Ball wrote:
> From: H Hartley Sweeten <hsweeten@visionengravers.com>
> Date: Mon, 14 Dec 2009 14:28:06 -0500
> 
> Use resource_size().
> 
> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
> [cjb: rebased patch against Linus]
> Signed-off-by: Chris Ball <cjb@laptop.org>
> ---
>  drivers/mmc/host/au1xmmc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/mmc/host/au1xmmc.c b/drivers/mmc/host/au1xmmc.c
> index c8da5d3..e14b866 100644
> --- a/drivers/mmc/host/au1xmmc.c
> +++ b/drivers/mmc/host/au1xmmc.c
> @@ -964,7 +964,7 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
>  		goto out1;
>  	}
>  
> -	host->ioarea = request_mem_region(r->start, r->end - r->start + 1,
> +	host->ioarea = request_mem_region(r->start, resource_size(r),
>  					   pdev->name);
>  	if (!host->ioarea) {
>  		dev_err(&pdev->dev, "mmio already in use\n");
> -- 
> 1.7.0.1
> 
> -- 
> Chris Ball   <cjb@laptop.org>   <http://printf.net/>
> One Laptop Per Child
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] au1xmmc.c: use resource_size()
  2010-08-27 19:26   ` Matt Fleming
@ 2010-08-27 22:38     ` Manuel Lauss
  2010-08-27 22:43       ` Matt Fleming
  0 siblings, 1 reply; 6+ messages in thread
From: Manuel Lauss @ 2010-08-27 22:38 UTC (permalink / raw)
  To: Matt Fleming; +Cc: Chris Ball, H Hartley Sweeten, akpm, linux-mmc, Manuel Lauss

On Fri, Aug 27, 2010 at 9:26 PM, Matt Fleming <matt@console-pimps.org> wrote:
> [Adding maintainer to CC]
>
> On Fri, Aug 27, 2010 at 08:11:34PM +0100, Chris Ball wrote:
>> From: H Hartley Sweeten <hsweeten@visionengravers.com>
>> Date: Mon, 14 Dec 2009 14:28:06 -0500
>>
>> Use resource_size().
>>
>> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
>> [cjb: rebased patch against Linus]
>> Signed-off-by: Chris Ball <cjb@laptop.org>
>> ---
>>  drivers/mmc/host/au1xmmc.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/mmc/host/au1xmmc.c b/drivers/mmc/host/au1xmmc.c
>> index c8da5d3..e14b866 100644
>> --- a/drivers/mmc/host/au1xmmc.c
>> +++ b/drivers/mmc/host/au1xmmc.c
>> @@ -964,7 +964,7 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
>>               goto out1;
>>       }
>>
>> -     host->ioarea = request_mem_region(r->start, r->end - r->start + 1,
>> +     host->ioarea = request_mem_region(r->start, resource_size(r),
>>                                          pdev->name);
>>       if (!host->ioarea) {
>>               dev_err(&pdev->dev, "mmio already in use\n");

Wasn't that posted ages ago?  I'm fine with it.

Acked-by: Manuel Lauss <manuel.lauss@gmail.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] au1xmmc.c: use resource_size()
  2010-08-27 22:38     ` Manuel Lauss
@ 2010-08-27 22:43       ` Matt Fleming
  0 siblings, 0 replies; 6+ messages in thread
From: Matt Fleming @ 2010-08-27 22:43 UTC (permalink / raw)
  To: Manuel Lauss; +Cc: Chris Ball, H Hartley Sweeten, akpm, linux-mmc, Manuel Lauss

On Sat, Aug 28, 2010 at 12:38:56AM +0200, Manuel Lauss wrote:
>
> Wasn't that posted ages ago?  I'm fine with it.

Yep, but it never got merged.

Thanks for the ACK.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-08-27 22:43 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-12-14 19:28 [PATCH] au1xmmc.c: use resource_size() H Hartley Sweeten
2009-12-14 19:28 ` H Hartley Sweeten
2010-08-27 19:11 ` [PATCH RESEND] " Chris Ball
2010-08-27 19:26   ` Matt Fleming
2010-08-27 22:38     ` Manuel Lauss
2010-08-27 22:43       ` Matt Fleming

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.