[coreboot] ACPI support on M57SLI
Harald Gutmann
harald.gutmann at gmx.net
Thu Jun 18 13:47:10 CEST 2009
On Tuesday 16 June 2009 05:52:38 Myles Watson wrote:
> I don't have the hardware, so I can't test.
>
> +unsigned long write_acpi_tables(unsigned long start)
> +{
> + get_bus_conf();
> + sbdn = sysconf.sbdn;
Done.
> Since sbdn isn't used in write_acpi_tables, could you make it a local
> variable in acpi_fill_madt instead and move that code there?
> write_acpi_tables is pretty generic, and will eventually be factored out at
> least per northbridge, so it makes sense to keep it as clean as possible.
>
> fadt.c should be in the southbridge/nvidia/mcp55/ directory. Won't it work
> for any board that has that chipset?
Done.
> In Config.lb it seems like the indenting isn't uniform. Could you use tabs
> to make it cleaner? Also in dsdt.asl there are multiple lines that have
> indented blank lines.
The dsdt.asl is now whitspace/indenting cleared.
And got some further improvements to be able to install and boot windows 7
from coreboot with acpi support on m57sli.
I have already done a whitespace/indenting fix on Config.lb but attaching this
to the acpi patch would just blow it up too much.
> Thanks,
> Myles
Thanks for your review,
Harald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: acpi_support.diff
Type: text/x-patch
Size: 23561 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090618/a9deca88/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090618/a9deca88/attachment.sig>
More information about the coreboot
mailing list