Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding

2017-02-08 Thread Rob Herring
On Fri, Feb 03, 2017 at 04:10:30PM +0200, Laurent Pinchart wrote: > Hi Rob, > > On Monday 29 Feb 2016 23:33:09 Magnus Damm wrote: > > From: Magnus Damm > > > > Update the IPMMU DT binding documentation to include the r8a7795 compat > > string as well as the

Re: [PATCH v9 8/8] perf/amd/iommu: Enable support for multiple IOMMUs

2017-02-08 Thread Borislav Petkov
On Tue, Feb 07, 2017 at 02:40:36AM -0600, Suravee Suthikulpanit wrote: > From: Suravee Suthikulpanit > > Add multi-IOMMU support for perf by exposing an AMD IOMMU PMU > for each IOMMU found in the system via: > > /bus/event_source/devices/amd_iommu_x > > where

Re: [PATCH v9 6/8] perf/amd/iommu: Modify amd_iommu_pc_get_set_reg_val() to allow specifying IOMMU

2017-02-08 Thread Borislav Petkov
On Tue, Feb 07, 2017 at 02:40:34AM -0600, Suravee Suthikulpanit wrote: > From: Suravee Suthikulpanit > > The current amd_iommu_pc_get_set_reg_val() cannot support multiple IOMMUs. > So, modify it to allow callers to specify IOMMU. This prepares the driver > for

Re: [PATCH v9 5/8] perf/amd/iommu: Modify functions to query max banks and counters

2017-02-08 Thread Borislav Petkov
On Tue, Feb 07, 2017 at 02:40:33AM -0600, Suravee Suthikulpanit wrote: > Currently, amd_iommu_pc_get_max_[banks|counters]() use end-point > device ID to locate an IOMMU and check the reported max banks/counters. > The logic assumes that the IOMMU_BASE_DEVID belongs to the first IOMMU, > and uses

Re: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Robin Murphy
On 08/02/17 13:52, Mark Rutland wrote: > On Wed, Feb 08, 2017 at 07:15:37PM +0530, Sricharan wrote: >>> Clocks are not architectural, so it only makes sense to associate them >>> with an implementation-specific compatible string. There's also no >> >> ok, it for this the QCOM specific

Re: [PATCH 09/10] iommu/exynos: Make use of iommu_device_register interface

2017-02-08 Thread Marek Szyprowski
Hi Joerg, On 2017-02-08 14:57, Joerg Roedel wrote: On Tue, Feb 07, 2017 at 01:36:15PM +0100, Marek Szyprowski wrote: + ret = iommu_device_sysfs_add(>iommu, >dev, NULL, +"sysmmu.%pa", ); Can we stick to the common name across the /sysfs and use

Re: [PATCH 09/10] iommu/exynos: Make use of iommu_device_register interface

2017-02-08 Thread Joerg Roedel
Hi Marek, On Tue, Feb 07, 2017 at 01:36:15PM +0100, Marek Szyprowski wrote: > >+ret = iommu_device_sysfs_add(>iommu, >dev, NULL, > >+ "sysmmu.%pa", ); > > Can we stick to the common name across the /sysfs and use > dev_name(data->sysmmu) > or even

Re: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Mark Rutland
On Wed, Feb 08, 2017 at 07:15:37PM +0530, Sricharan wrote: > >Clocks are not architectural, so it only makes sense to associate them > >with an implementation-specific compatible string. There's also no > > ok, it for this the QCOM specific implementation binding is tried(going to). > >

RE: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Sricharan
Hi Robin, > On Thu, Feb 02, 2017 at 10:40:18PM +0530, Sricharan R wrote: >> +- clock-names:Should be a pair of "smmu_iface_clk" and >> "smmu_bus_clk" >> + required for smmu's register group access and >> interface >> + clk for the

Re: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Robin Murphy
On 08/02/17 12:30, Sricharan wrote: > Hi Mark, > >> On Wed, Feb 08, 2017 at 04:23:17PM +0530, Sricharan wrote: On Thu, Feb 02, 2017 at 10:40:18PM +0530, Sricharan R wrote: > +- clock-names:Should be a pair of "smmu_iface_clk" and "smmu_bus_clk" > + required for

RE: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Sricharan
Hi Mark, >On Wed, Feb 08, 2017 at 04:23:17PM +0530, Sricharan wrote: >> >On Thu, Feb 02, 2017 at 10:40:18PM +0530, Sricharan R wrote: >> >> +- clock-names:Should be a pair of "smmu_iface_clk" and "smmu_bus_clk" >> >> + required for smmu's register group access and interface

Re: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Mark Rutland
On Wed, Feb 08, 2017 at 04:23:17PM +0530, Sricharan wrote: > >On Thu, Feb 02, 2017 at 10:40:18PM +0530, Sricharan R wrote: > >> +- clock-names:Should be a pair of "smmu_iface_clk" and "smmu_bus_clk" > >> + required for smmu's register group access and interface > >> +

RE: [PATCH V2 1/3] iommu/arm-smmu: Add pm_runtime/sleep ops

2017-02-08 Thread Sricharan
Hi Mark, > >On Thu, Feb 02, 2017 at 10:40:18PM +0530, Sricharan R wrote: >> +- clock-names:Should be a pair of "smmu_iface_clk" and "smmu_bus_clk" >> + required for smmu's register group access and interface >> + clk for the smmu's underlying bus access. >> +