[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAJOFmgysbHqCzkW6NLnJ+o7D8F7ZGv+A2MjVYTV06=_jC_jJ+w@mail.gmail.com>
Date: Thu, 30 Jun 2016 09:24:29 -0700
From: Stephen Boyd <stephen.boyd@...aro.org>
To: Peter Chen <hzpeterchen@...il.com>
Cc: linux-usb@...r.kernel.org, Felipe Balbi <balbi@...nel.org>,
Arnd Bergmann <arnd@...db.de>,
Neil Armstrong <narmstrong@...libre.com>,
linux-arm-msm@...r.kernel.org,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Peter Chen <peter.chen@....com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Andy Gross <andy.gross@...aro.org>,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 16/21] usb: chipidea: msm: Restore wrapper settings after reset
On 30 June 2016 at 01:54, Peter Chen <hzpeterchen@...il.com> wrote:
> On Wed, Jun 29, 2016 at 12:13:45PM -0700, Stephen Boyd wrote:
>> Quoting Peter Chen (2016-06-29 01:26:48
>> > Why not using ci->platdata->vbus_extcon directly?
>>
>> Because ci->platdata->vbus_extcon is assigned after the child platform
>> driver probes, and we have no idea when that will happen from the
>> ci_hdrc_msm driver probe. If we try after ci_hdrc_add_device() we'll
>> race with the driver probe and only get the pointer sometimes.
>
> ci->platdata->vbus_extcon->edev is assigned at ci_get_platdata which is
> called before ci core device is created.
>
Ok.
Powered by blists - more mailing lists