Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755485AbaG3SIs (ORCPT ); Wed, 30 Jul 2014 14:08:48 -0400 Received: from mail-vc0-f174.google.com ([209.85.220.174]:57372 "EHLO mail-vc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753333AbaG3SIp (ORCPT ); Wed, 30 Jul 2014 14:08:45 -0400 MIME-Version: 1.0 In-Reply-To: <20140730143037.GD8989@arm.com> References: <1404487757-18829-1-git-send-email-thierry.reding@gmail.com> <20140730110425.GI12239@arm.com> <20140730132348.GM29590@ulmo> <20140730143037.GD8989@arm.com> From: Rob Herring Date: Wed, 30 Jul 2014 13:08:23 -0500 Message-ID: Subject: Re: [PATCH v4] devicetree: Add generic IOMMU device tree bindings To: Will Deacon Cc: Thierry Reding , Joerg Roedel , Arnd Bergmann , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Stephen Warren , Cho KyongHo , Grant Grundler , Dave P Martin , Marc Zyngier , Hiroshi Doyu , Olav Haugan , Varun Sethi , "devicetree@vger.kernel.org" , "iommu@lists.linux-foundation.org" , "linux-arm-kernel@lists.infradead.org" , "linux-tegra@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "arm@kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 30, 2014 at 9:30 AM, Will Deacon wrote: > On Wed, Jul 30, 2014 at 02:23:50PM +0100, Thierry Reding wrote: >> On Wed, Jul 30, 2014 at 12:04:25PM +0100, Will Deacon wrote: >> > On Fri, Jul 04, 2014 at 04:29:17PM +0100, Thierry Reding wrote: >> > > From: Thierry Reding >> > > >> > > This commit introduces a generic device tree binding for IOMMU devices. >> > > Only a very minimal subset is described here, but it is enough to cover >> > > the requirements of both the Exynos System MMU and Tegra SMMU as >> > > discussed here: >> > > >> > > https://lkml.org/lkml/2014/4/27/346 >> > > >> > > Signed-off-by: Thierry Reding >> > > --- >> > > Changes in v4: >> > > - clarify that disabling an IOMMU DT node may not disable translation >> > > - be more explicit that examples are only examples >> > > - add multi-ID master example >> > > >> > > Changes in v3: >> > > - use #iommu-cells instead of #address-cells/#size-cells >> > > - drop optional iommu-names property >> > > >> > > Changes in v2: >> > > - add notes about "dma-ranges" property (drop note from commit message) >> > > - document priorities of "iommus" property vs. "dma-ranges" property >> > > - drop #iommu-cells in favour of #address-cells and #size-cells >> > > - remove multiple-master device example >> > > >> > > Documentation/devicetree/bindings/iommu/iommu.txt | 172 ++++++++++++++++++++++ >> > > 1 file changed, 172 insertions(+) >> > > create mode 100644 Documentation/devicetree/bindings/iommu/iommu.txt >> > >> > I'm concerned that this patch hasn't been picked up for 3.17 (I can't see it >> > in -next). If we want to move the ARM SMMU driver over to this new binding, >> > we can't keep dragging our feet for much longer as I *really* don't plan to >> > support two bindings in parallel (one is complicated enough already). >> > >> > Any chance we can see this merged, please? >> >> I think there weren't any comments left for me to address and I've >> mostly been waiting for Joerg to pick it up. >> >> Joerg, can you take this through the iommu tree for 3.17? Will acked >> this, but perhaps you were waiting for an ACK from the device tree >> bindings maintainers? > > Rob, Mark: can one or both of you take a look at this please? I've been quiet on this round, but I think prior input I've had has been addressed. If we believe this will work for ARM SMMU and MSM IOMMU and some of the crazy chaining scenarios, then I'm fine with the binding. Acked-by: Rob Herring Rob P.S. Thankfully, there are no Calxeda systems with the SMMU enabled, so a binding change should not cause much pain. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/