Raffaele Recalcati <lamiapost...@gmail.com> writes:

> From: Davide Bonfanti <davide.bonfa...@bticino.it>

Missing changelog.

First, UART0 is correct, you've given the same address.
Second, UART1 has been addressed by Thomas Koeller in :

   [PATCH] DM365: fixed second serial port

(although it needs one more update before going upstream.)

Kevin

> Signed-off-by: Davide Bonfanti <davide.bonfa...@bticino.it>
> Signed-off-by: Raffaele Recalcati <raffaele.recalc...@bticino.it>
> ---
>  arch/arm/mach-davinci/dm365.c               |    4 ++--
>  arch/arm/mach-davinci/include/mach/serial.h |    3 +++
>  2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c
> index 42fd4a4..e4ead0a 100644
> --- a/arch/arm/mach-davinci/dm365.c
> +++ b/arch/arm/mach-davinci/dm365.c
> @@ -1024,7 +1024,7 @@ static struct davinci_timer_info dm365_timer_info = {
>  
>  static struct plat_serial8250_port dm365_serial_platform_data[] = {
>       {
> -             .mapbase        = DAVINCI_UART0_BASE,
> +             .mapbase        = DM365_UART0_BASE,
>               .irq            = IRQ_UARTINT0,
>               .flags          = UPF_BOOT_AUTOCONF | UPF_SKIP_TEST |
>                                 UPF_IOREMAP,
> @@ -1032,7 +1032,7 @@ static struct plat_serial8250_port 
> dm365_serial_platform_data[] = {
>               .regshift       = 2,
>       },
>       {
> -             .mapbase        = DAVINCI_UART1_BASE,
> +             .mapbase        = DM365_UART1_BASE,
>               .irq            = IRQ_UARTINT1,
>               .flags          = UPF_BOOT_AUTOCONF | UPF_SKIP_TEST |
>                                 UPF_IOREMAP,
> diff --git a/arch/arm/mach-davinci/include/mach/serial.h 
> b/arch/arm/mach-davinci/include/mach/serial.h
> index f6c4f34..dd69cd3 100644
> --- a/arch/arm/mach-davinci/include/mach/serial.h
> +++ b/arch/arm/mach-davinci/include/mach/serial.h
> @@ -17,6 +17,9 @@
>  #define DAVINCI_UART1_BASE   (IO_PHYS + 0x20400)
>  #define DAVINCI_UART2_BASE   (IO_PHYS + 0x20800)
>  
> +#define DM365_UART0_BASE     0x01C20000

You've noticed that this results in the same value
as the existing UART0, right?

> +#define DM365_UART1_BASE     0x01D06000
> +
>  #define DA8XX_UART0_BASE     (IO_PHYS + 0x042000)
>  #define DA8XX_UART1_BASE     (IO_PHYS + 0x10c000)
>  #define DA8XX_UART2_BASE     (IO_PHYS + 0x10d000)
> -- 
> 1.7.0.4
>
> _______________________________________________
> Davinci-linux-open-source mailing list
> Davinci-linux-open-source@linux.davincidsp.com
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to