diff mbox

ARM: davinci: fix build breakage with DEBUG_LL

Message ID 1378819415-28666-1-git-send-email-nsekhar@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sekhar Nori Sept. 10, 2013, 1:23 p.m. UTC
commit fcf7157 ("ARM: davinci: serial: get rid of davinci_uart_config")
introduced build breakage because of a misplaced
header file include which cause a bunch of errors when
the file is included in assembly code (like debug-macro.S)

Fix the build breakage.

Reported-by: Alexander Holler <holler@ahsoftware.de>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
---
Applies to latest linus/master

 arch/arm/mach-davinci/include/mach/serial.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Alexander Holler Sept. 10, 2013, 1:36 p.m. UTC | #1
Hello Sekhar,

just what I did here to fix it. So you can add a Tested-By or Acked-By 
if you wish.

Thanks,

Alexander Holler

Am 10.09.2013 15:23, schrieb Sekhar Nori:
> commit fcf7157 ("ARM: davinci: serial: get rid of davinci_uart_config")
> introduced build breakage because of a misplaced
> header file include which cause a bunch of errors when
> the file is included in assembly code (like debug-macro.S)
>
> Fix the build breakage.
>
> Reported-by: Alexander Holler <holler@ahsoftware.de>
> Signed-off-by: Sekhar Nori <nsekhar@ti.com>
> ---
> Applies to latest linus/master
>
>   arch/arm/mach-davinci/include/mach/serial.h |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/include/mach/serial.h b/arch/arm/mach-davinci/include/mach/serial.h
> index 52b8571..ce402cd 100644
> --- a/arch/arm/mach-davinci/include/mach/serial.h
> +++ b/arch/arm/mach-davinci/include/mach/serial.h
> @@ -15,8 +15,6 @@
>
>   #include <mach/hardware.h>
>
> -#include <linux/platform_device.h>
> -
>   #define DAVINCI_UART0_BASE	(IO_PHYS + 0x20000)
>   #define DAVINCI_UART1_BASE	(IO_PHYS + 0x20400)
>   #define DAVINCI_UART2_BASE	(IO_PHYS + 0x20800)
> @@ -39,6 +37,8 @@
>   #define UART_DM646X_SCR_TX_WATERMARK	0x08
>
>   #ifndef __ASSEMBLY__
> +#include <linux/platform_device.h>
> +
>   extern int davinci_serial_init(struct platform_device *);
>   #endif
>
>
Sekhar Nori Sept. 10, 2013, 2:50 p.m. UTC | #2
On 9/10/2013 7:06 PM, Alexander Holler wrote:
> Hello Sekhar,
> 
> just what I did here to fix it. So you can add a Tested-By or Acked-By
> if you wish.

Sure. I will add your Tested-By.

Regards,
Sekhar
diff mbox

Patch

diff --git a/arch/arm/mach-davinci/include/mach/serial.h b/arch/arm/mach-davinci/include/mach/serial.h
index 52b8571..ce402cd 100644
--- a/arch/arm/mach-davinci/include/mach/serial.h
+++ b/arch/arm/mach-davinci/include/mach/serial.h
@@ -15,8 +15,6 @@ 
 
 #include <mach/hardware.h>
 
-#include <linux/platform_device.h>
-
 #define DAVINCI_UART0_BASE	(IO_PHYS + 0x20000)
 #define DAVINCI_UART1_BASE	(IO_PHYS + 0x20400)
 #define DAVINCI_UART2_BASE	(IO_PHYS + 0x20800)
@@ -39,6 +37,8 @@ 
 #define UART_DM646X_SCR_TX_WATERMARK	0x08
 
 #ifndef __ASSEMBLY__
+#include <linux/platform_device.h>
+
 extern int davinci_serial_init(struct platform_device *);
 #endif