[yocto] [PATCH 0/7][KERNEL] add usb and mei features
Tom Zanussi
tom.zanussi at intel.com
Fri May 4 17:06:53 PDT 2012
On Fri, 2012-05-04 at 16:04 -0700, Darren Hart wrote:
>
> On 05/04/2012 03:07 PM, tom.zanussi at intel.com wrote:
> > From: Tom Zanussi <tom.zanussi at 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
> >
>
More information about the yocto
mailing list