[PATCH] Board enable for Abit VA6

[Mattias Mattsson]

Hi,

This adds board enable for Abit VA6.

I'm not sure if the DMI matching here is good enough, maybe we should
require the -m option. There is support in flashrom for a similar abit
board (VT6X4) which currently requires this.

-mattias

[Michael Karcher]

Am Sonntag, den 15.08.2010, 07:05 +0200 schrieb Mattias Mattsson:
> I'm not sure if the DMI matching here is good enough, maybe we should
> require the -m option. There is support in flashrom for a similar abit
> board (VT6X4) which currently requires this.

Add a '$' to the DMI match (to anchor the end, it is a substring match
otherwise), and I think it should be fine. The VT6X4 had no usable DMI
strings at all, the VA6 seems to be an improvement already.

with "(VA6)$" as DMI match, this is:
Acked-by: Michael Karcher

Thanks for your patch,
  Michael Karcher

[Mattias Mattsson]

On Sun, Aug 15, 2010 at 11:47, Michael Karcher
 wrote:
> Am Sonntag, den 15.08.2010, 07:05 +0200 schrieb Mattias Mattsson:
> > I'm not sure if the DMI matching here is good enough, maybe we should
> > require the -m option. There is support in flashrom for a similar abit
> > board (VT6X4) which currently requires this.
>
> Add a '$' to the DMI match (to anchor the end, it is a substring match
> otherwise), and I think it should be fine. The VT6X4 had no usable DMI
> strings at all, the VA6 seems to be an improvement already.
>
> with "(VA6)$" as DMI match, this is:
> Acked-by: Michael Karcher

No problem. New patch attached. Also added status and URL in print.c.

Signed-off-by: Mattias Mattsson

Thanks
-mattias

[Michael Karcher]

Am Sonntag, den 15.08.2010, 21:09 +0200 schrieb Mattias Mattsson:
> > Add a '$' to the DMI match (to anchor the end, it is a substring match
> > otherwise), and I think it should be fine. The VT6X4 had no usable DMI
> > strings at all, the VA6 seems to be an improvement already.
> >
> > with "(VA6)$" as DMI match, this is:
> > Acked-by: Michael Karcher
>
> No problem. New patch attached. Also added status and URL in print.c.
>
> Signed-off-by: Mattias Mattsson

Thanks for adding the URL and resending.

Committed as r1143.

Regards,
  Michael Karcher