[meta-xilinx] [meta-xilinx-tools][PATCH] xilinx-bootbin: Convert to recipe instead of bbclass

[Manjukumar Harthikote Matha]

Hi Jean,

> -----Original Message-----
> From: Jean-Francois Dagenais [mailto:jeff.dagenais at gmail.com]
> Sent: Thursday, January 04, 2018 6:22 AM
> To: Manjukumar Harthikote Matha <MANJUKUM at xilinx.com>
> Cc: meta-xilinx at yoctoproject.org
> Subject: Re: [meta-xilinx] [meta-xilinx-tools][PATCH] xilinx-bootbin: Convert to
> recipe instead of bbclass
> 
> Hi Manju!
> 
> It's great to hear back about this topic. FYI I have been running my version of the
> patch
> since the day I sent it without a hitch.

Thanks :)

With changes to meta-xilinx-bsp layer, it makes easier to add provider switch between this recipe and SPL.

> 
> See my comments below...
> 
> > On Jan 3, 2018, at 19:15, Manjukumar Matha <manjukumar.harthikote-
> matha at xilinx.com> wrote:
> >
> > From: Cyril Chemparathy <cyril.chemparathy at xilinx.com>
> >
> > boot.bin should be deployed either using SPL methodology or using
> > bootgen. Change the bbclass to recipe so that it can be used as recipe
> > with a provider setting.
> >
> > This patch is based of the inital patch sent to mailing list by
> > Jean-Francois Dagenais
> >
<.....>
> > new file mode 100644
> > index 0000000..b18c924
> > --- /dev/null
> > +++ b/recipes-bsp/bootbin/xilinx-bootbin_1.0.bb
> > @@ -0,0 +1,103 @@
> > +SUMMARY = "Generates boot.bin using bootgen tool"
> > +DESCRIPTION = "Manages task dependencies and creation of boot.bin. Use the \
> > +BIF_PARTITION_xyz global variables and flags to determine what makes it into \
> > +the image."
> > +
> > +LICENSE = "BSD"
> > +
> > +include machine-xilinx-${SOC_FAMILY}.inc
> 
> Was it intentional to leave out the sections I had near here to remove useless
> inherited
> tasks and add proper deploy task support?
> 
> Looked like this:
> 
> inherit nopackages base deploy
> # this recipe is almost like an image recipe, taken from image.bbclass:
> deltask do_fetch
> deltask do_unpack
> deltask do_patch
> do_install[noexec] = "1"

I would like to keep packaging, will send a v2 to skip tasks as you have mentioned below. 

Thanks,
Manju