[PATCH] Enable VX855 chipset

[John Schmerge]

This patch adds support for the Via VX855 chipset

[Stefan Tauner]

On Mon, 25 Apr 2011 20:10:02 -0400
John Schmerge  wrote:
> This patch adds support for the Via VX855 chipset

hey

my shot in the dark wasnt THAT bad then after all :)
the patch you sent is basically fine, but please resend it after you
have corrected the order of the chipsets.
the vx855 should be added just after the vx800 not at the end
also, please add a signed-off tag as described in
http://www.coreboot.org/Development_Guidelines#Sign-off_Procedure
thanks!

[Paul Menzel]

Dear John, dear Stefan,

Am Dienstag, den 26.04.2011, 09:56 +0200 schrieb Stefan Tauner:
> On Mon, 25 Apr 2011 20:10:02 -0400 John Schmerge wrote:
> > This patch adds support for the Via VX855 chipset
>
> my shot in the dark wasnt THAT bad then after all :)

what do you refer to? Did you send a patch before or did you suggest
this for example in an IRC conversion?
> the patch you sent is basically fine, but please resend it after you
> have corrected the order of the chipsets.
> the vx855 should be added just after the vx800 not at the end
> also, please add a signed-off tag as described in
> http://www.coreboot.org/Development_Guidelines#Sign-off_Procedure
> thanks!

John, could you please add also the board you used to verify this. This
is not only for completeness but also for my curiosity what boards are
out there using this chipset.

Thanks,

Paul

[Stefan Tauner]

On Tue, 26 Apr 2011 10:17:30 +0200
Paul Menzel  wrote:
> > my shot in the dark wasnt THAT bad then after all :)
>
> what do you refer to? Did you send a patch before or did you suggest
> this for example in an IRC conversion?

we talked on irc about it and i suggested trying one of the existing
via chipset enables, but apparently the wrong one ;)

[John Schmerge]

Attached is an updated version of the patch to enable the VX855
chipset. It addresses the chipset ordering issue that Stefan had with
my original patch. As for the board used to verify operation, it is a
custom board that we manufacture for this product:
  http://www.devonit.com/hardware/tc2-thin-client/technical-specifications

Signed-off-by: John Schmerge  for Devon IT

[Stefan Tauner]

On Wed, 4 May 2011 17:51:18 -0400
John Schmerge  wrote:
> Attached is an updated version of the patch to enable the VX855
> chipset. It addresses the chipset ordering issue that Stefan had with
> my original patch. As for the board used to verify operation, it is a
> custom board that we manufacture for this product:
>   http://www.devonit.com/hardware/tc2-thin-client/technical-specifications
>
> Signed-off-by: John Schmerge  for Devon IT

Thanks!
Acked-by: Stefan Tauner

[Michael Karcher]

Am Donnerstag, den 05.05.2011, 16:57 +0200 schrieb Stefan Tauner:
> > Attached is an updated version of the patch to enable the VX855
> > chipset. It addresses the chipset ordering issue that Stefan had with
> > Signed-off-by: John Schmerge  for Devon IT
>
> Acked-by: Stefan Tauner

Thanks for writing and acking.

Committed in r1295.

Regards,
  Michael Karcher

[Stefan Tauner]

Hello John,

i am writing to you because of your previous work on VIA-based boards.
I have refined flashrom's handling of all VIA VX-family chipsets in a
new patch. If you still have access to some VIA systems could you
please apply the patch[1] and at least probe for a chip?
Be aware that we require the -p internal option now to select the
internal programmer (there is no implicit default programmer set at
compilation time any more).

[1]: http://patchwork.coreboot.org/patch/3695/

[John Schmerge]

Hi Stefan, i'm sorry for the rather slow response, i've been taking
some time away from everything tech related. Someone you may wish to
reach out to to ask about this is Ian Geiser at Devon IT (my former
employer)... His email address is [e-mail]

-John

On Tue, Jul 24, 2012 at 7:46 AM, Stefan Tauner
 wrote:
> Hello John,
>
> i am writing to you because of your previous work on VIA-based boards.
> I have refined flashrom's handling of all VIA VX-family chipsets in a
> new patch. If you still have access to some VIA systems could you
> please apply the patch[1] and at least probe for a chip?
> Be aware that we require the -p internal option now to select the
> internal programmer (there is no implicit default programmer set at
> compilation time any more).