Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-21 Thread Ray Jui
On 10/21/2014 3:45 AM, Vinod Koul wrote: On Fri, Oct 17, 2014 at 06:39:41PM +0200, Lars-Peter Clausen wrote: On 10/17/2014 06:18 PM, Ray Jui wrote: On 10/17/2014 4:15 AM, Lars-Peter Clausen wrote: On 10/17/2014 09:35 AM, Vinod Koul wrote: On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-21 Thread Vinod Koul
On Fri, Oct 17, 2014 at 06:39:41PM +0200, Lars-Peter Clausen wrote: > On 10/17/2014 06:18 PM, Ray Jui wrote: > >On 10/17/2014 4:15 AM, Lars-Peter Clausen wrote: > >>On 10/17/2014 09:35 AM, Vinod Koul wrote: > >>>On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: > On 10/17/2014

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-21 Thread Vinod Koul
On Fri, Oct 17, 2014 at 01:15:55PM +0200, Lars-Peter Clausen wrote: > On 10/17/2014 09:35 AM, Vinod Koul wrote: > >On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: > >>On 10/17/2014 02:48 AM, Ray Jui wrote: > >>>As part of subsystem that many slave drivers depend on, it's more >

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Ray Jui
On 10/17/2014 9:39 AM, Lars-Peter Clausen wrote: On 10/17/2014 06:18 PM, Ray Jui wrote: On 10/17/2014 4:15 AM, Lars-Peter Clausen wrote: On 10/17/2014 09:35 AM, Vinod Koul wrote: On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: On 10/17/2014 02:48 AM, Ray Jui wrote: As pa

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Lars-Peter Clausen
On 10/17/2014 06:18 PM, Ray Jui wrote: On 10/17/2014 4:15 AM, Lars-Peter Clausen wrote: On 10/17/2014 09:35 AM, Vinod Koul wrote: On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: On 10/17/2014 02:48 AM, Ray Jui wrote: As part of subsystem that many slave drivers depend on,

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Ray Jui
On 10/17/2014 4:15 AM, Lars-Peter Clausen wrote: On 10/17/2014 09:35 AM, Vinod Koul wrote: On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: On 10/17/2014 02:48 AM, Ray Jui wrote: As part of subsystem that many slave drivers depend on, it's more appropriate for the pl330 DMA

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Lars-Peter Clausen
On 10/17/2014 09:35 AM, Vinod Koul wrote: On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: On 10/17/2014 02:48 AM, Ray Jui wrote: As part of subsystem that many slave drivers depend on, it's more appropriate for the pl330 DMA driver to be initialized at subsys_initcall than d

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Krzysztof Kozłowski
On 17.10.2014 02:48, Ray Jui wrote: > As part of subsystem that many slave drivers depend on, it's more > appropriate for the pl330 DMA driver to be initialized at > subsys_initcall than device_initcall > > Signed-off-by: Ray Jui > --- > drivers/dma/pl330.c | 12 +++- > 1 file changed,

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Vinod Koul
On Fri, Oct 17, 2014 at 09:45:45AM +0200, Lars-Peter Clausen wrote: > On 10/17/2014 02:48 AM, Ray Jui wrote: > >As part of subsystem that many slave drivers depend on, it's more > >appropriate for the pl330 DMA driver to be initialized at > >subsys_initcall than device_initcall > > Well, we do hav

Re: [PATCH] dmaengine: pl330: use subsys_initcall

2014-10-17 Thread Lars-Peter Clausen
On 10/17/2014 02:48 AM, Ray Jui wrote: As part of subsystem that many slave drivers depend on, it's more appropriate for the pl330 DMA driver to be initialized at subsys_initcall than device_initcall Well, we do have -EPROBE_DEFER these days to handle these kinds of dependencies so we no longe

[PATCH] dmaengine: pl330: use subsys_initcall

2014-10-16 Thread Ray Jui
As part of subsystem that many slave drivers depend on, it's more appropriate for the pl330 DMA driver to be initialized at subsys_initcall than device_initcall Signed-off-by: Ray Jui --- drivers/dma/pl330.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/drive