[PATCH] tty: atmel_serial: include <mach/hardware.h> explicitly

[PATCH] tty: atmel_serial: include <mach/hardware.h> explicitly

[Uwe Kleine-König]

When compiled for at91rm9200 (i.e. ARM) the driver needs among others
the symbol AT91RM9200_BASE_US0 which is defined in
arch/arm/mach-at91/include/mach/at91rm9200.h. This file is included
implicitly via several steps. As ARM's <asm/timex.h> will stop including
<mach/hardware.h> later in this series explicitly include
<mach/hardware.h>

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---

Notes:
    Hello,
    
    I want to send this patch as part of a series dropping <mach/timex.h>
    altogether. So please don't take this patch but only give your Ack if
    you're OK with me sending this patch via arm-soc.
    
    Thanks
    Uwe

 drivers/tty/serial/atmel_serial.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/tty/serial/atmel_serial.c b/drivers/tty/serial/atmel_serial.c
index c7d99af..de51964 100644
--- a/drivers/tty/serial/atmel_serial.c
+++ b/drivers/tty/serial/atmel_serial.c
@@ -47,6 +47,7 @@
 
 #ifdef CONFIG_ARM
 #include <mach/cpu.h>
+#include <mach/hardware.h>
 #include <asm/gpio.h>
 #endif
 
-- 
1.8.4.2

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] tty: atmel_serial: include <mach/hardware.h> explicitly

[Nicolas Ferre]

On 27/11/2013 10:38, Uwe Kleine-König :
> When compiled for at91rm9200 (i.e. ARM) the driver needs among others
> the symbol AT91RM9200_BASE_US0 which is defined in
> arch/arm/mach-at91/include/mach/at91rm9200.h. This file is included
> implicitly via several steps. As ARM's <asm/timex.h> will stop including
> <mach/hardware.h> later in this series explicitly include
> <mach/hardware.h>
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Uwe, this AT91RM9200_BASE_US0 value is supposed to disappear in 3.14 
tanks to a patch from Linus W. ([PATCH v4] ARM/serial: at91: switch 
atmel serial to use gpiolib) (not in linux-next yet).

So I think that this patch is not needed for 3.14.

Bye,

> ---
>
> Notes:
>      Hello,
>
>      I want to send this patch as part of a series dropping <mach/timex.h>
>      altogether. So please don't take this patch but only give your Ack if
>      you're OK with me sending this patch via arm-soc.
>
>      Thanks
>      Uwe
>
>   drivers/tty/serial/atmel_serial.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/tty/serial/atmel_serial.c b/drivers/tty/serial/atmel_serial.c
> index c7d99af..de51964 100644
> --- a/drivers/tty/serial/atmel_serial.c
> +++ b/drivers/tty/serial/atmel_serial.c
> @@ -47,6 +47,7 @@
>
>   #ifdef CONFIG_ARM
>   #include <mach/cpu.h>
> +#include <mach/hardware.h>
>   #include <asm/gpio.h>
>   #endif
>
>


-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] tty: atmel_serial: include <mach/hardware.h> explicitly

[Uwe Kleine-König]

On Wed, Nov 27, 2013 at 10:48:19AM +0100, Nicolas Ferre wrote:
> On 27/11/2013 10:38, Uwe Kleine-König :
> >When compiled for at91rm9200 (i.e. ARM) the driver needs among others
> >the symbol AT91RM9200_BASE_US0 which is defined in
> >arch/arm/mach-at91/include/mach/at91rm9200.h. This file is included
> >implicitly via several steps. As ARM's <asm/timex.h> will stop including
> ><mach/hardware.h> later in this series explicitly include
> ><mach/hardware.h>
> >
> >Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> 
> Uwe, this AT91RM9200_BASE_US0 value is supposed to disappear in 3.14
> tanks to a patch from Linus W. ([PATCH v4] ARM/serial: at91: switch
> atmel serial to use gpiolib) (not in linux-next yet).
> 
> So I think that this patch is not needed for 3.14.
So I'll keep this patch in my queue for now, but keep an eye on this
one. The other symbols were dropped, too?

Thanks
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] tty: atmel_serial: include <mach/hardware.h> explicitly

[Uwe Kleine-König]

On Wed, Nov 27, 2013 at 11:02:43AM +0100, Uwe Kleine-König wrote:
> On Wed, Nov 27, 2013 at 10:48:19AM +0100, Nicolas Ferre wrote:
> > On 27/11/2013 10:38, Uwe Kleine-König :
> > >When compiled for at91rm9200 (i.e. ARM) the driver needs among others
> > >the symbol AT91RM9200_BASE_US0 which is defined in
> > >arch/arm/mach-at91/include/mach/at91rm9200.h. This file is included
> > >implicitly via several steps. As ARM's <asm/timex.h> will stop including
> > ><mach/hardware.h> later in this series explicitly include
> > ><mach/hardware.h>
> > >
> > >Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > 
> > Uwe, this AT91RM9200_BASE_US0 value is supposed to disappear in 3.14
> > tanks to a patch from Linus W. ([PATCH v4] ARM/serial: at91: switch
> > atmel serial to use gpiolib) (not in linux-next yet).
> > 
> > So I think that this patch is not needed for 3.14.
> So I'll keep this patch in my queue for now, but keep an eye on this
> one. The other symbols were dropped, too?
I just verified that Linus W.'s patch is good enough for me.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html