linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: tom.leiming@gmail.com (Ming Lei)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/5] drivercore: Add driver probe deferral mechanism
Date: Tue, 11 Oct 2011 20:29:18 +0800	[thread overview]
Message-ID: <CACVXFVMnt-ChUDhawhoyMm-wJ8xSQAWs1c8ykzmyshfX49HEsw@mail.gmail.com> (raw)
In-Reply-To: <361419345.198335.1318268242916.JavaMail.root@zimbra-prod-mbox-2.vmware.com>

On Tue, Oct 11, 2011 at 1:37 AM, Andrei Warkentin <awarkentin@vmware.com> wrote:
> Hi,
>
> ----- Original Message -----
>> From: "Greg KH" <greg@kroah.com>
>> To: "Josh Triplett" <josh@joshtriplett.org>
>> Cc: "G, Manjunath Kondaiah" <manjugk@ti.com>, linux-arm-kernel at lists.infradead.org, "Grant Likely"
>> <grant.likely@secretlab.ca>, linux-omap at vger.kernel.org, linux-mmc at vger.kernel.org, linux-kernel at vger.kernel.org,
>> "Dilan Lee" <dilee@nvidia.com>, "Mark Brown" <broonie@opensource.wolfsonmicro.com>, Manjunath at jasper.es
>> Sent: Saturday, October 8, 2011 11:55:02 AM
>> Subject: Re: [PATCH 2/5] drivercore: Add driver probe deferral mechanism
>>
>
> I'm a bit of a fly on the wall here, but I'm curious how this impacts suspend/resume.
> device_initialize->device_pm_init are called from device_register, so certainly this
> patch doesn't also ensure that the PM ordering matches probe ordering, which is bound
> to break suspend, right? Was this ever tested with the OMAP target? Shouldn't the

Inside device_add(), device_pm_add is called before bus_probe_device,
so the patch can't change the device order in pm list, and just change
the driver probe order.

> PM change be also part of this patch set? I don't see why you would want to have this in
> without the PM changes.
>


thanks,
-- 
Ming Lei

  reply	other threads:[~2011-10-11 12:29 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1317963790-29426-1-git-send-email-manjugk@ti.com>
     [not found] ` <1317963790-29426-2-git-send-email-manjugk@ti.com>
2011-10-07  6:43   ` [PATCH 1/5] drivercore: add new error value for deferred probe Greg KH
2011-10-07 10:00     ` Mark Brown
2011-10-07 22:12     ` Grant Likely
2011-10-07 23:28       ` Valdis.Kletnieks at vt.edu
2011-10-08  0:12         ` Greg KH
2011-10-09 22:59           ` Grant Likely
2011-10-10  1:06             ` Greg KH
2011-10-12  6:18               ` G, Manjunath Kondaiah
2011-10-13  4:10                 ` Grant Likely
     [not found] ` <1317963790-29426-3-git-send-email-manjugk@ti.com>
2011-10-07  6:49   ` [PATCH 2/5] drivercore: Add driver probe deferral mechanism Greg KH
2011-10-07 20:57     ` Josh Triplett
2011-10-07 21:23       ` Greg KH
2011-10-08  4:03         ` Josh Triplett
2011-10-08 15:55           ` Greg KH
2011-10-08 18:18             ` Josh Triplett
2011-10-10 17:37             ` Andrei Warkentin
2011-10-11 12:29               ` Ming Lei [this message]
2011-10-13  4:09                 ` Grant Likely
2011-10-13 14:18                   ` Ming Lei
2011-10-13 14:31                     ` Alan Stern
2011-10-13 15:21                       ` Ming Lei
2011-10-13 16:04                         ` Alan Stern
2011-10-14  0:13                           ` Ming Lei
2011-10-13 17:15                       ` Grant Likely
2011-10-13 18:16                         ` Alan Stern
2011-10-13 18:28                           ` Grant Likely
2011-10-14 15:39                             ` Alan Stern
2011-10-14 16:17                               ` Grant Likely
2011-10-14 16:33                                 ` Alan Stern
2011-10-14 17:20                                   ` Grant Likely
2011-10-14 17:33                                     ` Alan Stern
2011-10-14 18:25                                       ` Grant Likely
2011-10-14 18:39                                         ` Alan Stern
2011-10-14 19:07                                           ` Grant Likely
2011-10-14 18:56                                     ` David Daney
2011-10-14 19:03                                       ` Grant Likely
2011-10-14 19:09                                         ` David Daney
2011-10-14 15:37                         ` Alan Stern
2011-10-12  7:04               ` G, Manjunath Kondaiah
2011-10-07 21:28     ` Grant Likely
2011-10-07  6:50 ` [PATCH 0/5] Driver Probe Deferral Mechanism Greg KH
2011-10-07  7:37   ` G, Manjunath Kondaiah
     [not found] ` <1317963790-29426-5-git-send-email-manjugk@ti.com>
2011-10-07 10:06   ` [PATCH 4/5] gpiolib: handle deferral probe error Alan Cox
2011-10-07 22:09     ` Grant Likely
2011-10-12  6:14       ` G, Manjunath Kondaiah
2011-10-13  4:12         ` Grant Likely

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=CACVXFVMnt-ChUDhawhoyMm-wJ8xSQAWs1c8ykzmyshfX49HEsw@mail.gmail.com \
    --to=tom.leiming@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).