[PATCH v6 05/12] memory: Add NVIDIA Tegra memory controller support

Thierry Reding thierry.reding at gmail.com
Wed Nov 12 06:47:16 PST 2014


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20141112/9b541f1f/attachment.sig>


More information about the linux-arm-kernel mailing list