Re: [PATCH v4 0/8] iommu/rockchip: Fix bugs and enable on ARM64
From: Joerg Roedel
Date: Tue Jun 21 2016 - 10:01:29 EST
On Tue, Jun 21, 2016 at 03:18:49PM +0200, Heiko Stübner wrote:
> Am Dienstag, 21. Juni 2016, 14:54:35 schrieb Joerg Roedel:
> > Hi Tomasz,
> >
> > On Tue, Jun 21, 2016 at 09:42:16PM +0900, Tomasz Figa wrote:
> > > In simple words, DRM patches depend on IOMMU patches.
> > >
> > > More precisely: The IOMMU patches alone are supposed to not break
> > > anything. Same goes for the first DRM patch (7/8). Only second DRM
> > > patch (8/8) depends on changes introduced by its predecessors.
> >
> > The first DRM patch is 6/7, so it is 7/8 with the iommu dependency,
> > right? Anyway, I think the best is I take the iommu patches when Heiko
> > is ok with them and then the DRM tree can merge that branch in to apply
> > the DRM patches.
> >
> > But first Heiko should have a look at the patches.
>
> I think from all his previous work on the rockchip iommus Tomasz is a lot more
> qualified to judge them - which I guess he did when picking up the ones from
> Rockchip devs :-) .
In that case you guys should probably co-maintain that driver?
>
> >From a style-side, please don't carry the Reviewed-on gerrit tags over to
> mainline patches (patches 1 and 2).
Okay.
Joerg