From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH/RFC v2 0/4] iommu/ipmmu-vmsa: IPMMU slave device whitelist V2 Date: Wed, 22 Mar 2017 15:25:01 +0100 Message-ID: <20170322142501.GM29659@8bytes.org> References: <148549764733.22203.16459470412085471955.sendpatchset@little-apple> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <148549764733.22203.16459470412085471955.sendpatchset@little-apple> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Magnus Damm Cc: laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org, geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org, linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, horms+renesas-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: iommu@lists.linux-foundation.org On Fri, Jan 27, 2017 at 03:14:07PM +0900, Magnus Damm wrote: > iommu/ipmmu-vmsa: IPMMU slave device whitelist V2 > > [PATCH/RFC v2 1/4] iommu/of: Skip IOMMU devices disabled in DT > [PATCH/RFC v2 2/4] iommu/ipmmu-vmsa: Get rid of disabled device check > [PATCH/RFC v2 3/4] iommu/ipmmu-vmsa: Check devices in xlate() > [PATCH/RFC v2 3/4] iommu/ipmmu-vmsa: Opt-in slave devices based on ES version > > Here's an updated prototype that shows how DT integration of IPMMU details > may be integrated and merged upstream based on SoC data sheet ahead of > time followed by enablement in the IPMMU driver code once the appropriate > SoC ES version has been released and the hardware has been tested. > > Changes since V1: > - Broke out patch 1 from the IPMMU driver > - Moved slave device check from ->add_device() to ->xlate() (Thanks Robin!) > - Updated white list patch to hook into ->xlate() > > Patch 1 may be suitable for upstream merge, however other patches should > in the future if agreed on be rolled into the IPMMU driver series. > > Signed-off-by: Magnus Damm > --- > > Developed on top of renesas-drivers-2017-01-24-v4.10-rc5 > > drivers/iommu/ipmmu-vmsa.c | 59 +++++++++++++++++++++++--------------------- > drivers/iommu/of_iommu.c | 2 - > 2 files changed, 33 insertions(+), 28 deletions(-) For the series: Reviewed-by: Joerg Roedel From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from 8bytes.org ([81.169.241.247]:39360 "EHLO theia.8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759385AbdCVO0D (ORCPT ); Wed, 22 Mar 2017 10:26:03 -0400 Date: Wed, 22 Mar 2017 15:25:01 +0100 From: Joerg Roedel To: Magnus Damm Cc: iommu@lists.linux-foundation.org, laurent.pinchart+renesas@ideasonboard.com, geert+renesas@glider.be, linux-renesas-soc@vger.kernel.org, horms+renesas@verge.net.au, robin.murphy@arm.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH/RFC v2 0/4] iommu/ipmmu-vmsa: IPMMU slave device whitelist V2 Message-ID: <20170322142501.GM29659@8bytes.org> References: <148549764733.22203.16459470412085471955.sendpatchset@little-apple> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <148549764733.22203.16459470412085471955.sendpatchset@little-apple> Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: On Fri, Jan 27, 2017 at 03:14:07PM +0900, Magnus Damm wrote: > iommu/ipmmu-vmsa: IPMMU slave device whitelist V2 > > [PATCH/RFC v2 1/4] iommu/of: Skip IOMMU devices disabled in DT > [PATCH/RFC v2 2/4] iommu/ipmmu-vmsa: Get rid of disabled device check > [PATCH/RFC v2 3/4] iommu/ipmmu-vmsa: Check devices in xlate() > [PATCH/RFC v2 3/4] iommu/ipmmu-vmsa: Opt-in slave devices based on ES version > > Here's an updated prototype that shows how DT integration of IPMMU details > may be integrated and merged upstream based on SoC data sheet ahead of > time followed by enablement in the IPMMU driver code once the appropriate > SoC ES version has been released and the hardware has been tested. > > Changes since V1: > - Broke out patch 1 from the IPMMU driver > - Moved slave device check from ->add_device() to ->xlate() (Thanks Robin!) > - Updated white list patch to hook into ->xlate() > > Patch 1 may be suitable for upstream merge, however other patches should > in the future if agreed on be rolled into the IPMMU driver series. > > Signed-off-by: Magnus Damm > --- > > Developed on top of renesas-drivers-2017-01-24-v4.10-rc5 > > drivers/iommu/ipmmu-vmsa.c | 59 +++++++++++++++++++++++--------------------- > drivers/iommu/of_iommu.c | 2 - > 2 files changed, 33 insertions(+), 28 deletions(-) For the series: Reviewed-by: Joerg Roedel From mboxrd@z Thu Jan 1 00:00:00 1970 From: joro@8bytes.org (Joerg Roedel) Date: Wed, 22 Mar 2017 15:25:01 +0100 Subject: [PATCH/RFC v2 0/4] iommu/ipmmu-vmsa: IPMMU slave device whitelist V2 In-Reply-To: <148549764733.22203.16459470412085471955.sendpatchset@little-apple> References: <148549764733.22203.16459470412085471955.sendpatchset@little-apple> Message-ID: <20170322142501.GM29659@8bytes.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Jan 27, 2017 at 03:14:07PM +0900, Magnus Damm wrote: > iommu/ipmmu-vmsa: IPMMU slave device whitelist V2 > > [PATCH/RFC v2 1/4] iommu/of: Skip IOMMU devices disabled in DT > [PATCH/RFC v2 2/4] iommu/ipmmu-vmsa: Get rid of disabled device check > [PATCH/RFC v2 3/4] iommu/ipmmu-vmsa: Check devices in xlate() > [PATCH/RFC v2 3/4] iommu/ipmmu-vmsa: Opt-in slave devices based on ES version > > Here's an updated prototype that shows how DT integration of IPMMU details > may be integrated and merged upstream based on SoC data sheet ahead of > time followed by enablement in the IPMMU driver code once the appropriate > SoC ES version has been released and the hardware has been tested. > > Changes since V1: > - Broke out patch 1 from the IPMMU driver > - Moved slave device check from ->add_device() to ->xlate() (Thanks Robin!) > - Updated white list patch to hook into ->xlate() > > Patch 1 may be suitable for upstream merge, however other patches should > in the future if agreed on be rolled into the IPMMU driver series. > > Signed-off-by: Magnus Damm > --- > > Developed on top of renesas-drivers-2017-01-24-v4.10-rc5 > > drivers/iommu/ipmmu-vmsa.c | 59 +++++++++++++++++++++++--------------------- > drivers/iommu/of_iommu.c | 2 - > 2 files changed, 33 insertions(+), 28 deletions(-) For the series: Reviewed-by: Joerg Roedel