Sudhakar Rajashekhara wrote:
> Source: Texas Instruments Inc.
> Type: Bug-fix
> Signed-off-by: Sudhakar Rajashekhara <[EMAIL PROTECTED]>
> Description:
> Fixes the UART mis-behavior on dm6467.
> ---
>  arch/arm/mach-davinci/serial.c |   14 ++++++++++++++
>  1 files changed, 14 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/mach-davinci/serial.c b/arch/arm/mach-davinci/serial.c
> index 8e041d3..e6ae903 100644
> --- a/arch/arm/mach-davinci/serial.c
> +++ b/arch/arm/mach-davinci/serial.c
> @@ -72,6 +72,18 @@ static void __init davinci_serial_reset(struct 
> plat_serial8250_port *p)
>       davinci_serial_outs(p, UART_DAVINCI_PWREMU, pwremu);
>  }
>  
> +#define UART_DM6467_SCR                __REG(DAVINCI_UART0_BASE + 0x40)
> +/*
> + * Internal UARTs need to be initialized for the 8250 autoconfig to work
> + * properly. Note that the TX watermark initialization may not be needed
> + * once the 8250.c watermark handling code is merged.
> + */
> +static int __init dm646x_serial_reset(void)
> +{
> +     UART_DM6467_SCR = 0x08;
> +     return 0;
> +}
> +
>  void __init davinci_serial_init(struct platform_device *pdev)
>  {
>       int i;
> @@ -111,3 +123,5 @@ void __init davinci_serial_init(struct platform_device 
> *pdev)
>               }
>       }
>  }
> +
> +late_initcall(dm646x_serial_reset);

This bug fix does not look specific to the 6467 to me. What happen to the 6446 
when
a non-existent register is accessed? Would it be better to put this in your 
board
file?


Troy

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to