Re: [PATCH v6 00/21] IOMMU: Tegra GART driver clean up and optimization

Thierry Reding
2018-12-12T10:24:15Z

[-- Attachment #1: Type: text/plain, Size: 2015 bytes --]

On Tue, Dec 11, 2018 at 10:53:17AM +0100, Joerg Roedel wrote:
> On Sun, Dec 09, 2018 at 11:29:29PM +0300, Dmitry Osipenko wrote:
> > Dmitry Osipenko (21):
> > iommu/tegra: gart: Remove pr_fmt and clean up includes
> > iommu/tegra: gart: Clean up driver probe errors handling
> > iommu/tegra: gart: Ignore devices without IOMMU phandle in DT
> > iommu: Introduce iotlb_sync_map callback
> > iommu/tegra: gart: Optimize mapping / unmapping performance
> > dt-bindings: memory: tegra: Squash tegra20-gart into tegra20-mc
> > ARM: dts: tegra20: Update Memory Controller node to the new binding
> > memory: tegra: Don't invoke Tegra30+ specific memory timing setup on
> > Tegra20
> > memory: tegra: Adapt to Tegra20 device-tree binding changes
> > memory: tegra: Read client ID on GART page fault
> > memory: tegra: Use of_device_get_match_data()
> > memory: tegra: Use relaxed versions of readl/writel
> > iommu/tegra: gart: Integrate with Memory Controller driver
> > iommu/tegra: gart: Fix spinlock recursion
> > iommu/tegra: gart: Fix NULL pointer dereference
> > iommu/tegra: gart: Allow only one active domain at a time
> > iommu/tegra: gart: Don't use managed resources
> > iommu/tegra: gart: Prepend error/debug messages with "gart:"
> > iommu/tegra: gart: Don't detach devices from inactive domains
> > iommu/tegra: gart: Simplify clients-tracking code
> > iommu/tegra: gart: Perform code refactoring
>
> This is making progress, but some parts have no Ack or Review yet, I'll
> wait for Thierry before applying these patches into the iommu tree.

So appart from the one issue in the "memory controller integration"
patch this looks good and I've acked the remaining patches. Once the one
remaining issue is fixed I think this is ready to be merged.

Joerg, given the dependencies between the various parts of the series, I
think it may be better for you to merge everything through the IOMMU
tree if you don't mind.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]