[coreboot] [commit] r5286 - ...
mylesgw at gmail.com
Fri Apr 9 05:44:47 CEST 2010
On Thu, Apr 8, 2010 at 5:03 PM, Stefan Reinauer <stepan at coresystems.de>wrote:
> On 4/8/10 7:07 PM, Myles Watson wrote:
> Here's one for all boards.
> Signed-off-by: Myles Watson <mylesgw at gmail.com>
> Acked-by: Stefan Reinauer <stepan at coresystems.de> <stepan at coresystems.de>
> Index: svn/src/mainboard/amd/dbm690t/acpi_tables.c
> --- svn.orig/src/mainboard/amd/dbm690t/acpi_tables.c
> +++ svn/src/mainboard/amd/dbm690t/acpi_tables.c
> @@ -57,7 +57,7 @@ static void dump_mem(u32 start, u32 end)
> -extern const acpi_header_t AmlCode;
> +extern const unsigned char AmlCode;
> I don't think defining them const here gains us much, as they're not
> defined const when they're created.
I thought that making them const here would make it so that you couldn't do:
AmlCode = 'c';
without a warning.
Can we change that, maybe? Extra sed s/unsigned char AmlCode/const unsigned
> char AmlCode/g on the iasl output maybe?
That would be fine with me. It might be more effort than it's worth. As
long as we copy it before anything else, there shouldn't be any problems.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the coreboot