On Thu, 2012-05-03 at 08:40 -0400, Bruce Ashfield wrote: > On 12-05-02 11:35 PM, tom.zanu...@intel.com wrote: > > From: Tom Zanussi<tom.zanu...@intel.com> > > > > This adds a few new features, one for xhci-hcd and another for > > amt/mei, and refactors some existing config options into a new > > echi-hcd, which is then used in crownbay. > > > > If this looks like the way to go, I'll add similar USB features > > for ohci and uhci and fix up all the meta-intel BSPs to use > > them. > > I had a look, and while at first I thought it was perhaps an over > splitting and categorization. It does make things very clear, and > gets us a split that can be used to keep configs minimal and reusable. > > I also wasn't sure about directory splitting, since we do end up > with the names both in .scc/.cfg and the directory name. We could > flatten the directory down to just 'usb' and keep the names of the > files as the differentiator. And if we don't think we'll have to > carry any patches, we could put it under cfg/usb/<Tom's stuff>. > > I don't have a really strong opinion (but gave my preference) on this > split (several directories vs single), and I'd bet that you considered > the same thing. Comments ? >
The directory splitting is definitely a result of personal preference on my part i.e. directories are cheap and I hate having bunches of files in a single directory. This is actually the reason I tend to avoid /cfg. ;-) In this case and most others, I find the directory splitting maps more cleanly 'at-a-glance' for me to the split in functionality, but as I said it's a personal preference and if most people prefer a more flattened tree, then I don't have a problem making that change... Tom > But definitely, this is the way to go, just a minor question about the > organization of the files. > > Bruce > > > > > > 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 (5): > > meta: add usb/host/base feature > > meta: add usb/xhci-hcd feature > > meta: add usb/ehci-hcd feature > > meta/crownbay: use ehci-hcd feature > > meta: add mei feature > > > > meta/cfg/kernel-cache/bsp/crownbay/crownbay.cfg | 4 ---- > > meta/cfg/kernel-cache/bsp/crownbay/crownbay.scc | 2 ++ > > meta/cfg/kernel-cache/features/amt/mei/mei.cfg | 3 +++ > > meta/cfg/kernel-cache/features/amt/mei/mei.scc | 4 ++++ > > .../features/usb/ehci-hcd/ehci-hcd.cfg | 1 + > > .../features/usb/ehci-hcd/ehci-hcd.scc | 6 ++++++ > > meta/cfg/kernel-cache/features/usb/host/base.cfg | 3 +++ > > meta/cfg/kernel-cache/features/usb/host/base.scc | 4 ++++ > > .../features/usb/xhci-hcd/xhci-hcd.cfg | 1 + > > .../features/usb/xhci-hcd/xhci-hcd.scc | 6 ++++++ > > 10 files changed, 30 insertions(+), 4 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/ehci-hcd/ehci-hcd.cfg > > create mode 100644 > > meta/cfg/kernel-cache/features/usb/ehci-hcd/ehci-hcd.scc > > create mode 100644 meta/cfg/kernel-cache/features/usb/host/base.cfg > > create mode 100644 meta/cfg/kernel-cache/features/usb/host/base.scc > > create mode 100644 > > meta/cfg/kernel-cache/features/usb/xhci-hcd/xhci-hcd.cfg > > create mode 100644 > > meta/cfg/kernel-cache/features/usb/xhci-hcd/xhci-hcd.scc > > > _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto