* Re: [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-08 10:12 [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h Uwe Kleine-König
@ 2009-10-08 11:26 ` Linus Walleij
2009-10-08 12:14 ` Nicolas Ferre
` (2 subsequent siblings)
3 siblings, 0 replies; 11+ messages in thread
From: Linus Walleij @ 2009-10-08 11:26 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: linux-mmc, Catalin Marinas, Linus Walleij, Russell King,
Pierre Ossman, linux-arm-kernel
2009/10/8 Uwe Kleine-König <u.kleine-koenig@pengutronix.de>:
> This fixes a compile bug introduced in
>
> 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
>
> That commit moved arch/arm/include/asm/mach/mmc.h to
> include/linux/amba/mmci.h. Just removing the include was enough.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: Linus Walleij <linus.walleij@stericsson.com>
Acked-by: Linus Walleij <linus.walleij@stericsson.com>
The include paths in the kernel lead me astray... Weird that this
driver included that file at all though, but I guess the driver was derived
from MMCI at some early staget and it got left in?
Linus Walleij
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-08 10:12 [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h Uwe Kleine-König
2009-10-08 11:26 ` Linus Walleij
@ 2009-10-08 12:14 ` Nicolas Ferre
2009-10-08 12:18 ` Bill Gatliff
2009-10-13 7:19 ` Uwe Kleine-König
3 siblings, 0 replies; 11+ messages in thread
From: Nicolas Ferre @ 2009-10-08 12:14 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Linus Walleij, Catalin Marinas, linux-mmc, Pierre Ossman,
Russell King, linux-arm-kernel
Uwe Kleine-König :
> This fixes a compile bug introduced in
>
> 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
>
> That commit moved arch/arm/include/asm/mach/mmc.h to
> include/linux/amba/mmci.h. Just removing the include was enough.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: Linus Walleij <linus.walleij@stericsson.com>
> Cc: Russell King <rmk+kernel@arm.linux.org.uk>
> Cc: Pierre Ossman <drzeus@drzeus.cx>
> Cc: linux-arm-kernel@lists.infradead.org
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> drivers/mmc/host/at91_mci.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/host/at91_mci.c b/drivers/mmc/host/at91_mci.c
> index e556d42..63924e0 100644
> --- a/drivers/mmc/host/at91_mci.c
> +++ b/drivers/mmc/host/at91_mci.c
> @@ -72,7 +72,6 @@
> #include <asm/irq.h>
> #include <asm/gpio.h>
>
> -#include <asm/mach/mmc.h>
> #include <mach/board.h>
> #include <mach/cpu.h>
> #include <mach/at91_mci.h>
--
Nicolas Ferre
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-08 10:12 [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h Uwe Kleine-König
2009-10-08 11:26 ` Linus Walleij
2009-10-08 12:14 ` Nicolas Ferre
@ 2009-10-08 12:18 ` Bill Gatliff
2009-10-13 7:19 ` Uwe Kleine-König
3 siblings, 0 replies; 11+ messages in thread
From: Bill Gatliff @ 2009-10-08 12:18 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Linus Walleij, Catalin Marinas, linux-mmc, Pierre Ossman,
Russell King, linux-arm-kernel
Uwe Kleine-König wrote:
> This fixes a compile bug introduced in
>
> 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
>
> That commit moved arch/arm/include/asm/mach/mmc.h to
> include/linux/amba/mmci.h. Just removing the include was enough.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: Linus Walleij <linus.walleij@stericsson.com>
> Cc: Russell King <rmk+kernel@arm.linux.org.uk>
> Cc: Pierre Ossman <drzeus@drzeus.cx>
> Cc: linux-arm-kernel@lists.infradead.org
> ---
> drivers/mmc/host/at91_mci.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/host/at91_mci.c b/drivers/mmc/host/at91_mci.c
> index e556d42..63924e0 100644
> --- a/drivers/mmc/host/at91_mci.c
> +++ b/drivers/mmc/host/at91_mci.c
> @@ -72,7 +72,6 @@
> #include <asm/irq.h>
> #include <asm/gpio.h>
>
> -#include <asm/mach/mmc.h>
> #include <mach/board.h>
> #include <mach/cpu.h>
> #include <mach/at91_mci.h>
>
Acked-by: Bill Gatliff <bgat@billgatliff.com>
b.g.
--
Bill Gatliff
bgat@billgatliff.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-08 10:12 [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h Uwe Kleine-König
` (2 preceding siblings ...)
2009-10-08 12:18 ` Bill Gatliff
@ 2009-10-13 7:19 ` Uwe Kleine-König
2009-10-13 7:36 ` Nicolas Ferre
2009-10-20 22:37 ` [PATCH] " Linus Walleij
3 siblings, 2 replies; 11+ messages in thread
From: Uwe Kleine-König @ 2009-10-13 7:19 UTC (permalink / raw)
To: linux-mmc
Cc: Catalin Marinas, Andrew Morton, Pierre Ossman, linux-arm-kernel,
Russell King
This fixes a compile error introduced in
6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
That commit moved arch/arm/include/asm/mach/mmc.h to
include/linux/amba/mmci.h. Just removing the include was enough.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Linus Walleij <linus.walleij@stericsson.com>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
Acked-by: Bill Gatliff <bgat@billgatliff.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Cc: Pierre Ossman <drzeus@drzeus.cx>
Cc: linux-arm-kernel@lists.infradead.org
Cc: Andrew Morton <akpm@linux-foundation.org>
---
Hello,
what is the right path to Linus for this patch?
Best regards
Uwe
drivers/mmc/host/at91_mci.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/host/at91_mci.c b/drivers/mmc/host/at91_mci.c
index e556d42..63924e0 100644
--- a/drivers/mmc/host/at91_mci.c
+++ b/drivers/mmc/host/at91_mci.c
@@ -72,7 +72,6 @@
#include <asm/irq.h>
#include <asm/gpio.h>
-#include <asm/mach/mmc.h>
#include <mach/board.h>
#include <mach/cpu.h>
#include <mach/at91_mci.h>
--
1.6.4.3
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-13 7:19 ` Uwe Kleine-König
@ 2009-10-13 7:36 ` Nicolas Ferre
2009-10-20 17:27 ` Bill Gatliff
2009-10-20 22:37 ` [PATCH] " Linus Walleij
1 sibling, 1 reply; 11+ messages in thread
From: Nicolas Ferre @ 2009-10-13 7:36 UTC (permalink / raw)
To: Uwe Kleine-König, linux-mmc, Andrew Morton
Cc: Catalin Marinas, Russell King, Pierre Ossman, linux-arm-kernel
Uwe Kleine-König :
> This fixes a compile error introduced in
>
> 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
>
> That commit moved arch/arm/include/asm/mach/mmc.h to
> include/linux/amba/mmci.h. Just removing the include was enough.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> Acked-by: Linus Walleij <linus.walleij@stericsson.com>
> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> Acked-by: Bill Gatliff <bgat@billgatliff.com>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: Russell King <rmk+kernel@arm.linux.org.uk>
> Cc: Pierre Ossman <drzeus@drzeus.cx>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: Andrew Morton <akpm@linux-foundation.org>
> ---
> Hello,
>
> what is the right path to Linus for this patch?
Uwe,
I gess that Andrew Morton will have to handle it.
Let's help him not forget it ;-)
Bye,
> drivers/mmc/host/at91_mci.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/host/at91_mci.c b/drivers/mmc/host/at91_mci.c
> index e556d42..63924e0 100644
> --- a/drivers/mmc/host/at91_mci.c
> +++ b/drivers/mmc/host/at91_mci.c
> @@ -72,7 +72,6 @@
> #include <asm/irq.h>
> #include <asm/gpio.h>
>
> -#include <asm/mach/mmc.h>
> #include <mach/board.h>
> #include <mach/cpu.h>
> #include <mach/at91_mci.h>
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-13 7:36 ` Nicolas Ferre
@ 2009-10-20 17:27 ` Bill Gatliff
2009-10-21 7:46 ` [PATCH RESEND#2] " Uwe Kleine-König
0 siblings, 1 reply; 11+ messages in thread
From: Bill Gatliff @ 2009-10-20 17:27 UTC (permalink / raw)
To: Nicolas Ferre
Cc: Uwe Kleine-König, linux-mmc, Andrew Morton, Catalin Marinas,
Pierre Ossman, linux-arm-kernel, Russell King
Nicolas Ferre wrote:
> Uwe Kleine-König :
>
>> This fixes a compile error introduced in
>>
>> 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
>>
>> That commit moved arch/arm/include/asm/mach/mmc.h to
>> include/linux/amba/mmci.h. Just removing the include was enough.
>>
>> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
>> Acked-by: Linus Walleij <linus.walleij@stericsson.com>
>> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
>> Acked-by: Bill Gatliff <bgat@billgatliff.com>
>> Cc: Catalin Marinas <catalin.marinas@arm.com>
>> Cc: Russell King <rmk+kernel@arm.linux.org.uk>
>> Cc: Pierre Ossman <drzeus@drzeus.cx>
>> Cc: linux-arm-kernel@lists.infradead.org
>> Cc: Andrew Morton <akpm@linux-foundation.org>
>> ---
>> Hello,
>>
>> what is the right path to Linus for this patch?
>>
>
> Uwe,
>
> I gess that Andrew Morton will have to handle it.
>
> Let's help him not forget it ;-)
>
Indeed! This one _really_ annoys me.
b.g.
--
Bill Gatliff
bgat@billgatliff.com
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH RESEND#2] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-20 17:27 ` Bill Gatliff
@ 2009-10-21 7:46 ` Uwe Kleine-König
2009-10-21 7:53 ` Nicolas Ferre
0 siblings, 1 reply; 11+ messages in thread
From: Uwe Kleine-König @ 2009-10-21 7:46 UTC (permalink / raw)
To: linux-mmc
Cc: Catalin Marinas, Russell King, Pierre Ossman, linux-arm-kernel,
Andrew Morton, Linus Torvalds
This fixes a compile bug introduced in
6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
That commit moved arch/arm/include/asm/mach/mmc.h to
include/linux/amba/mmci.h. Just removing the include was enough.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Linus Walleij <linus.walleij@stericsson.com>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
Acked-by: Bill Gatliff <bgat@billgatliff.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Cc: Pierre Ossman <drzeus@drzeus.cx>
Cc: linux-arm-kernel@lists.infradead.org
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
---
Hello,
this patch is a build fix that seems to bite quite some people, but
nobody has picked it up until now.
Linus Walleij suggested to send it to Russell's patch tracker, but I
don't think this is the right way for that patch. It should either go
via Pierre or (IMO preferably) via Andrew.
Best regards
Uwe
drivers/mmc/host/at91_mci.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/host/at91_mci.c b/drivers/mmc/host/at91_mci.c
index e556d42..63924e0 100644
--- a/drivers/mmc/host/at91_mci.c
+++ b/drivers/mmc/host/at91_mci.c
@@ -72,7 +72,6 @@
#include <asm/irq.h>
#include <asm/gpio.h>
-#include <asm/mach/mmc.h>
#include <mach/board.h>
#include <mach/cpu.h>
#include <mach/at91_mci.h>
--
1.6.5
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH RESEND#2] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-21 7:46 ` [PATCH RESEND#2] " Uwe Kleine-König
@ 2009-10-21 7:53 ` Nicolas Ferre
2009-10-21 8:12 ` Uwe Kleine-König
0 siblings, 1 reply; 11+ messages in thread
From: Nicolas Ferre @ 2009-10-21 7:53 UTC (permalink / raw)
To: Uwe Kleine-König, linux-mmc
Cc: Pierre Ossman, linux-arm-kernel, Andrew Morton
Uwe Kleine-König :
> This fixes a compile bug introduced in
>
> 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
>
> That commit moved arch/arm/include/asm/mach/mmc.h to
> include/linux/amba/mmci.h. Just removing the include was enough.
FYI it is already in -mm patchsed as:
mmc-at91_mci-dont-include-asm-mach-mmch.patch
We now just have to track its way to mainline.
Bye,
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH RESEND#2] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-21 7:53 ` Nicolas Ferre
@ 2009-10-21 8:12 ` Uwe Kleine-König
0 siblings, 0 replies; 11+ messages in thread
From: Uwe Kleine-König @ 2009-10-21 8:12 UTC (permalink / raw)
To: Nicolas Ferre; +Cc: linux-mmc, Pierre Ossman, linux-arm-kernel, Andrew Morton
On Wed, Oct 21, 2009 at 09:53:47AM +0200, Nicolas Ferre wrote:
> Uwe Kleine-König :
> > This fixes a compile bug introduced in
> >
> > 6ef297f (ARM: 5720/1: Move MMCI header to amba include dir)
> >
> > That commit moved arch/arm/include/asm/mach/mmc.h to
> > include/linux/amba/mmci.h. Just removing the include was enough.
>
> FYI it is already in -mm patchsed as:
> mmc-at91_mci-dont-include-asm-mach-mmch.patch
oh right. I was pretty sure I didn't get a "patch was added to mm" mail
by Andrew, but searching my inbox proofed me wrong.
Sorry
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] mmc: at91_mci: Don't include asm/mach/mmc.h
2009-10-13 7:19 ` Uwe Kleine-König
2009-10-13 7:36 ` Nicolas Ferre
@ 2009-10-20 22:37 ` Linus Walleij
1 sibling, 0 replies; 11+ messages in thread
From: Linus Walleij @ 2009-10-20 22:37 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: linux-mmc, Catalin Marinas, Russell King, Pierre Ossman,
linux-arm-kernel, Andrew Morton
2009/10/13 Uwe Kleine-König <u.kleine-koenig@pengutronix.de>:
> what is the right path to Linus for this patch?
I would simply put it into Russells patch tracker as it's an ARM system.
Linus Walleij
^ permalink raw reply [flat|nested] 11+ messages in thread