On 10/24/2012 09:43 AM, Tom Zanussi wrote: > 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.kam...@intel.com wrote: >>>> From: Nitin A Kamble <nitin.a.kam...@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.kam...@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?)
I like the Cedar Trail description with the graphics mentioned explicitly. Doing this in the machine conf as Tom original suggested makes perfect sense to me. If that means two entries for some machines, that's preferable to me than extra complexity of subtexts, etc. -- Darren Hart Intel Open Source Technology Center Yocto Project - Linux Kernel _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto
