Hi Pali,
On Sun, 6 Mar 2022 at 08:25, Pali Rohár pali@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