[coreboot] [PATCH] move fadt.c closer to chipset (VT8237)

Uwe Hermann uwe at hermann-uwe.de
Thu May 28 13:45:03 CEST 2009


On Thu, May 28, 2009 at 02:30:20AM +0200, Peter Stuge wrote:
> Rudolf Marek wrote:
> > Following patch moves all vt8237 fadt.c from mainboard/* file to
> > chipset directory just with one common file.
> > 
> > Signed-off-by: Rudolf Marek <r.marek at assembler.cz>

Yep, very nice. Factoring out common code is always a good thing.

Acked-by: Uwe Hermann <uwe at hermann-uwe.de>


> > +/**
> > + * Create the Fixed ACPI Description Tables (FADT) for this board.
> 
> Please change or yank 'board' here.

Ack.

 
> > +	memcpy(header->asl_compiler_id, "IASL", 4);
> 
> Why not "CORE" ?

What's the purpose of this field? Does it mean the user
should use 'iasl'? That iasl was used to create the file?
Depending on the meaning, IASL or CORE may be more appropriate,
but I don't know enough about this stuff to decide...


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




More information about the coreboot mailing list