[U-Boot] [PATCH V2 09/12] VIDEO: davinci: add framebuffer to da8xx

Anatolij Gustschin agust at denx.de
Tue Oct 11 18:10:47 CEST 2011


Hi Stefano,

On Sat,  8 Oct 2011 11:27:34 +0200
Stefano Babic <sbabic at denx.de> wrote:

> The patch is a port from the framebuffer driver
> of the Linux driver drivers/video/da8xx-fb.c, used
> on davinci da8xx and OMAP-L138 boards.
> 
> As base for the port, the following commit (last changes
> for this driver at the moment in the Linux kernel tree)
> was taken:
> 
> commit 1db41e032d563eb47deab40dc5595be306b143ba
> Author: axel lin <axel.lin at gmail.com>
> Date:   Tue Feb 22 01:52:42 2011 +0000
> 
>     video: da8xx-fb: fix section mismatch warning
> 
>     Signed-off-by: Axel Lin <axel.lin at gmail.com>
>     Signed-off-by: Paul Mundt <lethal at linux-sh.org>
> 
> Signed-off-by: Stefano Babic <sbabic at denx.de>
> CC: Sandeep Paulraj <s-paulraj at ti.com>
> Cc: Anatolij Gustschin <agust at denx.de>
> ---
> 
> Changes:
> 
> - set a correct license header and drop dead code in da8xx-fb.h
> 	 (Anatolij Gustschin)
> - change commit message adding better references to driver in linux
> used for porting (Wolfgang Denk)

Thanks for these fixes. When applying this patch I see:

$ git am /tmp/\[PATCH\ V2\ 09_12\]\ VIDEO\:\ davinci\:\ add\ framebuffer\ to\ da8xx 
Applying: VIDEO: davinci: add framebuffer to da8xx
/home/ag/u-boot/u-boot-move-new-host/u-boot-video/.git/rebase-apply/patch:22: new blank line at EOF.
+
warning: 1 line adds whitespace errors.

This is due to the trailing empty line in
"arch/arm/include/asm/arch-davinci/da8xx-fb.h" I'think.
Can you please fix it also?

Then a fixed patch can go via TI ARM tree together with other patches
in this series. For a fixed patch you can add my
Acked-by: Anatolij Gustschin <agust at denx.de>

Thanks,
Anatolij


More information about the U-Boot mailing list