From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 78DAFE0044D for ; Fri, 4 May 2012 18:00:41 -0700 (PDT) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga102.jf.intel.com with ESMTP; 04 May 2012 18:00:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.67,351,1309762800"; d="scan'208";a="140312711" Received: from unknown (HELO envy.home) ([10.255.12.74]) by orsmga002.jf.intel.com with ESMTP; 04 May 2012 18:00:41 -0700 Message-ID: <4FA47B9A.1090109@linux.intel.com> Date: Fri, 04 May 2012 18:00:10 -0700 From: Darren Hart User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120424 Thunderbird/12.0 MIME-Version: 1.0 To: Tom Zanussi References: <4FA4608D.3000803@linux.intel.com> <1336176413.21923.23.camel@elmorro> In-Reply-To: <1336176413.21923.23.camel@elmorro> X-Enigmail-Version: 1.4.1 Cc: yocto@yoctoproject.org Subject: Re: [PATCH 0/7][KERNEL] add usb and mei features X-BeenThere: yocto@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Discussion of all things Yocto List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 05 May 2012 01:00:41 -0000 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 05/04/2012 05:06 PM, Tom Zanussi wrote: > On Fri, 2012-05-04 at 16:04 -0700, Darren Hart wrote: >> >> On 05/04/2012 03:07 PM, tom.zanussi@intel.com wrote: >>> From: Tom Zanussi >>> >>> This adds a few new features, one each for usb/[xeou]hci-hcd and >>> another for amt/mei, and refactors existing config options into a new >>> usb/base, which are then used in crownbay. >>> >>> v2: after suggestions from Bruce and Darren of a preference for a more >>> flattened feature tree structure, moved all the features into a single >>> usb/ feature dir. I did however retain the common 'base' feature >>> because inlining that in each feature would mean that the same common >>> set would be repeated in multiple config fragments if more than one usb >>> feature was included in a BSP. >> >> As things stand now, a BSP would have to include both the base and the >> *hcd right? >> >> include usb/base.scc >> include ehci-hcd.scc >> >> Seems to me we should manage the dependencies wherever we can. When I've >> created fragments I've been including their requirements as well to make >> it easier on the integrator. Since you've put the dependencies in their >> own fragment, why not include the fragment in the *hcd.scc files? >> >> +++ b/meta/cfg/kernel-cache/features/usb/ehci-hcd.scc >> @@ -0,0 +1,4 @@ >> +define KFEATURE_DESCRIPTION "Enable options for ehci (USB 2.0)" >> +define KFEATURE_COMPATIBILITY board >> + >> + include base.scc >> +kconf hardware ehci-hcd.cfg >> >> Now if someone need only include the one line for the *hcd. Since it's >> only a total of 3 lines of CONFIG settings we're packaging up here, it >> seems to me we should be able to enable it with only 1 line of meta data. >> > > Yes, this is exactly what I did in the previous patchset. > > The problem is that if we do this for each of the *hcd fragments, we end > up with the options in base.cfg repeated for each, which may or may not > produce a config-check warning. "May or may not"? If our tools complain about setting a config to the same value, I would prefer to address that in the tools. Bruce, any comment on the intended behavior of the tools? -- Darren > > To avoid that, we need to be able to add the base.cfg items once > followed by the individual *hcd items as I did with the crownbay > example: > > +include features/usb/base.scc > +include features/usb/ehci-hcd.scc > +include features/usb/ohci-hcd.scc > +include features/usb/uhci-hcd.scc > > > Tom > >> -- >> Darren >> >>> >>> Please pull into linux-yocto-3.2. >>> >>> Thanks, >>> >>> Tom >>> >>> The following changes since commit b14a08f5c7b469a5077c10942f4e1aec171faa9d: >>> Yang Shi (1): >>> meta: Clean up BSPs kernel config >>> >>> are available in the git repository at: >>> >>> git://git.yoctoproject.org/linux-yocto-2.6.37-contrib.git tzanussi/xhcd-mei-features >>> http://git.yoctoproject.org/cgit.cgi//log/?h=tzanussi/xhcd-mei-features >>> >>> Tom Zanussi (7): >>> meta: add usb/base feature >>> meta: add usb/xhci-hcd feature >>> meta: add usb/ehci-hcd feature >>> meta: add usb/ohci-hcd feature >>> meta: add usb/uhci-hcd feature >>> meta/crownbay: use usb features >>> meta: add mei feature >>> >>> meta/cfg/kernel-cache/bsp/crownbay/crownbay.cfg | 6 ------ >>> meta/cfg/kernel-cache/bsp/crownbay/crownbay.scc | 5 +++++ >>> meta/cfg/kernel-cache/features/amt/mei/mei.cfg | 3 +++ >>> meta/cfg/kernel-cache/features/amt/mei/mei.scc | 4 ++++ >>> meta/cfg/kernel-cache/features/usb/base.cfg | 3 +++ >>> meta/cfg/kernel-cache/features/usb/base.scc | 4 ++++ >>> meta/cfg/kernel-cache/features/usb/ehci-hcd.cfg | 1 + >>> meta/cfg/kernel-cache/features/usb/ehci-hcd.scc | 4 ++++ >>> meta/cfg/kernel-cache/features/usb/ohci-hcd.cfg | 1 + >>> meta/cfg/kernel-cache/features/usb/ohci-hcd.scc | 4 ++++ >>> meta/cfg/kernel-cache/features/usb/uhci-hcd.cfg | 1 + >>> meta/cfg/kernel-cache/features/usb/uhci-hcd.scc | 4 ++++ >>> meta/cfg/kernel-cache/features/usb/xhci-hcd.cfg | 1 + >>> meta/cfg/kernel-cache/features/usb/xhci-hcd.scc | 4 ++++ >>> 14 files changed, 39 insertions(+), 6 deletions(-) >>> create mode 100644 meta/cfg/kernel-cache/features/amt/mei/mei.cfg >>> create mode 100644 meta/cfg/kernel-cache/features/amt/mei/mei.scc >>> create mode 100644 meta/cfg/kernel-cache/features/usb/base.cfg >>> create mode 100644 meta/cfg/kernel-cache/features/usb/base.scc >>> create mode 100644 meta/cfg/kernel-cache/features/usb/ehci-hcd.cfg >>> create mode 100644 meta/cfg/kernel-cache/features/usb/ehci-hcd.scc >>> create mode 100644 meta/cfg/kernel-cache/features/usb/ohci-hcd.cfg >>> create mode 100644 meta/cfg/kernel-cache/features/usb/ohci-hcd.scc >>> create mode 100644 meta/cfg/kernel-cache/features/usb/uhci-hcd.cfg >>> create mode 100644 meta/cfg/kernel-cache/features/usb/uhci-hcd.scc >>> create mode 100644 meta/cfg/kernel-cache/features/usb/xhci-hcd.cfg >>> create mode 100644 meta/cfg/kernel-cache/features/usb/xhci-hcd.scc >>> >> > > -- Darren Hart Intel Open Source Technology Center Yocto Project - Linux Kernel