Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-15 Thread Tomeu Vizoso
On 11 September 2015 at 17:35, Mark Brown wrote: > On Fri, Sep 11, 2015 at 04:06:07PM +0200, Tomeu Vizoso wrote: > >> Once a platform device (with the platform bus as its parent) is >> retrieved from the deferred queue, both the parent and the device in >> question are locked (because of the USB

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-15 Thread Tomeu Vizoso
On 11 September 2015 at 17:35, Mark Brown wrote: > On Fri, Sep 11, 2015 at 04:06:07PM +0200, Tomeu Vizoso wrote: > >> Once a platform device (with the platform bus as its parent) is >> retrieved from the deferred queue, both the parent and the device in >> question are locked

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-11 Thread Mark Brown
On Fri, Sep 11, 2015 at 04:06:07PM +0200, Tomeu Vizoso wrote: > Once a platform device (with the platform bus as its parent) is > retrieved from the deferred queue, both the parent and the device in > question are locked (because of the USB stuff mentioned below). If > that device depends on

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-11 Thread Tomeu Vizoso
On 11 September 2015 at 11:57, Mark Brown wrote: > On Mon, Sep 07, 2015 at 02:31:06PM +0200, Tomeu Vizoso wrote: >> On 11 August 2015 at 11:37, Tomeu Vizoso wrote: >> > On 7 August 2015 at 14:19, Mark Brown wrote: > >> >> This sounds like it's going to break in the case where we have MFDs that

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-11 Thread Mark Brown
On Mon, Sep 07, 2015 at 02:31:06PM +0200, Tomeu Vizoso wrote: > On 11 August 2015 at 11:37, Tomeu Vizoso wrote: > > On 7 August 2015 at 14:19, Mark Brown wrote: > >> This sounds like it's going to break in the case where we have MFDs that > >> represent their functions in DT (not a pattern I'm

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-11 Thread Mark Brown
On Mon, Sep 07, 2015 at 02:31:06PM +0200, Tomeu Vizoso wrote: > On 11 August 2015 at 11:37, Tomeu Vizoso wrote: > > On 7 August 2015 at 14:19, Mark Brown wrote: > >> This sounds like it's going to break in the case where we have MFDs that > >>

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-11 Thread Tomeu Vizoso
On 11 September 2015 at 11:57, Mark Brown wrote: > On Mon, Sep 07, 2015 at 02:31:06PM +0200, Tomeu Vizoso wrote: >> On 11 August 2015 at 11:37, Tomeu Vizoso wrote: >> > On 7 August 2015 at 14:19, Mark Brown wrote: > >> >> This

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-11 Thread Mark Brown
On Fri, Sep 11, 2015 at 04:06:07PM +0200, Tomeu Vizoso wrote: > Once a platform device (with the platform bus as its parent) is > retrieved from the deferred queue, both the parent and the device in > question are locked (because of the USB stuff mentioned below). If > that device depends on

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-07 Thread Tomeu Vizoso
On 11 August 2015 at 11:37, Tomeu Vizoso wrote: > On 7 August 2015 at 14:19, Mark Brown wrote: >> On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: >> >>> Walks the OF tree up and finds the closest ancestor that has a platform >>> device associated with it, probing it if isn't bound

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-07 Thread Tomeu Vizoso
On 11 August 2015 at 11:37, Tomeu Vizoso wrote: > On 7 August 2015 at 14:19, Mark Brown wrote: >> On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: >> >>> Walks the OF tree up and finds the closest ancestor that has a platform >>>

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-08-11 Thread Tomeu Vizoso
On 7 August 2015 at 14:19, Mark Brown wrote: > On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: > >> Walks the OF tree up and finds the closest ancestor that has a platform >> device associated with it, probing it if isn't bound to a driver yet. > >> The above should ensure that the

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-08-11 Thread Tomeu Vizoso
On 7 August 2015 at 14:19, Mark Brown broo...@kernel.org wrote: On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: Walks the OF tree up and finds the closest ancestor that has a platform device associated with it, probing it if isn't bound to a driver yet. The above should ensure

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-08-07 Thread Mark Brown
On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: > Walks the OF tree up and finds the closest ancestor that has a platform > device associated with it, probing it if isn't bound to a driver yet. > The above should ensure that the dependency represented by the passed OF > node is

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-08-07 Thread Mark Brown
On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: Walks the OF tree up and finds the closest ancestor that has a platform device associated with it, probing it if isn't bound to a driver yet. The above should ensure that the dependency represented by the passed OF node is

[PATCH v3 02/18] of/platform: add of_platform_probe

2015-08-06 Thread Tomeu Vizoso
Walks the OF tree up and finds the closest ancestor that has a platform device associated with it, probing it if isn't bound to a driver yet. The above should ensure that the dependency represented by the passed OF node is available, because probing a platform device should cause its descendants

[PATCH v3 02/18] of/platform: add of_platform_probe

2015-08-06 Thread Tomeu Vizoso
Walks the OF tree up and finds the closest ancestor that has a platform device associated with it, probing it if isn't bound to a driver yet. The above should ensure that the dependency represented by the passed OF node is available, because probing a platform device should cause its descendants