On Sunday 06 March 2022 11:44:22 Simon Glass wrote: > 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 :-)
Yea, something is broken there. > > > > > 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. I have looked at it and do not know. But probably it is because I somehow have not caught how this new video code works... Anyway, I sent documentation update how to run u-boot for n900 in qemu: https://patchwork.ozlabs.org/project/uboot/patch/20220308175956.4522-1-p...@kernel.org/ So it could help other people to look at issues which do not work correctly in n900's u-boot without need to have n900 hardware.