Hi Pali, On Sun, 6 Mar 2022 at 08:25, Pali Rohár <p...@kernel.org> wrote: > > On Sunday 06 March 2022 07:42:03 Simon Glass wrote: > > Yes let's go with that. > > Done, I sent final version of the patch. > > > > > > > > > Second thing is that CONFIG_VIDEO_LOGO is broken and does not > > > > > > > > work even it > > > > > > > > is enabled in config file. I do not know why too. > > > > > > > > > > > > > > > > Any idea? > > > > > > > > > > > > > > Not yet. There were some logo related changes recently, but if I > > > > > > > remember correctly, I tested them on wandboard and nitrogen6q > > > > > > > targets and with sandbox, and logo drawing worked there. > > > > > > > > Can you be more specific than 'broken'? What is broken about it? > > > > > > Does not work, logo is not drown on the screen. > > > > See video_bmp_display() - I wonder if the particular depth you are > > using is not supported? > > Depth is 16bit which should be supported.
But apparently isn't , at least in this case :-) > > > Anyway you should be able to debug it there or > > using the bmp command. The file is drivers/video/u_boot_logo.bmp > > CONFIG_CMD_BMP=y is broken too. When I enable it then U-Boot freeze > after the line 'Loading Environment from <NULL>... OK'. OK I'm not sure why, but please investigate if you have time. Regards, Simon