On Fri, 2012-05-04 at 16:04 -0700, Darren Hart wrote: > > On 05/04/2012 03:07 PM, tom.zanu...@intel.com wrote: > > From: Tom Zanussi <tom.zanu...@intel.com> > > > > 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. 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 > > > _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto