Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <1533937924-6774-1-git-send-email-rishabhb@codeaurora.org>
In-Reply-To: <1533937924-6774-1-git-send-email-rishabhb@codeaurora.org>
From:   "Rafael J. Wysocki" <rafael@kernel.org>
Date:   Sun, 12 Aug 2018 10:26:02 +0200
Message-ID: <CAJZ5v0hRw-bkOZPLtKohW6aPYnFFsVGNQbYo9yvZPcUFzioOGQ@mail.gmail.com>
Subject: Re: [PATCH v1] dd: Invoke one probe retry cycle after some initcall levels
To:     Rishabh Bhatnagar <rishabhb@codeaurora.org>
Cc:     "Rafael J. Wysocki" <rafael@kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Sodagudi Prasad <psodagud@codeaurora.org>,
        tsoni@codeaurora.org, ckadabi@codeaurora.org,
        Vikram Mulukutla <markivx@codeaurora.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Aug 10, 2018 at 11:52 PM Rishabh Bhatnagar
<rishabhb@codeaurora.org> wrote:
>
> Drivers that are registered at an initcall level may have to
> wait until late_init before the probe deferral mechanism can
> retry their probe functions. It is possible that their
> dependencies were resolved much earlier, in some cases even
> before the next initcall level. Invoke one probe retry cycle
> at every _sync initcall level after subsys initcall, allowing
> these drivers to be probed earlier.
>
> Signed-off-by: Vikram Mulukutla <markivx@codeaurora.org>
> Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
> ---
>
> To give an example many Qualcomm drivers are dependent on the regulator and
> bus driver. Both the regulator and bus driver are probed in the
> subsys_initcall level. Now the probe of bus driver requires regulator to be
> working. If the probe of bus driver happens before regulator, then bus
> driver's probe will be deferred and all other device's probes which depend
> on bus driver will also be deferred.
> The impact of this problem is reduced if we have this patch.

Please move the above to the changelog proper (ie. above the tags) and
resend the patch with CCs to linux-pm@vger.kernel.org and LKML.

Thanks,
Rafael