On Wed, Nov 12, 2014 at 03:21:50PM +0100, Joerg Roedel wrote: > On Fri, Nov 07, 2014 at 05:00:56PM +0100, Thierry Reding wrote: > > drivers/iommu/tegra-smmu.c | 1295 ------------------------------ > > drivers/memory/tegra/smmu.c | 716 +++++++++++++++++ > > This new smmu.c is an IOMMU driver, why do you put it in drivers/memory > and not in drivers/iommu? The SMMU is part of a larger IP block that's also a memory controller. Having it in drivers/iommu would mean that we need to provide a mechanism to synchronize between the two drivers. They also share a number of data tables, so that would need to be shared somehow as well. By keeping both in the same directory we don't have to expose any of this publicly. Thierry
Attachment:
pgppeL3Rph8gl.pgp
Description: PGP signature
