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

Peter Stuge peter at stuge.se
Thu May 28 02:30:20 CEST 2009


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>

..
> +/**
> + * Create the Fixed ACPI Description Tables (FADT) for this board.

Please change or yank 'board' here.


> +	memcpy(header->signature, "FACP", 4);

Can you explain FADT vs. FACP ?


> +	memcpy(header->asl_compiler_id, "IASL", 4);

Why not "CORE" ?


Other than that:

Acked-by: Peter Stuge <peter at stuge.se>




More information about the coreboot mailing list