Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-08-30 Thread Stephen Boyd
Quoting Stephen Boyd (2019-07-16 16:47:07) > Quoting Vivek Gautam (2019-06-12 02:26:20) > > > > > > On 6/11/2019 4:51 AM, Stephen Boyd wrote: > > > hardware signal like the NS bit and/or the Execution Level. Hopefully > > > it's a config and then our difference from MTP can be minimized. > > >

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-07-16 Thread Stephen Boyd
Quoting Vivek Gautam (2019-06-12 02:26:20) > > > On 6/11/2019 4:51 AM, Stephen Boyd wrote: > > hardware signal like the NS bit and/or the Execution Level. Hopefully > > it's a config and then our difference from MTP can be minimized. > > I don't think SMMU limits any such programming of SIDs.

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-12 Thread Vivek Gautam
On 6/11/2019 4:51 AM, Stephen Boyd wrote: Quoting Vivek Gautam (2019-06-06 04:17:16) Hi Stephen, On Thu, Jun 6, 2019 at 2:27 AM Stephen Boyd wrote: Quoting Vivek Gautam (2019-06-04 21:55:26) Cheza will throw faults for anything that is programmed with TZ on mtp as all of that should be

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-10 Thread Stephen Boyd
Quoting Vivek Gautam (2019-06-06 04:17:16) > Hi Stephen, > > On Thu, Jun 6, 2019 at 2:27 AM Stephen Boyd wrote: > > > > Quoting Vivek Gautam (2019-06-04 21:55:26) > > > > > > > > Cheza will throw faults for anything that is programmed with TZ on mtp > > > as all of that should be handled in

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-06 Thread Marc Gonzalez
On 06/06/2019 13:17, Vivek Gautam wrote: > <_smmu 0x6c0 0x3> // for both 0x6c0 (TZ) and 0x6c3 (HLOS) Another possibility is to list both: <_smmu 0x6c0 0x0> <_smmu 0x6c3 0x0> which leaves 0x6c1 and 0x6c2 out of the picture, and makes 0x6c3 appear explicitly (for anyone

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-06 Thread Vivek Gautam
Hi Stephen, On Thu, Jun 6, 2019 at 2:27 AM Stephen Boyd wrote: > > Quoting Vivek Gautam (2019-06-04 21:55:26) > > On Wed, Jun 5, 2019 at 4:16 AM Stephen Boyd wrote: > > > > > > Quoting Bjorn Andersson (2019-06-04 15:37:00) > > > > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > > > > >

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-05 Thread Stephen Boyd
Quoting Vivek Gautam (2019-06-04 21:55:26) > On Wed, Jun 5, 2019 at 4:16 AM Stephen Boyd wrote: > > > > Quoting Bjorn Andersson (2019-06-04 15:37:00) > > > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > > > > > The SMMU that sits in front of the QUP needs to be programmed properly > > >

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Vivek Gautam
On Wed, Jun 5, 2019 at 4:16 AM Stephen Boyd wrote: > > Quoting Bjorn Andersson (2019-06-04 15:37:00) > > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > > > The SMMU that sits in front of the QUP needs to be programmed properly > > > so that the i2c geni driver can allocate DMA

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Bjorn Andersson
On Tue 04 Jun 15:46 PDT 2019, Stephen Boyd wrote: > Quoting Bjorn Andersson (2019-06-04 15:37:00) > > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > > > The SMMU that sits in front of the QUP needs to be programmed properly > > > so that the i2c geni driver can allocate DMA descriptors.

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Bjorn Andersson
On Tue 04 Jun 15:48 PDT 2019, Doug Anderson wrote: > Hi, > > On Tue, Jun 4, 2019 at 3:37 PM Bjorn Andersson > wrote: > > > > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > > > The SMMU that sits in front of the QUP needs to be programmed properly > > > so that the i2c geni driver can

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Doug Anderson
Hi, On Tue, Jun 4, 2019 at 3:37 PM Bjorn Andersson wrote: > > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > The SMMU that sits in front of the QUP needs to be programmed properly > > so that the i2c geni driver can allocate DMA descriptors. Failure to do > > this leads to faults when

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Stephen Boyd
Quoting Bjorn Andersson (2019-06-04 15:37:00) > On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > > > The SMMU that sits in front of the QUP needs to be programmed properly > > so that the i2c geni driver can allocate DMA descriptors. Failure to do > > this leads to faults when using devices

Re: [PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Bjorn Andersson
On Tue 04 Jun 15:29 PDT 2019, Stephen Boyd wrote: > The SMMU that sits in front of the QUP needs to be programmed properly > so that the i2c geni driver can allocate DMA descriptors. Failure to do > this leads to faults when using devices such as an i2c touchscreen where > the transaction is

[PATCH] arm64: dts: sdm845: Add iommus property to qup1

2019-06-04 Thread Stephen Boyd
The SMMU that sits in front of the QUP needs to be programmed properly so that the i2c geni driver can allocate DMA descriptors. Failure to do this leads to faults when using devices such as an i2c touchscreen where the transaction is larger than 32 bytes and we use a DMA buffer. arm-smmu