On Tue, Sep 07, 2010 at 09:18:52AM +0200, Michael Karcher wrote: > Am Dienstag, den 07.09.2010, 09:06 +0200 schrieb Tim ter Laak: > > This patch adds support for the Abit BM6 board, using DMI string > > identification, and lists it in print.c . > > > > Signed-off-by: Tim ter Laak <[email protected]> > Acked-by: Michael Karcher <[email protected]> > > If no one else beats me to it, I will commit tonight.
Looks good to me too, but two small requests: > Index: board_enable.c > =================================================================== > --- board_enable.c (revision 1153) > +++ board_enable.c (working copy) > @@ -1093,6 +1093,15 @@ > > /* > * Suited for: > + * - Abit AB-BM6 s/Abit/abit/ for consistency (though we may change the convention later). And: "AB-BM6" should be just "BM6" as that's the name on the website. Where does the "AB-" prefix come from? Does it say so on the PCB or something? Either way: Acked-by: Uwe Hermann <[email protected]> Uwe. -- http://hermann-uwe.de | http://sigrok.org http://randomprojects.org | http://unmaintained-free-software.org -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
