[U-Boot] [PATCH V3 0/4] common/lcd cleanup

Marek Vasut marex at denx.de
Wed Sep 5 08:56:50 CEST 2012


Dear Igor Grinberg,

> Hi guys,
> 
> This series is waiting for several month already.
> Only the V3 is one month old and hasn't got any comments so far.
> So, if Anatolij for some reason cannot deal with it,
> please merge it via a staging tree of yours.

Anatolij, poke?

It was probably forgotten, simple bump would suffice. Sorry for the delay.

> Thanks
> 
> On 08/09/12 13:14, Nikita Kiryanov wrote:
> > This patch series attempts to simplify #ifdef complexity in common/lcd.c.
> > 
> > It was compile tested on Arm and PowerPC using MAKEALL
> > 
> > Changes in V3:
> > 	- Rebased on latest u-boot-video
> > 	- Removed volatile qualifiers in patch 3 since the use of volatile is
> > 	discouraged and it looks like it is safe to remove them.
> > 
> > Changes in V2:
> > 	- Rebased on u-boot-video
> > 	- patches 2 and 3 of original patchset dropped because I'm not sure
> > 	what to do about them
> > 	- simplify lcd_logo: used bitmap_display() to further simplify code
> > 	- simplify lcd_display_bitmap: fixed pointer increment error
> > 	- simplify lcd_display_bitmap: change to simplify lcd_logo breaks
> > 	MCC200 board because it does not #define CONFIG_CMD_BMP. Added a local
> > 	implementation of bitmap_display().
> > 
> > Nikita Kiryanov (4):
> >   common lcd: simplify lcd_logo
> >   common lcd: simplify lcd_display
> >   common lcd: simplify core functions
> >   common lcd: simplify lcd_display_bitmap
> >  
> >  board/mcc200/lcd.c |   20 +++++++
> >  common/lcd.c       |  145
> >  +++++++++++++++++++++++++++------------------------- 2 files changed,
> >  96 insertions(+), 69 deletions(-)

Best regards,
Marek Vasut


More information about the U-Boot mailing list