Re: [RFC 26/29] xen/arm: Add Exynos 4210 UART support for early printk

[Anthony PERARD <anthony.perard@citrix.com>]

On 29/04/13 00:02, Julien Grall wrote:
> diff --git a/xen/arch/arm/arm32/debug-exynos5.S b/xen/arch/arm/arm32/debug-exynos5.S
> new file mode 100644
> index 0000000..cbe1705
> --- /dev/null
> +++ b/xen/arch/arm/arm32/debug-exynos5.S
> @@ -0,0 +1,81 @@
> +/*
> + * xen/arch/arm/arm32/debug-exynos5.S
> + *
> + * Exynos 5 specific debug code
> + *
> + * Copyright (c) 2013 Citrix Systems.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
> +
> +#include <asm/asm_defns.h>
> +
> +#define EXYNOS5_UART_BASE_ADDRESS 0x12c20000
> +
> +.globl early_uart_paddr
> +early_uart_paddr: .word EXYNOS5_UART_BASE_ADDRESS
> +
> +/* Exynos 5 UART initialization
> + * r11: UART base address
> + * Clobber r0-r1 */
> +.globl early_uart_init
> +early_uart_init:
> +        /* init clock */
> +        ldr   r1, =0x10020000
> +        /* select MPLL (800MHz) source clock */
> +        ldr   r0, [r1, #0x250]
> +        and   r0, r0, #(~(0xf<<8))
> +        orr   r0, r0, #(0x6<<8)
> +        str   r0, [r1, #0x250]
> +        /* ration 800/(7+1) */
> +        ldr   r0, [r1, #0x558]
> +        and   r0, r0, #(~(0xf<<8))
> +        orr   r0, r0, #(0x7<<8)
> +        str   r0, [r1, #0x558]
> +
> +        mov   r1, #4
> +        str   r1, [r11, #0x2c]          /* -> UARTIBRD (Baud divisor fraction) */

Could you replace UARTIBRD by UFRACVAL? The former is the name for the
PL011, and the latter is the name given by the Exynos 5 manual.

> +        mov   r1, #53
> +        str   r1, [r11, #0x28]          /* -> UARTIBRD (Baud divisor integer) */

Same here with UBRDIV.

-- 
Anthony PERARD