Re: [PATCH] of: dma: protect list write operation by spin_lock

[Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>]

On Tue, 2013-02-12 at 09:20 -0800, Vinod Koul wrote: 
> On Tue, Jan 29, 2013 at 09:36:45AM -0600, Rob Herring wrote:
> > >> Vinod, it's probably better to move drivers/of/dma.c to drivers/dma.
> > >> That's been the direction we've been taking with new OF binding code. I
> > >> must have missed that in the review.
> > > ok, do you have any patches on your tree for this if not then I can update my tree
> > > for this change and then we can apply changes thru slave-dma tree.
> > 
> > No, I don't have one prepared.
> Okay, I have committed and pushed drivers/of/dma.c to be moved to
> drivers/dma/of-dma.c
> 
> From: Vinod Koul <vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> Date: Tue, 12 Feb 2013 09:15:02 -0800
> Subject: [PATCH] dmaengine: move drivers/of/dma.c -> drivers/dma/of-dma.c
> 
> as requested by Rob
> 
> Suggested-by: Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>
> Signed-off-by: Vinod Koul <vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> ---
>  drivers/{of/dma.c => dma/of-dma.c} |    0
>  1 files changed, 0 insertions(+), 0 deletions(-)
>  rename drivers/{of/dma.c => dma/of-dma.c} (100%)
> 

What about Kconfig(s) / Makefile(s)?

And I think the of_dma.c looks better as a name of the file.

> diff --git a/drivers/of/dma.c b/drivers/dma/of-dma.c
> similarity index 100%
> rename from drivers/of/dma.c
> rename to drivers/dma/of-dma.c
> --
> 1.7.0.4
> 
> Anyone cares to Ack?
> 
> --
> ~Vinod

-- 
Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Intel Finland Oy