On Mon, Apr 22, 2013 at 12:22:32PM +0200, Nicolas Ferre wrote: > On 04/19/2013 11:11 AM, ludovic.desroches@xxxxxxxxx : > > From: Ludovic Desroches <ludovic.desroches@xxxxxxxxx> > > +#ifdef CONFIG_OF > > +static bool at_dma_filter(struct dma_chan *chan, void *slave) this is not defined for else case here. Also this could be CONFIG_DMA_OF...? > > +{ > > + struct at_dma_slave *atslave = slave; > > + > > + if (atslave->dma_dev == chan->device->dev) { > > + chan->private = atslave; > > + return true; > > + } else { > > + return false; > > + } > > +} -- ~Vinod -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html