On Mon, May 11, 2009 at 10:08:47AM -0700, ron minnich wrote: > On Mon, May 11, 2009 at 10:01 AM, Myles Watson <[email protected]> wrote: > > > I agree. Let's do that first, since it would simplify this fix. I'll > > review and test. > > Let's see what other comments we get ... anyone out there see an issue > with 32 byte character names? If you want, we can go with 40, which > will size the struct up to a nice simple 64 bytes ...
Just my 2 cents, but I like the variable length file names. BTW, you can simplify the existing coreboot code a bit if you add a "char filename[0];" entry to 'struct cbfs_file' - as SeaBIOS does. -Kevin -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
