Re: [PATCH 3/6] crownbay README: add WebTitle & Compliance information

[Tom Zanussi <tom.zanussi@intel.com>]

On Wed, 2012-10-24 at 11:06 -0500, Kamble, Nitin A wrote:
> 
> > -----Original Message-----
> > From: Zanussi, Tom
> > Sent: Tuesday, October 23, 2012 2:16 PM
> > To: Kamble, Nitin A
> > Cc: yocto@yoctoproject.org; Hart, Darren
> > Subject: Re: [PATCH 3/6] crownbay README: add WebTitle & Compliance
> > information
> > 
> > On Tue, 2012-10-23 at 13:24 -0700, nitin.a.kamble@intel.com wrote:
> > > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> > >
> > > The WebTitle will be used to publish the BSP on the Yocto Project Website.
> > > And adding the Yocto Project Compliance information for the 1.3 release.
> > > Also specifying all the layers used from meta-intel repository.
> > >
> > > Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
> > > ---
> > >  meta-crownbay/README |   13 +++++++++++--
> > >  1 files changed, 11 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/meta-crownbay/README b/meta-crownbay/README index
> > > 4bc9f31..3996a94 100644
> > > --- a/meta-crownbay/README
> > > +++ b/meta-crownbay/README
> > > @@ -2,13 +2,22 @@ This README file contains information on building
> > > the meta-crownbay  BSP layer, and booting the images contained in the
> > /binary directory.
> > >  Please see the corresponding sections below for details.
> > >
> > > -The Crown Bay platform consists of the Intel Atom Z6xx processor,
> > > +The Crown Bay platform consists of the Intel Atom E6xx processor,
> > >  plus the Intel EG20T Platform Controller Hub (Tunnel Creek + Topcliff).
> > >
> > >  It also supports the E6xx embedded on-chip graphics via the Intel
> > > Embedded Media and Graphics Driver (EMGD) 1.14 Driver.
> > >
> > >
> > > +WebTitle: Intel Atom E6xx processor with Intel EG20T Controller Hub
> > > +development kit (crownbay)
> > > +
> > 
> > I'm not sure this kind of thing should be in the README since we can have
> > multiple downloadable BSPs per layer e.g. crownbay vs crownbay-noemgd.  I
> > suppose in keeping with the build system you could have separate
> > WebTitle_crownbay and WebTitle_crownbay-noemgd lines. ;-) (and it would
> > be nice if you could get rid of the CamelCaps too)
> > 
> > Why not put this info in the machine.conf, where we already have fields
> > meant to be machine parseable e.g.
> > 
> > #@TYPE: Machine
> > #@NAME: crownbay
> > 
> > #@WEBTITLE: ...
> > 
> > Or maybe just use the exisiting #@DESCRIPTION for that...
> 
> For example in the current way the BSPs are published, this is shown on the YP website for crownbay 
> 
> Intel Atom Processor E660 with Intel Platform Controller Hub EG20T Development Kit
> Version: 7.0 "Denzil"
> Release date: 29 Jun 2012
> Type: BSP
> Download Links:
> Crown Bay
> Crown Bay no EMGD
> Release Notes
> 
> So there is one BSP list item per h/w with multiple links to different BSP tarballs for the same hardware.
> If we move the WebTitle in machine file, then we will have multiple items in the BSP list for the same hardware.
> I am not sure which is better from the downloader's point of view. But this is worth considering for this change.
> 

Yeah, on the one hand if we have text that's the same for all BSPs in
the layer, it could go in the README for lack of a better common place.

But we should consider whether we want to lay things out as the crownbay
above, or more like the cedartrail, which has:

Intel® Atom™ Processor N2000 and D2000 Series-based Platform (CEDAR
TRAIL) with PowerVR Graphics

(there's no corresponding -nopvr version available, though I suspect
that's an oversight and would have been something like:

Intel® Atom™ Processor N2000 and D2000 Series-based Platform (CEDAR
TRAIL) with VESA Graphics

I'm not sure the field(s) need to map exactly to the page layout, but
all the information should be there to allow the page to be generated or
laid out by hand without having to ask questions, in either case.  Did
you have any idea as to how for example to distinguish between the emgd
and -noemgd versions (side note: there's nothing in the current entry
that tells the user what EMGD even is - should it at least be spelled
out in the title, or do we need a separate subtext element to describe
that?)

Tom

> > 
> > > +
> > > +Compliance:
> > > +============
> > 
> > For consistency with the rest of the README, please remove the colon and
> > clean up the underlining.
> 
> Will do.
> 
> Thanks,
> Nitin
> > 
> > Thanks,
> > 
> > Tom
> > 
> > > +This BSP is compliant with Yocto Project as per requirements listed here:
> > > +http://www.yoctoproject.org/yocto-project-compatible-registration
> > > +
> > > +
> > >  Dependencies
> > >  ============
> > >
> > > @@ -22,7 +31,7 @@ This layer depends on:
> > >    branch: master
> > >
> > >    URI: git://git.yoctoproject.org/meta-intel
> > > -  layers: intel
> > > +  layers: meta-intel, meta-crownbay
> > >    branch: master
> > >
> > >
> > 
>