Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: openembedded-core@lists.openembedded.org
Cc: Denys Dmytriyenko <denys@ti.com>
Subject: Re: [PATCH 1/2] udev: update init script for conditional probing of platform bus
Date: Fri, 20 Jun 2014 13:13:38 -0400	[thread overview]
Message-ID: <20140620171338.GX27324@denix.org> (raw)
In-Reply-To: <1403218403-3303-1-git-send-email-denis@denix.org>

On Thu, Jun 19, 2014 at 06:53:22PM -0400, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
> 
> Make probing of "platform" bus conditional based on PROBE_PLATFORM_BUS
> variable from /etc/default/udev-cache on subsequent boots when udev
> cache is used. PROBE_PLATFORM_BUS has to be set to "yes" in order to
> force probing on every boot, otherwise it uses the old default behaviour
> of probing it just the first time.
> 
> This is helpful on modern SoCs where most of the low level peripheral
> drivers are registered on the platform bus and need to be probed to load
> the necessary modules and enable the connected buses and subsystems.

Any questions, comments? I would like to keep a dialog open on this one. 
Thanks.


> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  meta/recipes-core/udev/udev/init | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index 410a650..f2c84d5 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -102,7 +102,12 @@ case "$1" in
>  
>      udevadm control --env=STARTUP=1
>      if [ "$not_first_boot" != "" ];then
> -            udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus  --subsystem-nomatch=graphics	 --subsystem-nomatch=backlight --subsystem-nomatch=video4linux	--subsystem-nomatch=platform
> +            if [ "$PROBE_PLATFORM_BUS" != "yes" ]; then
> +                PLATFORM_BUS_NOMATCH="--subsystem-nomatch=platform"
> +            else
> +                PLATFORM_BUS_NOMATCH=""
> +            fi
> +            udevadm trigger --action=add --subsystem-nomatch=tty --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole --subsystem-nomatch=misc --subsystem-nomatch=dcon --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics --subsystem-nomatch=backlight --subsystem-nomatch=video4linux $PLATFORM_BUS_NOMATCH
>              (udevadm settle --timeout=3; udevadm control --env=STARTUP=)&
>      else
>              udevadm trigger --action=add
> -- 
> 2.0.0
> 
> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


  parent reply	other threads:[~2014-06-20 17:13 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-19 22:53 [PATCH 1/2] udev: update init script for conditional probing of platform bus Denys Dmytriyenko
2014-06-19 22:53 ` [PATCH 2/2] udev-cache.default: set PROBE_PLATFORM_BUS to "yes" by default Denys Dmytriyenko
2014-06-20 17:13 ` Denys Dmytriyenko [this message]
2014-06-25  0:52   ` [PATCH 1/2] udev: update init script for conditional probing of platform bus Denys Dmytriyenko
2014-06-25 16:20     ` Paul Eggleton
2014-06-25 16:22       ` Denys Dmytriyenko
2014-06-25 22:41         ` Richard Purdie
2014-06-26 17:43           ` Denys Dmytriyenko
2014-07-15  2:41             ` Denys Dmytriyenko
2014-07-15 15:30               ` Saul Wold
2014-07-15 15:38                 ` Denys Dmytriyenko
2014-07-21 22:30                   ` Denys Dmytriyenko
2014-06-26 17:27 ` Otavio Salvador

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=20140620171338.GX27324@denix.org \
    --to=denis@denix.org \
    --cc=denys@ti.com \
    --cc=openembedded-core@lists.openembedded.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox