On Wed, Apr 25, 2018 at 08:20:49AM -0700, Christoph Hellwig wrote:
> > +void arch_iommu_detach_device(struct device *dev)
> > +{
> > +#ifdef CONFIG_ARM_DMA_USE_IOMMU
> > +   struct dma_iommu_mapping *mapping = to_dma_iommu_mapping(dev);
> > +   const struct dma_map_ops *dma_ops;
> > +
> > +   if (!mapping)
> > +           return;
> > +
> > +   arm_iommu_release_mapping(mapping);
> > +   arm_iommu_detach_device(dev);
> > +
> > +   dma_ops = arm_get_dma_map_ops(dev->archdata.dma_coherent);
> > +   set_dma_ops(dev, dma_ops);
> 
> Why not simply:
> 
>       set_dma_ops(dev, arm_get_dma_map_ops(dev->archdata.dma_coherent));

I had that initially, but it looked cluttered to me, so I split it up.
I don't care much either way, so I can revert to that if you prefer.

Thierry

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to