From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH v4 0/8] iommu/rockchip: Fix bugs and enable on ARM64 Date: Tue, 21 Jun 2016 15:52:02 +0200 Message-ID: <20160621135202.GK28248@8bytes.org> References: <1466483681-36110-1-git-send-email-tfiga@chromium.org> <20160621125435.GE11432@8bytes.org> <2015742.xhx642kbrC@diego> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <2015742.xhx642kbrC@diego> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Heiko =?iso-8859-1?Q?St=FCbner?= Cc: "linux-kernel@vger.kernel.org" , dri-devel , Tomasz Figa , "open list:ARM/Rockchip SoC..." , "open list:IOMMU DRIVERS" , Marek Szyprowski , Shunqian Zheng , "linux-arm-kernel@lists.infradead.org" List-Id: iommu@lists.linux-foundation.org T24gVHVlLCBKdW4gMjEsIDIwMTYgYXQgMDM6MTg6NDlQTSArMDIwMCwgSGVpa28gU3TDvGJuZXIg d3JvdGU6Cj4gQW0gRGllbnN0YWcsIDIxLiBKdW5pIDIwMTYsIDE0OjU0OjM1IHNjaHJpZWIgSm9l cmcgUm9lZGVsOgo+ID4gSGkgVG9tYXN6LAo+ID4gCj4gPiBPbiBUdWUsIEp1biAyMSwgMjAxNiBh dCAwOTo0MjoxNlBNICswOTAwLCBUb21hc3ogRmlnYSB3cm90ZToKPiA+ID4gSW4gc2ltcGxlIHdv cmRzLCBEUk0gcGF0Y2hlcyBkZXBlbmQgb24gSU9NTVUgcGF0Y2hlcy4KPiA+ID4gCj4gPiA+IE1v cmUgcHJlY2lzZWx5OiBUaGUgSU9NTVUgcGF0Y2hlcyBhbG9uZSBhcmUgc3VwcG9zZWQgdG8gbm90 IGJyZWFrCj4gPiA+IGFueXRoaW5nLiBTYW1lIGdvZXMgZm9yIHRoZSBmaXJzdCBEUk0gcGF0Y2gg KDcvOCkuIE9ubHkgc2Vjb25kIERSTQo+ID4gPiBwYXRjaCAoOC84KSBkZXBlbmRzIG9uIGNoYW5n ZXMgaW50cm9kdWNlZCBieSBpdHMgcHJlZGVjZXNzb3JzLgo+ID4gCj4gPiBUaGUgZmlyc3QgRFJN IHBhdGNoIGlzIDYvNywgc28gaXQgaXMgNy84IHdpdGggdGhlIGlvbW11IGRlcGVuZGVuY3ksCj4g PiByaWdodD8gQW55d2F5LCBJIHRoaW5rIHRoZSBiZXN0IGlzIEkgdGFrZSB0aGUgaW9tbXUgcGF0 Y2hlcyB3aGVuIEhlaWtvCj4gPiBpcyBvayB3aXRoIHRoZW0gYW5kIHRoZW4gdGhlIERSTSB0cmVl IGNhbiBtZXJnZSB0aGF0IGJyYW5jaCBpbiB0byBhcHBseQo+ID4gdGhlIERSTSBwYXRjaGVzLgo+ ID4gCj4gPiBCdXQgZmlyc3QgSGVpa28gc2hvdWxkIGhhdmUgYSBsb29rIGF0IHRoZSBwYXRjaGVz Lgo+IAo+IEkgdGhpbmsgZnJvbSBhbGwgaGlzIHByZXZpb3VzIHdvcmsgb24gdGhlIHJvY2tjaGlw IGlvbW11cyBUb21hc3ogaXMgYSBsb3QgbW9yZSAKPiBxdWFsaWZpZWQgdG8ganVkZ2UgdGhlbSAt IHdoaWNoIEkgZ3Vlc3MgaGUgZGlkIHdoZW4gcGlja2luZyB1cCB0aGUgb25lcyBmcm9tIAo+IFJv Y2tjaGlwIGRldnMgOi0pIC4KCkluIHRoYXQgY2FzZSB5b3UgZ3V5cyBzaG91bGQgcHJvYmFibHkg Y28tbWFpbnRhaW4gdGhhdCBkcml2ZXI/Cj4gCj4gPkZyb20gYSBzdHlsZS1zaWRlLCBwbGVhc2Ug ZG9uJ3QgY2FycnkgdGhlIFJldmlld2VkLW9uIGdlcnJpdCB0YWdzIG92ZXIgdG8gCj4gbWFpbmxp bmUgcGF0Y2hlcyAocGF0Y2hlcyAxIGFuZCAyKS4KCk9rYXkuCgoKCglKb2VyZwoKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcg bGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRl c2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: joro@8bytes.org (Joerg Roedel) Date: Tue, 21 Jun 2016 15:52:02 +0200 Subject: [PATCH v4 0/8] iommu/rockchip: Fix bugs and enable on ARM64 In-Reply-To: <2015742.xhx642kbrC@diego> References: <1466483681-36110-1-git-send-email-tfiga@chromium.org> <20160621125435.GE11432@8bytes.org> <2015742.xhx642kbrC@diego> Message-ID: <20160621135202.GK28248@8bytes.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751610AbcFUOB0 (ORCPT ); Tue, 21 Jun 2016 10:01:26 -0400 Received: from 8bytes.org ([81.169.241.247]:54787 "EHLO theia.8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751123AbcFUOBZ (ORCPT ); Tue, 21 Jun 2016 10:01:25 -0400 Date: Tue, 21 Jun 2016 15:52:02 +0200 From: Joerg Roedel To: Heiko =?iso-8859-1?Q?St=FCbner?= Cc: Tomasz Figa , "open list:IOMMU DRIVERS" , dri-devel , "linux-arm-kernel@lists.infradead.org" , "open list:ARM/Rockchip SoC..." , "linux-kernel@vger.kernel.org" , David Airlie , Mark Yao , Shunqian Zheng , Daniel Kurtz , Marek Szyprowski Subject: Re: [PATCH v4 0/8] iommu/rockchip: Fix bugs and enable on ARM64 Message-ID: <20160621135202.GK28248@8bytes.org> References: <1466483681-36110-1-git-send-email-tfiga@chromium.org> <20160621125435.GE11432@8bytes.org> <2015742.xhx642kbrC@diego> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <2015742.xhx642kbrC@diego> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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