[patch][buildrom] update geode vsa binary

[Marc Jones]

GPL VSA source is now hosted by coreboot @ svn://coreboot.org/vsa/trunk/gplvsa2
and I am hosting the binary since AMD has stopped support.
This patch updates buildrom to get the new binary.

Signed-off-by: Marc Jones

[Mart Raudsepp]

Ühel kenal päeval, K, 2009-04-08 kell 22:14, kirjutas Marc Jones:
> GPL VSA source is now hosted by coreboot @
> svn://coreboot.org/vsa/trunk/gplvsa2
> and Marc Jones is hosting the binary since AMD has stopped support.
> This patch updates buildrom to get the new binary.
>
> Signed-off-by: Marc Jones
> Index: buildrom/packages/geodevsa/amdvsa.inc
>
> --- buildrom.orig/packages/geodevsa/amdvsa.inc  2009-04-08
> 16:41:40.000000000 -0600
> +++ buildrom/packages/geodevsa/amdvsa.inc       2009-04-08
> 16:46:16.000000000 -0600
> @@ -1,7 +1,11 @@
>  # Target file for the AMD VSA binary - included from geodevsa.mk
>
> -AMDVSA_URL=http://www.amd.com/files/connectivitysolutions/geode/geode_lx
> -AMDVSA_BIN=amd_vsa_lx_1.01.bin
> +# 2009-04-08 - AMD is no longer supporting the GPL VSA.
> +# Updated binaries are provided by Marc Jones and the source
> +# is hosted by coreboot.org. svn://coreboot.org/vsa/trunk/gplvsa2
>
> +AMDVSA_URL=http://marcjonesconsulting.com/gplvsa/
> +AMDVSA_BIN=gpl_vsa_lx_102.bin
>
> # 2008-02-19 - the AMD URL above is broken, when trying to wget
>  # amd_vsa_lx_1.01.bin.gz it arrives uncompressed instead.  This

This comment should be removed or modified too then, as there is no AMD
URL above anymore.
Also we might want to get rid of the tricks played further below in the
code that were due to AMD URL having delivered the .gz in unpacked form.

Regards,
Mart Raudsepp

[Marc Jones]

On Thu, Apr 9, 2009 at 4:03 AM, Mart Raudsepp
 wrote:
> Ühel kenal päeval, K, 2009-04-08 kell 22:14, kirjutas Marc Jones:
> > GPL VSA source is now hosted by coreboot @
> > svn://coreboot.org/vsa/trunk/gplvsa2
> > and Marc Jones is hosting the binary since AMD has stopped support.
> > This patch updates buildrom to get the new binary.
> >
> > Signed-off-by: Marc Jones
> > Index: buildrom/packages/geodevsa/amdvsa.inc
> >
> > --- buildrom.orig/packages/geodevsa/amdvsa.inc  2009-04-08
> > 16:41:40.000000000 -0600
> > +++ buildrom/packages/geodevsa/amdvsa.inc       2009-04-08
> > 16:46:16.000000000 -0600
> > @@ -1,7 +1,11 @@
> >  # Target file for the AMD VSA binary - included from geodevsa.mk
> >
> > -AMDVSA_URL=http://www.amd.com/files/connectivitysolutions/geode/geode_lx
> > -AMDVSA_BIN=amd_vsa_lx_1.01.bin
> > +# 2009-04-08 - AMD is no longer supporting the GPL VSA.
> > +# Updated binaries are provided by Marc Jones and the source
> > +# is hosted by coreboot.org. svn://coreboot.org/vsa/trunk/gplvsa2
> >
> > +AMDVSA_URL=http://marcjonesconsulting.com/gplvsa/
> > +AMDVSA_BIN=gpl_vsa_lx_102.bin
> >
> > # 2008-02-19 - the AMD URL above is broken, when trying to wget
> >  # amd_vsa_lx_1.01.bin.gz it arrives uncompressed instead.  This
>
> This comment should be removed or modified too then, as there is no AMD
> URL above anymore.
> Also we might want to get rid of the tricks played further below in the
> code that were due to AMD URL having delivered the .gz in unpacked form.

updated patch attached.
Marc

[Peter Stuge]

Marc Jones wrote:
> GPL VSA source is now hosted by coreboot @ svn://coreboot.org/vsa/trunk/gplvsa2
> and Marc Jones is hosting the binary since AMD has stopped support.
> This patch updates buildrom to get the new binary.
>
> Signed-off-by: Marc Jones

Acked-by: Peter Stuge
> +++ buildrom/packages/geodevsa/amdvsa.inc	2009-04-09 15:30:54.000000000 -0600

..
> -AMDVSA_BIN=amd_vsa_lx_1.01.bin
> +AMDVSA_BIN=gpl_vsa_lx_102.bin

Is there any functional difference between the two?

//Peter

[Marc Jones]

On Thu, Apr 9, 2009 at 3:49 PM, Peter Stuge  wrote:
> Marc Jones wrote:
> > GPL VSA source is now hosted by coreboot @ svn://coreboot.org/vsa/trunk/gplvsa2
> > and Marc Jones is hosting the binary since AMD has stopped support.
> > This patch updates buildrom to get the new binary.
> >
> > Signed-off-by: Marc Jones
>
> Acked-by: Peter Stuge
> > +++ buildrom/packages/geodevsa/amdvsa.inc     2009-04-09 15:30:54.000000000 -0600
>
> ..
> > -AMDVSA_BIN=amd_vsa_lx_1.01.bin
> > +AMDVSA_BIN=gpl_vsa_lx_102.bin
>
> Is there any functional difference between the two?

Yes, It in the NAND IO stuff that we did several months ago.
Here is the one of the threads.
http://www.coreboot.org/pipermail/coreboot/2009-January/044320.html

Marc

[Peter Stuge]

Marc Jones wrote:
> > > -AMDVSA_BIN=amd_vsa_lx_1.01.bin
> > > +AMDVSA_BIN=gpl_vsa_lx_102.bin
> >
> > Is there any functional difference between the two?
>
> Yes, It in the NAND IO stuff that we did several months ago.

Ah yes - that's right. Thanks!

//Peter