Hi Ulf > > @@ -397,13 +397,13 @@ sh_mmcif_request_dma_one(struct sh_mmcif_host *host, > > dma_cap_set(DMA_SLAVE, mask); > > > > if (pdata) > > - slave_id = direction == DMA_MEM_TO_DEV > > - ? pdata->slave_id_tx : pdata->slave_id_rx; > > + slave_data = direction == DMA_MEM_TO_DEV > > + ? (void *)pdata->slave_id_tx : (void *)pdata->slave_id_rx; > > else > > You may get rid of this else clause. Just assign slave_data to NULL > while declaring it above. > > > - slave_id = 0; > > + slave_data = 0; Thank you Will fix in v2 patch Best regards --- Kuninori Morimoto -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
