[U-Boot] Pull request: u-boot-video/master

Igor Grinberg grinberg at compulab.co.il
Fri Sep 28 09:40:49 CEST 2012


On 09/25/12 22:12, Anatolij Gustschin wrote:
> On Fri, 21 Sep 2012 16:21:22 -0700
> Tom Rini <trini at ti.com> wrote:
> ...
>> lcd.c: In function 'configuration_get_cmap':
>> lcd.c:513:29: error: 'struct vidinfo' has no member named 'cmap'
>> lcd.c: In function 'lcd_display_bitmap':
>> lcd.c:787:6: warning: implicit declaration of function 'FB_PUT_BYTE' [-Wimplicit-function-declaration]
>> lcd.c: In function 'configuration_get_cmap':
>> lcd.c:515:1: warning: control reaches end of non-void function [-Wreturn-type]
>> make[1]: *** [/home/trini/work/ssd/u-boot/u-boot/trats/common/lcd.o] Error 1
>> make[1]: *** Waiting for unfinished jobs....
>> make: *** [/home/trini/work/ssd/u-boot/u-boot/trats/common/libcommon.o] Error 2
>> make: *** Waiting for unfinished jobs....
>>
>> --------------------- SUMMARY ----------------------------
>> Boards compiled: 1
>> Boards with errors: 1 ( trats )
>> ----------------------------------------------------------
>>
>> Rejected, please fix and re-submit.  Thanks!
> 
> Sorry, I trusted the statement of the submitter that he did run
> MAKEALL for arm and powerpc for these lcd patches, and thus didn't
> verify by my own MAKEALL tests. I'll fix and resubmit.

You can still trust the submitter, Nikita ran the MAKEALL for arm and ppc.
The problem is that it was back in the beginning of August...
It is just the time frame between the submission and
the pickup that changed things and probably made a difference.

Nikita, has not been around for a while and
therefore had no chance to look into your fix and ack it.
Nevertheless, thanks for your patch, that fixed the failing boards.

-- 
Regards,
Igor.


More information about the U-Boot mailing list