On 28.04.2009 14:24 Uhr, Patrick Georgi wrote: > Am 28.04.2009 10:41, schrieb Patrick Georgi: >> Am 27.04.2009 23:37, schrieb Peter Stuge: >>> This code doesn't seem mainboard specific so I think it must not be >>> in mainboard/. >> add_mainboard_resources is necessary for some boards (eg. kontron), but >> this generic code could be added to the caller of >> add_mainboard_resources (wrapped in HAVE_HIGH_TABLES, of course). >> That way, boards that really need it (for other things) can use this >> function, while others don't have to do anything. >> >> I'll prepare a patch for that. > Attached. > > Signed-off-by: Patrick Georgi <[email protected]>
Wonderful! This is much better. Can we enable HAVE_HIGH_TABLES for all amdk8 boards now? Acked-by: Stefan Reinauer <[email protected]> -- coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br. Tel.: +49 761 7668825 • Fax: +49 761 7664613 Email: [email protected] • http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg • HRB 7656 Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866 -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
