Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup
Hi, On Wed, 05 Sep 2012 09:51:12 +0300 Igor Grinberg wrote: > 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. Sorry for the delay, I'm really busy with other projects and have only very limitted capacity to process U-Boot patches. These are not lost, and assigned to me at patchwork. For now I've applied this series, hope it will not break anything for all these affected platforms. But please do not expect me pushing to u-boot/master this week, Wolfgang won't pull this week anyway, I'm afraid. Thanks for the patience, and sorry again Anatolij ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup
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 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup
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. 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(-) > -- Regards, Igor. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup
Gentle ping. -Original Message- From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Nikita Kiryanov Sent: Monday, August 20, 2012 9:16 AM To: Nikita Kiryanov Cc: u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup Gentle ping. On 08/09/2012 01:14 PM, 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(-) > ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH V3 0/4] common/lcd cleanup
Gentle ping. On 08/09/2012 01:14 PM, 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(-) ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH V3 0/4] common/lcd cleanup
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(-) -- 1.7.5.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot