[U-Boot] [PATCH 39/71] serial: arm: Implement CONFIG_SERIAL_MULTI into altera_jtag serial driver

Marek Vasut marex at denx.de
Tue Oct 2 12:07:05 CEST 2012


Dear Thomas Chou,

> On 10/01/2012 10:41 PM, Marek Vasut wrote:
> >> 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?
> 
> Dear Marek,
> 
> The serial.h defines the struct serial_device.
> And the linux/compiler.h defines the __weak attribute.
> Without them, it won't compile.
> 
> I have pulled your latest dm-serial-1 branch, compiled and tested
> altera_uart and altera_jtag_uart on nios2 board. They work without problem.
> 
> There is only minor size change.
> 
> Before..
> Configuring for nios2-generic board...
>     text	   data	    bss	    dec	    hex	filename
>   134782	   3206	  23173	 161161	  27589	/home/thomas/tmp/u-boot/u-boot
> 
> After..
> Configuring for nios2-generic board...
>     text	   data	    bss	    dec	    hex	filename
>   135450	   3254	  23181	 161885	  2785d	/home/thomas/tmp/u-boot/u-boot
> 
> Thank you very much.

Perfect! Thank YOU!

> Best regards
> Thomas Chou

Best regards,
Marek Vasut


More information about the U-Boot mailing list