On Monday 01 December 2014 16:58:09 Rob Herring wrote:
> > @@ -178,28 +180,21 @@ static void of_dma_configure(struct device *dev)
> > if (!dev->dma_mask)
> > dev->dma_mask = &dev->coherent_dma_mask;
> >
> > - /*
> > - * if dma-coherent property exist, call arch hook to setup
> > - * dma coherent operations.
> > - */
> > - if (of_dma_is_coherent(dev->of_node)) {
> > - set_arch_dma_coherent_ops(dev);
> > - dev_dbg(dev, "device is dma coherent\n");
> > - }
> > -
> > - /*
> > - * if dma-ranges property doesn't exist - just return else
> > - * setup the dma offset
> > - */
> > ret = of_dma_get_range(dev->of_node, &dma_addr, &paddr, &size);
> > if (ret < 0) {
> > - dev_dbg(dev, "no dma range information to setup\n");
> > - return;
> > + dma_addr = offset = 0;
> > + size = dev->coherent_dma_mask;
>
> It looks like size is not used.
Catalin has proposed a patch for handling the size right, but it's
still under discussion.
Arnd
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu
