[meta-intel] [PATCH 0/4] AMT feature re-enabling commits

[Tom Zanussi]

On Fri, 2013-06-07 at 10:06 -0500, Kamble, Nitin A wrote:
> 
> > -----Original Message-----
> > From: Zanussi, Tom
> > Sent: Thursday, June 06, 2013 5:07 PM
> > To: Kamble, Nitin A
> > Cc: Hart, Darren; meta-intel at yoctoproject.org
> > Subject: Re: [PATCH 0/4] AMT feature re-enabling commits
> > 
> > On Tue, 2013-06-04 at 18:29 +0000, nitin.a.kamble at intel.com wrote:
> > > From: Nitin A Kamble <nitin.a.kamble at intel.com>
> > >
> > > Here are few commits to re-enable AMT/MEI feature in the meta-intel
> > layer.
> > >
> > > The previous lms recipe is replaced by lms7 & lms8 recipes to cover
> > > boards with AMT v7 and also AMT v8 firmware.
> > >
> > 
> > Sorry, I was away at a conference and haven't been able to get to these yet,
> > but one question I had was why you're creating two new recipes withe
> > versions in the name rather than just using the normal PV for specifying
> > versions?
> 
> Tom,
>    There are 2 major versions, viz 7 & 8 of AMT/lms. And there is also further subversion. 
> The AMT firmware on boards today is either major version 7 or 8. And BSPs can specify which
> major version it needs. If the subversion of the lms recipe changes in future, then BSP need
> not change to pick up the newer version lms recipe. They automatically will get the latest subversion
> of the lms recipe with the desired major version. If not done this way, then full version need
> to be mentioned in the BSP configuration. And that will change every time the lms recipe
> version is changed.
> 

Couldn't we do something like what we do with the kernel e.g.

PREFERRED_VERSION_lms ?= "7%"

Tom

> Nitin
> 
> 
> 
> 
> > 
> > Tom
> > 
> > > The BSPs using AMT feature are updated to use the newer lms recipe
> > packages.
> > >
> > > This work has been validated with the NUC BSP on DQ77BK board.
> > >
> > > Thanks,
> > > Nitin
> > >
> > > The following changes since commit
> > c97b8ceddac999e5e993768f6b1b581c2746a561:
> > >
> > >   nuc: linux: fix AMT feature specification in recipe (2013-06-04
> > > 08:18:35 +0000)
> > >
> > > are available in the git repository at:
> > >
> > >   git://git.yoctoproject.org/meta-intel-contrib nitin/misc
> > >
> > > http://git.yoctoproject.org/cgit.cgi/meta-intel-contrib/log/?h=nitin/m
> > > isc
> > >
> > > Nitin A Kamble (4):
> > >   lms7: reorganize the lms recipe
> > >   lms8: new recipe to support AMT version 8
> > >   chiefriver: use package lms7 insted of lms for AMT
> > >   nuc: use package lms8 insted of lms for AMT
> > >
> > >  common/custom-licenses/BSD_LMS                     | 30
> > ++++++++++++++++++++++
> > >  .../{lms => files}/atnetworktool-printf-fix.patch  |  0  .../amt/{lms
> > > => files}/readlink-declaration.patch  |  0
> > >  .../amt/{lms_7.1.20.bb => lms7_7.1.20.bb}          | 20 +++++----------
> > >  common/recipes-bsp/amt/lms8_8.0.0-7.bb             | 30
> > ++++++++++++++++++++++
> > >  meta-chiefriver/conf/machine/chiefriver.conf       |  2 +-
> > >  meta-nuc/conf/machine/nuc.conf                     |  2 +-
> > >  7 files changed, 68 insertions(+), 16 deletions(-)  create mode
> > > 100755 common/custom-licenses/BSD_LMS  rename
> > > common/recipes-bsp/amt/{lms => files}/atnetworktool-printf-fix.patch
> > > (100%)  rename common/recipes-bsp/amt/{lms =>
> > > files}/readlink-declaration.patch (100%)  rename
> > > common/recipes-bsp/amt/{lms_7.1.20.bb => lms7_7.1.20.bb} (60%)  create
> > mode 100644 common/recipes-bsp/amt/lms8_8.0.0-7.bb
> > >
> > 
>