On Thu, Apr 2, 2015 at 9:31 AM, Peter Hurley <peter@xxxxxxxxxxxxxxxxxx> wrote:
> Would you please try the patch below instead?
>
> --- >% ---
> From: Peter Hurley <peter@xxxxxxxxxxxxxxxxxx>
> Subject: [PATCH] earlycon: Fix __earlycon_table stride
>
> Signed-off-by: Peter Hurley <peter@xxxxxxxxxxxxxxxxxx>
> ---
> include/asm-generic/vmlinux.lds.h | 2 +-
> include/linux/serial_core.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> index 7b0ef49..2e11f31 100644
> --- a/include/asm-generic/vmlinux.lds.h
> +++ b/include/asm-generic/vmlinux.lds.h
> @@ -151,7 +151,7 @@
> #endif
>
> #ifdef CONFIG_SERIAL_EARLYCON
> -#define EARLYCON_TABLE() . = ALIGN(8); \
> +#define EARLYCON_TABLE() STRUCT_ALIGN(); \
> VMLINUX_SYMBOL(__earlycon_table) = .; \
> *(__earlycon_table) \
> *(__earlycon_table_end)
> diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h
> index 34de168..025dad9 100644
> --- a/include/linux/serial_core.h
> +++ b/include/linux/serial_core.h
> @@ -342,7 +342,7 @@ struct earlycon_device {
> struct earlycon_id {
> char name[16];
> int (*setup)(struct earlycon_device *, const char *options);
> -};
> +} __aligned(32);
>
> extern int setup_earlycon(char *buf);
> extern int of_setup_earlycon(unsigned long addr,
> --
Great. that works, and less lines change than my version.
ffffffff832049a0 T __earlycon_table
ffffffff832049a0 t __earlycon_uart
ffffffff832049c0 t __earlycon_uart8250
ffffffff832049e0 t __earlycon_table_sentinel
[ 0.000000] bootconsole [uart0] enabled
[ 0.000000] uart8250 probed_baud_rate: 115200
[ 0.000000] size of earlycon_id: 0x20
Thanks
Yinghai
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
