On Mon, Jan 28, 2013 at 09:52:00AM -0600, Rob Herring wrote: > On 01/28/2013 09:24 AM, Andy Shevchenko wrote: > > Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com> > > --- > > drivers/of/dma.c | 2 ++ > > 1 file changed, 2 insertions(+) > > You need to cc whomever has this file in their tree which is Vinod. > > Acked-by: Rob Herring <rob.herr...@calxeda.com> > > 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.
-- ~Vinod > > Rob > > > > > diff --git a/drivers/of/dma.c b/drivers/of/dma.c > > index 59631b2..1ae6120 100644 > > --- a/drivers/of/dma.c > > +++ b/drivers/of/dma.c > > @@ -117,7 +117,9 @@ int of_dma_controller_register(struct device_node *np, > > ofdma->use_count = 0; > > > > /* Now queue of_dma controller structure in list */ > > + spin_lock(&of_dma_lock); > > list_add_tail(&ofdma->of_dma_controllers, &of_dma_list); > > + spin_unlock(&of_dma_lock); > > > > return 0; > > } > > _______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss