On Mon, 3 Nov 2014, Will Deacon wrote: > On Mon, Nov 03, 2014 at 10:46:03AM +0000, Stefano Stabellini wrote: > > On Mon, 27 Oct 2014, Stefano Stabellini wrote: > > > Introduce a boolean flag and an accessor function to check whether a > > > device is dma_coherent. Set the flag from set_arch_dma_coherent_ops. > > > > > > Signed-off-by: Stefano Stabellini <[email protected]> > > > Signed-off-by: Catalin Marinas <[email protected]> > > > CC: [email protected] > > > > Will, Catalin, > > are you OK with this patch? > > It would be nicer if the dma_coherent flag didn't have to be duplicated by > each architecture in dev_archdata. Is there any reason not to put it in the > core code?
Yes, there is a reason for it: if I added a boolean dma_coherent flag in struct device as Catalin initially suggested, what would be the default for each architecture? Where would I set it for arch that don't use device tree? It is not easy. I thought it would be better to introduce is_device_dma_coherent only on the architectures where it certainly makes sense to have it. In fact I checked and arm and arm64 are the only architectures to define set_arch_dma_coherent_ops at the moment. At that point if is_device_dma_coherent becomes arch-specific, it makes sense to store the flag in dev_archdata instead of struct device. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
