On 12-05-03 09:50 AM, Tom Zanussi wrote:
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. ;-)
Aha! A valid point :)
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...
Nope. I'd rather not have you re-do it for just that minor change, when
there's a valid reason on both sides.
I tossed the email .. and something has happened to my IMAP connection,
so I can't find them now. Can you resend just the pull request ? or
just the patches to me.
We can wait to see if Darren has a strong opinion one way or the other
as well.
Bruce
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
