[SeaBIOS] [coreboot] [PATCH] Bootsplash for SeaBIOS (with coreboot)

Kevin O'Connor kevin at koconnor.net
Thu Jun 10 04:57:30 CEST 2010


On Wed, Jun 09, 2010 at 09:46:27AM +0200, Stefan Reinauer wrote:
> See patch

Thanks Stefan!  It looks very useful.

I haven't had a chance to review the patch fully.  One thing I did
note is that the new file jpeg.c needs a copyright statement.

At a quick glance I also noticed a couple of minor things:

* It looks like you want malloc_tmplow/tmphigh instead of
  malloc_low/high - using the permanent allocations has an impact on
  reserved space even if they are eventually freed.

* I'd guess we'd want to malloc_tmp and cbfs_copyfile instead of
  cbfs_getfile - doing the jpeg uncompression with the rom as source
  could be slow.

* there seems to be some tab/space corruption in optionrom.c - seabios
  uses all spaces

* Do the vga changes justify a new file instead of an addition to
  optionrom.c?

-Kevin



More information about the SeaBIOS mailing list