Hi Wolfram, On Fri, Oct 31, 2014 at 11:51 AM, Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote: > --- a/drivers/i2c/busses/i2c-sh_mobile.c > +++ b/drivers/i2c/busses/i2c-sh_mobile.c > +static struct dma_chan *sh_mobile_i2c_request_dma_chan(struct device *dev, > + enum dma_transfer_direction dir, > + dma_addr_t port_addr) > +{ > + dma_cap_mask_t mask; > + struct dma_chan *chan; > + struct dma_slave_config cfg; > + int ret; > + > + dma_cap_zero(mask); > + dma_cap_set(DMA_SLAVE, mask); > + > + chan = dma_request_slave_channel_compat(mask, shdma_chan_filter, > + (void *)0UL, dev, dir == DMA_MEM_TO_DEV ? "tx" : "rx"); Given you only support DT DMA configuration (you always pass (void *)0UL as the channel ID), I think you can call dma_request_slave_channel(dev, dir == DMA_MEM_TO_DEV ? "tx" : "rx") directly instead of using the dma_request_slave_channel_compat() wrapper. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html