Re: [PATCH 3/3] u-boot-am33x: Update u-boot recipe to 2013.01.01

[Denys Dmytriyenko <denys@ti.com>]

On Mon, Apr 22, 2013 at 03:58:23PM -0400, Cooper Jr., Franklin wrote:
> 
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Monday, April 22, 2013 2:30 PM
> > To: Cooper Jr., Franklin
> > Cc: meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] [PATCH 3/3] u-boot-am33x: Update u-boot recipe to
> > 2013.01.01
> > 
> > On Mon, Apr 22, 2013 at 11:21:42AM -0500, Franklin S. Cooper Jr wrote:
> > > * Update to latest version of u-boot that supports the 3.2 kernel.
> > >
> > > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> > > ---
> > >  recipes-bsp/u-boot/u-boot-am33x_2012.10.bb    |   17 -----------------
> > >  recipes-bsp/u-boot/u-boot-am33x_2013.01.01.bb |   20
> > ++++++++++++++++++++
> > >  2 files changed, 20 insertions(+), 17 deletions(-)  delete mode
> > > 100644 recipes-bsp/u-boot/u-boot-am33x_2012.10.bb
> > >  create mode 100644 recipes-bsp/u-boot/u-boot-am33x_2013.01.01.bb
> > >
> > > diff --git a/recipes-bsp/u-boot/u-boot-am33x_2012.10.bb
> > > b/recipes-bsp/u-boot/u-boot-am33x_2012.10.bb
> > > deleted file mode 100644
> > > index 7320e02..0000000
> > > --- a/recipes-bsp/u-boot/u-boot-am33x_2012.10.bb
> > > +++ /dev/null
> > > @@ -1,17 +0,0 @@
> > > -require u-boot-ti.inc
> > > -
> > > -DESCRIPTION = "u-boot bootloader for ARM MPU devices"
> > > -
> > > -COMPATIBLE_MACHINE = "ti33x|omap3"
> > 
> > You are removing the old 2012.10 recipe that is also used for "omap3" SoCs.
> > Since you are adding the new one only for "ti33x" SoCs, there won't be any
> > compatible u-boot for AM37x...
> 
> The COMPATIBLE_MACHINE is incorrect. It really should only be ti33x. 
> 
> AM37x is using u-boot-am37x_2012.04.01 anyway.
> I don't know why am3517 is using this u-boot.
> I will need to add the proper 3517 u-boot recipe.

Ah, right, AM3517, not sure why I said AM37x :)

Ok, as long as you will be submitting the updated recipe for it, I'll go 
ahead and push this set.

-- 
Denys


> > > -
> > > -DEFAULT_PREFERENCE = "-1"
> > > -
> > > -PR = "r7+gitr${SRCPV}"
> > > -
> > > -SRC_URI = "git://arago-project.org/git/projects/u-boot-
> > am33x.git;protocol=git;branch=${BRANCH}"
> > > -
> > > -BRANCH = "amsdk-05.06.00.00"
> > > -
> > > -SRCREV = "6536bfa6b7f16bb49fdea8c72959e3ed54e77a82"
> > > -
> > > -SPL_BINARY = "MLO"
> > > diff --git a/recipes-bsp/u-boot/u-boot-am33x_2013.01.01.bb b/recipes-
> > bsp/u-boot/u-boot-am33x_2013.01.01.bb
> > > new file mode 100644
> > > index 0000000..19ec34e
> > > --- /dev/null
> > > +++ b/recipes-bsp/u-boot/u-boot-am33x_2013.01.01.bb
> > > @@ -0,0 +1,20 @@
> > > +require u-boot-ti.inc
> > > +
> > > +DESCRIPTION = "u-boot bootloader for ARM MPU devices"
> > > +
> > > +COMPATIBLE_MACHINE = "ti33x"
> > > +
> > > +DEFAULT_PREFERENCE = "-1"
> > > +
> > > +PR = "r5+gitr${SRCPV}"
> > > +
> > > +SRC_URI = "git://git.ti.com/ti-u-boot/ti-u-
> > boot.git;protocol=git;branch=${BRANCH}"
> > > +
> > > +# This version of u-boot is meant for 3.2 kernel which doesn't support
> > device tree.
> > > +BRANCH = "ti-u-boot-2013.01.01-amsdk-05.07.00.00"
> > > +
> > > +# Commit corresponds to tag "v2013.01.01_amsdk-05.07.00.00"
> > > +SRCREV = "8eb15a787c558fee98b0fa2a66ff0849c732edcc"
> > > +
> > > +# Set the name of the SPL that will built so that it is also packaged with u-
> > boot.
> > > +SPL_BINARY = "MLO"
> > > --
> > > 1.7.0.4
> > >
> > > _______________________________________________
> > > meta-ti mailing list
> > > meta-ti@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-ti
> > >