All of lore.kernel.org
 help / color / mirror / Atom feed
From: Darren Hart <dvhart@linux.intel.com>
To: Hollis Blanchard <hollis_blanchard@mentor.com>
Cc: "yocto@yoctoproject.org" <yocto@yoctoproject.org>
Subject: Re: [PATCH] include usbinit package with usbgadget feature
Date: Thu, 01 Mar 2012 14:45:11 -0800	[thread overview]
Message-ID: <4F4FFBF7.3020208@linux.intel.com> (raw)
In-Reply-To: <4F4FF77D.9060607@mentor.com>



On 03/01/2012 02:26 PM, Hollis Blanchard wrote:
> The g_ether kernel module, in the kernel-module-g-ether package, allows 
> targets to use a USB connection as a network interface. However, that's 
> not useful until the module is loaded.
> 
> The usbinit package provides the /etc/init.d/usb-gether script to load 
> the g_ether module at boot, so it should be included along with 
> kernel-module-g-ether.

So if the g-ether kernel module is built-in to the kernel, the
kernel-module-g-ether will not be installed (as it's RRECOMMENDS). The
usbinit package still will be (unless for some reason it can't be built
for the target).

I assume this doesn't pose a problem?

This does mean usbinit would be installed when it isn't strictly
necessary - but that's better than not installing it if it's required.
Looks good to me.

--
Darren



> 
> Tested with core-image-base on beagleboard.
> 
> Signed-off-by: Hollis Blanchard <hollis_blanchard@mentor.com>
> ---
> 
> diff --git a/meta/recipes-core/tasks/task-base.bb b/meta/recipes-core/tasks/task-base.bb
> index 2032432..fda0384 100644
> --- a/meta/recipes-core/tasks/task-base.bb
> +++ b/meta/recipes-core/tasks/task-base.bb
> @@ -2,7 +2,7 @@ DESCRIPTION = "Merge machine and distro options to create a basic machine task/p
>   LICENSE = "MIT"
>   LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58 \
>                       file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
> -PR = "r73"
> +PR = "r74"
> 
>   inherit task
> 
> @@ -248,7 +248,8 @@ RRECOMMENDS_task-base-usbgadget = "\
>       kernel-module-gadgetfs \
>       kernel-module-g-file-storage \
>       kernel-module-g-serial \
> -    kernel-module-g-ether"
> +    kernel-module-g-ether \
> +    usbinit"
> 
>   RDEPENDS_task-base-usbhost = "\
>       usbutils "
> 
> 
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel


  reply	other threads:[~2012-03-01 22:45 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-01 22:26 [PATCH] include usbinit package with usbgadget feature Hollis Blanchard
2012-03-01 22:45 ` Darren Hart [this message]
2012-03-01 23:01   ` Hollis Blanchard
2012-03-01 23:02     ` Darren Hart
2012-03-06 21:36 ` Hollis Blanchard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F4FFBF7.3020208@linux.intel.com \
    --to=dvhart@linux.intel.com \
    --cc=hollis_blanchard@mentor.com \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.