Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-30 Thread Archit Taneja
On 3/30/2017 7:16 PM, Rob Clark wrote: On Thu, Mar 30, 2017 at 2:19 AM, Archit Taneja wrote: Hi, On 03/14/2017 08:48 PM, Rob Clark wrote: An iommu driver for Qualcomm "B" family devices which do not completely implement the ARM SMMU spec. These devices have context-bank register layout th

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-30 Thread Rob Clark
On Thu, Mar 30, 2017 at 2:19 AM, Archit Taneja wrote: > Hi, > > On 03/14/2017 08:48 PM, Rob Clark wrote: >> >> An iommu driver for Qualcomm "B" family devices which do not completely >> implement the ARM SMMU spec. These devices have context-bank register >> layout that is similar to ARM SMMU, bu

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-29 Thread Archit Taneja
Hi, On 03/14/2017 08:48 PM, Rob Clark wrote: An iommu driver for Qualcomm "B" family devices which do not completely implement the ARM SMMU spec. These devices have context-bank register layout that is similar to ARM SMMU, but no global register space (or at least not one that is accessible).

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-20 Thread Rob Clark
On Mon, Mar 20, 2017 at 10:21 AM, Sricharan R wrote: > Hi Rob, > > sorry for the delayed response. Was not there mostly last week. > > > On 3/13/2017 11:49 PM, Rob Clark wrote: >> >> On Mon, Mar 13, 2017 at 9:38 AM, wrote: >>> >>> Hi Rob, >>> >>> [..] >>> >>> +static int qcom_iommu_init_dom

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-20 Thread Sricharan R
Hi Rob, sorry for the delayed response. Was not there mostly last week. On 3/13/2017 11:49 PM, Rob Clark wrote: On Mon, Mar 13, 2017 at 9:38 AM, wrote: Hi Rob, [..] +static int qcom_iommu_init_domain(struct iommu_domain *domain, + struct qcom_iommu_dev *qc

[PATCH 5/9] iommu: add qcom_iommu

2017-03-14 Thread Rob Clark
An iommu driver for Qualcomm "B" family devices which do not completely implement the ARM SMMU spec. These devices have context-bank register layout that is similar to ARM SMMU, but no global register space (or at least not one that is accessible). Signed-off-by: Rob Clark Signed-off-by: Stanimi

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-13 Thread Rob Clark
On Mon, Mar 13, 2017 at 9:38 AM, wrote: > Hi Rob, > > [..] > > >> +static int qcom_iommu_init_domain(struct iommu_domain *domain, >> + struct qcom_iommu_dev *qcom_iommu, >> + struct iommu_fwspec *fwspec) >> +{ >> + struct qcom_

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-13 Thread sricharan
Hi Rob, [..] +static int qcom_iommu_init_domain(struct iommu_domain *domain, + struct qcom_iommu_dev *qcom_iommu, + struct iommu_fwspec *fwspec) +{ + struct qcom_iommu_domain *qcom_domain = to_qcom_iommu_domain(domain); +

RE: [PATCH 5/9] iommu: add qcom_iommu

2017-03-09 Thread sricharan
Hi, >On Tue, Mar 7, 2017 at 12:48 PM, Robin Murphy >wrote: >> On 01/03/17 17:42, Rob Clark wrote: >>> An iommu driver for Qualcomm "B" family devices which do not >>> completely implement the ARM SMMU spec. >> >> Is that actually true, or is it just that it's a compliant SMMU on >> which firmware

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-07 Thread Rob Clark
On Tue, Mar 7, 2017 at 12:48 PM, Robin Murphy wrote: > On 01/03/17 17:42, Rob Clark wrote: >> An iommu driver for Qualcomm "B" family devices which do not completely >> implement the ARM SMMU spec. > > Is that actually true, or is it just that it's a compliant SMMU on which > firmware has set SCR1

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-07 Thread Robin Murphy
On 01/03/17 17:42, Rob Clark wrote: > An iommu driver for Qualcomm "B" family devices which do not completely > implement the ARM SMMU spec. Is that actually true, or is it just that it's a compliant SMMU on which firmware has set SCR1.GASRAE? (which makes the global address space secure-access-on

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-01 Thread Rob Clark
On Wed, Mar 1, 2017 at 6:54 PM, Stephen Boyd wrote: > On 03/01/2017 09:42 AM, Rob Clark wrote: >> diff --git a/drivers/iommu/qcom_iommu.c b/drivers/iommu/qcom_iommu.c >> new file mode 100644 >> index 000..5d3bb63 >> --- /dev/null >> +++ b/drivers/iommu/qcom_iommu.c >> @@ -0,0 +1,825 @@ >> +/*

Re: [PATCH 5/9] iommu: add qcom_iommu

2017-03-01 Thread Stephen Boyd
On 03/01/2017 09:42 AM, Rob Clark wrote: > diff --git a/drivers/iommu/qcom_iommu.c b/drivers/iommu/qcom_iommu.c > new file mode 100644 > index 000..5d3bb63 > --- /dev/null > +++ b/drivers/iommu/qcom_iommu.c > @@ -0,0 +1,825 @@ > +/* > + * IOMMU API for QCOM secure IOMMUs. Somewhat based on arm

[PATCH 5/9] iommu: add qcom_iommu

2017-03-01 Thread Rob Clark
An iommu driver for Qualcomm "B" family devices which do not completely implement the ARM SMMU spec. These devices have context-bank register layout that is similar to ARM SMMU, but no global register space (or at least not one that is accessible). Signed-off-by: Rob Clark --- drivers/iommu/Kco