Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-16 Thread Peter Hurley
On 04/15/2015 10:35 AM, Rob Herring wrote: On Wed, Apr 15, 2015 at 9:17 AM, Peter Hurley pe...@hurleysoftware.com wrote: 2. Should DT-specific drivers not be using irq_of_parse_and_map()? On probe failure irq_dispose_mapping() will be junking the mapping, thus invalidating the irq

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-16 Thread Rob Herring
On Thu, Apr 16, 2015 at 5:40 AM, Peter Hurley pe...@hurleysoftware.com wrote: On 04/15/2015 10:35 AM, Rob Herring wrote: On Wed, Apr 15, 2015 at 9:17 AM, Peter Hurley pe...@hurleysoftware.com wrote: 2. Should DT-specific drivers not be using irq_of_parse_and_map()? On probe failure

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-15 Thread Peter Hurley
On 04/08/2015 09:40 AM, Rob Herring wrote: This doesn't sound right. It ignores failures because platform_get_irq will parse the interrupts when called rather than just using the resource struct and will return EPROBE_DEFER if the irq resource is not ready. We left the of_device_alloc code in

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-15 Thread Rob Herring
On Wed, Apr 15, 2015 at 9:17 AM, Peter Hurley pe...@hurleysoftware.com wrote: On 04/08/2015 09:40 AM, Rob Herring wrote: This doesn't sound right. It ignores failures because platform_get_irq will parse the interrupts when called rather than just using the resource struct and will return

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-13 Thread Geert Uytterhoeven
Hi Rob, On Wed, Apr 8, 2015 at 3:40 PM, Rob Herring robherri...@gmail.com wrote: On Wed, Apr 8, 2015 at 3:42 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Thu, Apr 2, 2015 at 4:38 AM, Grant Likely grant.lik...@secretlab.ca wrote: On Mon, 30 Mar 2015 15:27:27 +0200 , Geert

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-08 Thread Pantelis Antoniou
Hi Geert, On Apr 8, 2015, at 11:42 , Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Thu, Apr 2, 2015 at 4:38 AM, Grant Likely grant.lik...@secretlab.ca wrote: On Mon, 30 Mar 2015 15:27:27 +0200 , Geert Uytterhoeven ge...@linux-m68k.org wrote: On Tue, Mar 24, 2015 at 6:56

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-08 Thread Geert Uytterhoeven
Hi Grant, On Thu, Apr 2, 2015 at 4:38 AM, Grant Likely grant.lik...@secretlab.ca wrote: On Mon, 30 Mar 2015 15:27:27 +0200 , Geert Uytterhoeven ge...@linux-m68k.org wrote: On Tue, Mar 24, 2015 at 6:56 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 24, 2015, at 07:50 ,

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-08 Thread Rob Herring
On Wed, Apr 8, 2015 at 9:43 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Wed, Apr 8, 2015 at 4:17 PM, Rob Herring robherri...@gmail.com wrote: Then we have dma controllers needing to be registered at subsys_initcall() for obvious reasons. These should be fixable with deferred probe.

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-08 Thread Rob Herring
On Wed, Apr 8, 2015 at 3:42 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Thu, Apr 2, 2015 at 4:38 AM, Grant Likely grant.lik...@secretlab.ca wrote: On Mon, 30 Mar 2015 15:27:27 +0200 , Geert Uytterhoeven ge...@linux-m68k.org wrote: On Tue, Mar 24, 2015 at 6:56 PM,

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-08 Thread Rob Herring
On Wed, Apr 8, 2015 at 4:16 AM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: Hi Geert, On Apr 8, 2015, at 11:42 , Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Thu, Apr 2, 2015 at 4:38 AM, Grant Likely grant.lik...@secretlab.ca wrote: On Mon, 30 Mar 2015 15:27:27

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-08 Thread Geert Uytterhoeven
On Wed, Apr 8, 2015 at 4:17 PM, Rob Herring robherri...@gmail.com wrote: Then we have dma controllers needing to be registered at subsys_initcall() for obvious reasons. These should be fixable with deferred probe. That requires fixing all the client drivers where as fixing the dependencies

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-04-02 Thread Pantelis Antoniou
Hi Grant, On Apr 2, 2015, at 05:38 , Grant Likely grant.lik...@secretlab.ca wrote: On Mon, 30 Mar 2015 15:27:27 +0200 , Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Pantelis, On Tue, Mar 24, 2015 at 6:56 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 24, 2015,

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-30 Thread Geert Uytterhoeven
Hi Pantelis, On Tue, Mar 24, 2015 at 6:56 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 24, 2015, at 07:50 , Geert Uytterhoeven ge...@linux-m68k.org wrote: On Fri, Mar 20, 2015 at 12:39 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 19, 2015, at 21:18

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-30 Thread Pantelis Antoniou
Hi Geert, On Mar 30, 2015, at 16:27 , Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Pantelis, On Tue, Mar 24, 2015 at 6:56 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 24, 2015, at 07:50 , Geert Uytterhoeven ge...@linux-m68k.org wrote: On Fri, Mar 20, 2015 at

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-24 Thread Geert Uytterhoeven
Hi Pantelis, Grant, On Fri, Mar 20, 2015 at 12:39 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 19, 2015, at 21:18 , Grant Likely grant.lik...@secretlab.ca wrote: On Tue, 16 Dec 2014 14:11:31 +0200 , Pantelis Antoniou pantelis.anton...@konsulko.com wrote: A nice

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-24 Thread Pantelis Antoniou
Hi Geert, On Mar 24, 2015, at 07:50 , Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Pantelis, Grant, On Fri, Mar 20, 2015 at 12:39 PM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On Mar 19, 2015, at 21:18 , Grant Likely grant.lik...@secretlab.ca wrote: On Tue, 16 Dec

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-20 Thread Grant Likely
On Tue, 16 Dec 2014 14:11:31 +0200 , Pantelis Antoniou pantelis.anton...@konsulko.com wrote: A nice side-effect of the changes in DTC for supporting overlays is that it is now possible to do dependency tracking of platform devices automatically. This patch implements dependency aware probe

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-20 Thread Pantelis Antoniou
Hi Grant, On Mar 19, 2015, at 21:18 , Grant Likely grant.lik...@secretlab.ca wrote: On Tue, 16 Dec 2014 14:11:31 +0200 , Pantelis Antoniou pantelis.anton...@konsulko.com wrote: A nice side-effect of the changes in DTC for supporting overlays is that it is now possible to do dependency

Re: [PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2015-03-20 Thread Grant Likely
On Fri, Mar 20, 2015 at 11:39 AM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: Hi Grant, On Mar 19, 2015, at 21:18 , Grant Likely grant.lik...@secretlab.ca wrote: On Tue, 16 Dec 2014 14:11:31 +0200 , Pantelis Antoniou pantelis.anton...@konsulko.com wrote: A nice side-effect of

[PATCH] [RFC] OF: probe order dependency aware of_platform_populate

2014-12-16 Thread Pantelis Antoniou
A nice side-effect of the changes in DTC for supporting overlays is that it is now possible to do dependency tracking of platform devices automatically. This patch implements dependency aware probe order for users of of_platform_populate. There are no changes in the syntax of the DTS bindings,