Dear Thomas Chou,
> On 09/17/2012 07:21 AM, Marek Vasut wrote:
> > Implement support for CONFIG_SERIAL_MULTI into altera_jtag serial driver.
> > This driver was so far only usable directly, but this patch also adds
> > support for the multi method. This allows using more than one serial
> > driver alongside the altera_jtag driver. Also, add a weak implementation
> > of default_serial_console() returning this driver.
> >
> > Signed-off-by: Marek Vasut <ma...@denx.de>
> > Cc: Marek Vasut <marek.va...@gmail.com>
> > Cc: Tom Rini <tr...@ti.com>
> > Cc: Thomas Chou <tho...@wytron.com.tw>
> > ---
> >
> > common/serial.c | 2 ++
> > drivers/serial/altera_jtag_uart.c | 71
> > +++++++++++++++++++++++++++++++++---- 2 files changed, 67
> > insertions(+), 6 deletions(-)
>
> Dear Marek,
>
> Please include the following headers.
>
> --- a/drivers/serial/altera_jtag_uart.c
> +++ b/drivers/serial/altera_jtag_uart.c
> @@ -25,6 +25,8 @@
> #include <watchdog.h>
> #include <asm/io.h>
> #include <nios2-io.h>
> +#include <linux/compiler.h>
> +#include <serial.h>
Are you seeing any issues without them?
> DECLARE_GLOBAL_DATA_PTR;
>
> And the .name string is too long. >16
>
> static struct serial_device altera_jtag_serial_drv = {
> .name = "altera_jtag_serial",
>
> Otherwise,
> Acked-by: Thomas Chou <tho...@wytron.com.tw>
>
> Best regards,
> Thomas Chou
[...]
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
