Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-28 Thread Bjorn Andersson
On Mon 25 May 04:53 PDT 2020, Jonathan Marek wrote: [..] > I guess the commit message is ambiguous, that's not what I meant. Is "Now > that the kernel initializes the iommu, the bypass mappings set by the > bootloader are cleared. Adding the iommus property is required so that new > mappings are

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Sai Prakash Ranjan
On 2020-05-25 17:23, Jonathan Marek wrote: On 5/25/20 7:40 AM, Sai Prakash Ranjan wrote: On 2020-05-25 16:57, Jonathan Marek wrote: On 5/25/20 7:17 AM, Sai Prakash Ranjan wrote: Hi, On 2020-05-25 16:38, Jonathan Marek wrote: On 5/25/20 6:54 AM, Sai Prakash Ranjan wrote: On 2020-05-25

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Jonathan Marek
On 5/25/20 7:40 AM, Sai Prakash Ranjan wrote: On 2020-05-25 16:57, Jonathan Marek wrote: On 5/25/20 7:17 AM, Sai Prakash Ranjan wrote: Hi, On 2020-05-25 16:38, Jonathan Marek wrote: On 5/25/20 6:54 AM, Sai Prakash Ranjan wrote: On 2020-05-25 15:39, Jonathan Marek wrote: Hi, On 5/25/20

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Sai Prakash Ranjan
On 2020-05-25 16:57, Jonathan Marek wrote: On 5/25/20 7:17 AM, Sai Prakash Ranjan wrote: Hi, On 2020-05-25 16:38, Jonathan Marek wrote: On 5/25/20 6:54 AM, Sai Prakash Ranjan wrote: On 2020-05-25 15:39, Jonathan Marek wrote: Hi, On 5/25/20 5:42 AM, Sai Prakash Ranjan wrote: Hi Jonathan,

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Jonathan Marek
On 5/25/20 7:17 AM, Sai Prakash Ranjan wrote: Hi, On 2020-05-25 16:38, Jonathan Marek wrote: On 5/25/20 6:54 AM, Sai Prakash Ranjan wrote: On 2020-05-25 15:39, Jonathan Marek wrote: Hi, On 5/25/20 5:42 AM, Sai Prakash Ranjan wrote: Hi Jonathan, On 2020-05-24 08:08, Jonathan Marek wrote:

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Sai Prakash Ranjan
Hi, On 2020-05-25 16:38, Jonathan Marek wrote: On 5/25/20 6:54 AM, Sai Prakash Ranjan wrote: On 2020-05-25 15:39, Jonathan Marek wrote: Hi, On 5/25/20 5:42 AM, Sai Prakash Ranjan wrote: Hi Jonathan, On 2020-05-24 08:08, Jonathan Marek wrote: Add the apps_smmu node for sm8250. Note that

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Jonathan Marek
On 5/25/20 6:54 AM, Sai Prakash Ranjan wrote: On 2020-05-25 15:39, Jonathan Marek wrote: Hi, On 5/25/20 5:42 AM, Sai Prakash Ranjan wrote: Hi Jonathan, On 2020-05-24 08:08, Jonathan Marek wrote: Add the apps_smmu node for sm8250. Note that adding the iommus field for UFS is required because

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Sai Prakash Ranjan
On 2020-05-25 15:39, Jonathan Marek wrote: Hi, On 5/25/20 5:42 AM, Sai Prakash Ranjan wrote: Hi Jonathan, On 2020-05-24 08:08, Jonathan Marek wrote: Add the apps_smmu node for sm8250. Note that adding the iommus field for UFS is required because initializing the iommu removes the bypass

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Jonathan Marek
Hi, On 5/25/20 5:42 AM, Sai Prakash Ranjan wrote: Hi Jonathan, On 2020-05-24 08:08, Jonathan Marek wrote: Add the apps_smmu node for sm8250. Note that adding the iommus field for UFS is required because initializing the iommu removes the bypass mapping that created by the bootloader. This

Re: [PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-25 Thread Sai Prakash Ranjan
Hi Jonathan, On 2020-05-24 08:08, Jonathan Marek wrote: Add the apps_smmu node for sm8250. Note that adding the iommus field for UFS is required because initializing the iommu removes the bypass mapping that created by the bootloader. This statement doesn't seem right, you can just say

[PATCH 2/6] arm64: dts: qcom: sm8250: add apps_smmu node

2020-05-23 Thread Jonathan Marek
Add the apps_smmu node for sm8250. Note that adding the iommus field for UFS is required because initializing the iommu removes the bypass mapping that created by the bootloader. Signed-off-by: Jonathan Marek --- arch/arm64/boot/dts/qcom/sm8250.dtsi | 107 +++ 1 file