Hi,

On Fri, Oct 30, 2009 at 12:14:06PM +0000, Sean Young wrote:
> > > + int16_t magic;
> > 
> > This can be uint16_t right? Or is there some reason to make it signed
> > explicitly?
> 
> There is no need for it to be signed. Unsigned makes more sense actually.

Fixed.

Patch comitted in r4931, thanks! I also updated the wiki and linked to
your sample dump there.


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.randomprojects.org
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to