[U-Boot] [PATCH 02/10] lcd, tegra: remove unused cursor functions

Tom Warren TWarren at nvidia.com
Fri Jan 25 22:18:15 CET 2013


Jeroen,

How will this all go in? Do you expect Anatolij to take it in via the main repo (u-boot.git/master)?

LGTM, BTW

Tom

> -----Original Message-----
> From: sjg at google.com [mailto:sjg at google.com] On Behalf Of Simon Glass
> Sent: Friday, January 25, 2013 1:38 PM
> To: Jeroen Hofstee
> Cc: u-boot at lists.denx.de; Tom Warren
> Subject: Re: [U-Boot] [PATCH 02/10] lcd, tegra: remove unused cursor
> functions
> 
> +Tom
> 
> On Sun, Jan 13, 2013 at 11:07 AM, Jeroen Hofstee <jeroen at myspectrum.nl>
> wrote:
> > cc: Anatolij Gustschin <agust at denx.de>
> > cc: Simon Glass <sjg.chromium.org>
> > Signed-off-by: Jeroen Hofstee <jeroen at myspectrum.nl>
> 
> Acked-by: Simon Glass <sjg at chromium.org>
> 
> > ---
> >  drivers/video/tegra.c |   52 -------------------------------------------------
> >  1 file changed, 52 deletions(-)
> >
> > diff --git a/drivers/video/tegra.c b/drivers/video/tegra.c index
> > 3709d0b..26a96a5 100644
> > --- a/drivers/video/tegra.c
> > +++ b/drivers/video/tegra.c
> > @@ -73,62 +73,10 @@ vidinfo_t panel_info = {
> >         .vl_col = -1,
> >  };
> >
> > -char lcd_cursor_enabled;
> > -
> > -ushort lcd_cursor_width;
> > -ushort lcd_cursor_height;
> > -
> >  #ifndef CONFIG_OF_CONTROL
> >  #error "You must enable CONFIG_OF_CONTROL to get Tegra LCD support"
> >  #endif
> >
> > -void lcd_cursor_size(ushort width, ushort height) -{
> > -       lcd_cursor_width = width;
> > -       lcd_cursor_height = height;
> > -}
> > -
> > -void lcd_toggle_cursor(void)
> > -{
> > -       ushort x, y;
> > -       uchar *dest;
> > -       ushort row;
> > -
> > -       x = console_col * lcd_cursor_width;
> > -       y = console_row * lcd_cursor_height;
> > -       dest = (uchar *)(lcd_base + y * lcd_line_length + x * (1 << LCD_BPP) /
> > -                       8);
> > -
> > -       for (row = 0; row < lcd_cursor_height; ++row, dest += lcd_line_length)
> {
> > -               ushort *d = (ushort *)dest;
> > -               ushort color;
> > -               int i;
> > -
> > -               for (i = 0; i < lcd_cursor_width; ++i) {
> > -                       color = *d;
> > -                       color ^= lcd_getfgcolor();
> > -                       *d = color;
> > -                       ++d;
> > -               }
> > -       }
> > -}
> > -
> > -void lcd_cursor_on(void)
> > -{
> > -       lcd_cursor_enabled = 1;
> > -       lcd_toggle_cursor();
> > -}
> > -void lcd_cursor_off(void)
> > -{
> > -       lcd_cursor_enabled = 0;
> > -       lcd_toggle_cursor();
> > -}
> > -
> > -char lcd_is_cursor_enabled(void)
> > -{
> > -       return lcd_cursor_enabled;
> > -}
> > -
> >  static void update_panel_size(struct fdt_disp_config *config)  {
> >         panel_info.vl_col = config->width;
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
--
nvpublic


More information about the U-Boot mailing list