On Tue, Dec 12, 2023 at 11:20:33AM +0300, Nikita Shubin wrote: > - add OF ID match table > - add device tree DMA request, so we can probe defer, in case DMA is not > ready yet > - drop DMA platform code ... > + espi->dma_rx = dma_request_chan(dev, "rx"); > + if (IS_ERR(espi->dma_rx)) { > + ret = PTR_ERR(espi->dma_rx); > + dev_err_probe(dev, ret, "rx DMA setup failed"); ret = dev_err_probe(...); > goto fail_free_page; > } ... > + espi->dma_tx = dma_request_chan(dev, "tx"); > + if (IS_ERR(espi->dma_tx)) { > + ret = PTR_ERR(espi->dma_tx); > + dev_err_probe(dev, ret, "tx DMA setup failed"); Ditto. > goto fail_release_rx; > } ... Otherwise LGTM. -- With Best Regards, Andy Shevchenko