On Friday 15 May 2015 10:15:03 Barry Song wrote: > @@ -658,8 +849,16 @@ static int sirfsoc_dma_probe(struct platform_device *op) > return -ENOMEM; > } > > - if (of_device_is_compatible(dn, "sirf,marco-dmac")) > - sdma->is_marco = true; > + if (of_device_is_compatible(dn, "sirf,atlas7-dmac")) { > + sdma->ip_ver = SIRFSOC_DMA_VER_A7V1; > + sdma->exec_desc = sirfsoc_dma_execute_hw_a7v1; > + } else if (of_device_is_compatible(dn, "sirf,atlas7-dmac-v2")) { > + sdma->ip_ver = SIRFSOC_DMA_VER_A7V2; > + sdma->exec_desc = sirfsoc_dma_execute_hw_a7v2; > + } else { > + sdma->ip_ver = SIRFSOC_DMA_VER_A6; > + sdma->exec_desc = sirfsoc_dma_execute_hw_a6; > + } > > if (of_property_read_u32(dn, "cell-index", &id)) { > dev_err(dev, "Fail to get DMAC index\n"); Please send a separate patch for the removal of the marco support, it's always better to keep those things separate. When you have noticeable differences like this, use the data field of the sirfsoc_dma_match[] array to pass a structure with the differences, to avoid the chain of 'of_device_is_compatible' calls. You can use data = of_match_device(dev->driver->of_match_table, dev)->data; to get that pointer from the probe function. Arnd -- 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