* [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
@ 2022-12-12 10:06 Konrad Dybcio
2022-12-12 10:58 ` Dmitry Baryshkov
0 siblings, 1 reply; 7+ messages in thread
From: Konrad Dybcio @ 2022-12-12 10:06 UTC (permalink / raw)
To: linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Konrad Dybcio, Will Deacon, Robin Murphy,
Joerg Roedel, Dmitry Baryshkov, Konrad Dybcio,
Sai Prakash Ranjan, Rob Clark, Adam Skladowski, linux-arm-kernel,
iommu, linux-kernel
From: Marijn Suijten <marijn.suijten@somainline.org>
Add the SM8150 DPU compatible to clients compatible list, as it also
needs the workarounds.
Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Depends on the binding here:
[1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
index 6610f5d2877d..cd74c1efcfde 100644
--- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
+++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
@@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
{ .compatible = "qcom,sdm845-mss-pil" },
{ .compatible = "qcom,sm6350-mdss" },
{ .compatible = "qcom,sm6375-mdss" },
+ { .compatible = "qcom,sm8150-mdss" },
{ .compatible = "qcom,sm8250-mdss" },
{ }
};
--
2.38.1
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
2022-12-12 10:06 [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible Konrad Dybcio
@ 2022-12-12 10:58 ` Dmitry Baryshkov
2022-12-12 11:00 ` Konrad Dybcio
0 siblings, 1 reply; 7+ messages in thread
From: Dmitry Baryshkov @ 2022-12-12 10:58 UTC (permalink / raw)
To: Konrad Dybcio, linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Will Deacon, Robin Murphy, Joerg Roedel,
Konrad Dybcio, Sai Prakash Ranjan, Rob Clark, Adam Skladowski,
linux-arm-kernel, iommu, linux-kernel
On 12 December 2022 13:06:16 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>From: Marijn Suijten <marijn.suijten@somainline.org>
>
>Add the SM8150 DPU compatible to clients compatible list, as it also
>needs the workarounds.
Please rebase on top of linux-next, this part was rewritten.
>
>Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
>Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>---
>Depends on the binding here:
>
>[1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
>
> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>index 6610f5d2877d..cd74c1efcfde 100644
>--- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>+++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>@@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
> { .compatible = "qcom,sdm845-mss-pil" },
> { .compatible = "qcom,sm6350-mdss" },
> { .compatible = "qcom,sm6375-mdss" },
>+ { .compatible = "qcom,sm8150-mdss" },
> { .compatible = "qcom,sm8250-mdss" },
> { }
> };
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
2022-12-12 10:58 ` Dmitry Baryshkov
@ 2022-12-12 11:00 ` Konrad Dybcio
2022-12-12 11:20 ` Dmitry Baryshkov
2022-12-12 11:27 ` Dmitry Baryshkov
0 siblings, 2 replies; 7+ messages in thread
From: Konrad Dybcio @ 2022-12-12 11:00 UTC (permalink / raw)
To: Dmitry Baryshkov, linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Will Deacon, Robin Murphy, Joerg Roedel,
Konrad Dybcio, Sai Prakash Ranjan, Rob Clark, Adam Skladowski,
linux-arm-kernel, iommu, linux-kernel
On 12.12.2022 11:58, Dmitry Baryshkov wrote:
>
>
> On 12 December 2022 13:06:16 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>> From: Marijn Suijten <marijn.suijten@somainline.org>
>>
>> Add the SM8150 DPU compatible to clients compatible list, as it also
>> needs the workarounds.
>
> Please rebase on top of linux-next, this part was rewritten.
This one wasn't, but yeah it is based on some local changes
(see 6350/6375)..
Konrad
>
>
>>
>> Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>> ---
>> Depends on the binding here:
>>
>> [1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
>>
>> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>> index 6610f5d2877d..cd74c1efcfde 100644
>> --- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>> +++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>> @@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
>> { .compatible = "qcom,sdm845-mss-pil" },
>> { .compatible = "qcom,sm6350-mdss" },
>> { .compatible = "qcom,sm6375-mdss" },
>> + { .compatible = "qcom,sm8150-mdss" },
>> { .compatible = "qcom,sm8250-mdss" },
>> { }
>> };
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
2022-12-12 11:00 ` Konrad Dybcio
@ 2022-12-12 11:20 ` Dmitry Baryshkov
2022-12-12 11:21 ` Konrad Dybcio
2022-12-12 11:27 ` Dmitry Baryshkov
1 sibling, 1 reply; 7+ messages in thread
From: Dmitry Baryshkov @ 2022-12-12 11:20 UTC (permalink / raw)
To: Konrad Dybcio, linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Will Deacon, Robin Murphy, Joerg Roedel,
Konrad Dybcio, Sai Prakash Ranjan, Rob Clark, Adam Skladowski,
linux-arm-kernel, iommu, linux-kernel
On 12 December 2022 14:00:10 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>
>
>On 12.12.2022 11:58, Dmitry Baryshkov wrote:
>>
>>
>> On 12 December 2022 13:06:16 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>> From: Marijn Suijten <marijn.suijten@somainline.org>
>>>
>>> Add the SM8150 DPU compatible to clients compatible list, as it also
>>> needs the workarounds.
>>
>> Please rebase on top of linux-next, this part was rewritten.
>This one wasn't, but yeah it is based on some local changes
>(see 6350/6375)..
It was,
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c?h=next-20221208#n498
>
>Konrad
>>
>>
>>>
>>> Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>> ---
>>> Depends on the binding here:
>>>
>>> [1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
>>>
>>> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>> index 6610f5d2877d..cd74c1efcfde 100644
>>> --- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>> +++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>> @@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
>>> { .compatible = "qcom,sdm845-mss-pil" },
>>> { .compatible = "qcom,sm6350-mdss" },
>>> { .compatible = "qcom,sm6375-mdss" },
>>> + { .compatible = "qcom,sm8150-mdss" },
>>> { .compatible = "qcom,sm8250-mdss" },
>>> { }
>>> };
>>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
2022-12-12 11:20 ` Dmitry Baryshkov
@ 2022-12-12 11:21 ` Konrad Dybcio
0 siblings, 0 replies; 7+ messages in thread
From: Konrad Dybcio @ 2022-12-12 11:21 UTC (permalink / raw)
To: Dmitry Baryshkov, linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Will Deacon, Robin Murphy, Joerg Roedel,
Konrad Dybcio, Sai Prakash Ranjan, Rob Clark, Adam Skladowski,
linux-arm-kernel, iommu, linux-kernel
On 12.12.2022 12:20, Dmitry Baryshkov wrote:
>
>
> On 12 December 2022 14:00:10 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>
>>
>> On 12.12.2022 11:58, Dmitry Baryshkov wrote:
>>>
>>>
>>> On 12 December 2022 13:06:16 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>>> From: Marijn Suijten <marijn.suijten@somainline.org>
>>>>
>>>> Add the SM8150 DPU compatible to clients compatible list, as it also
>>>> needs the workarounds.
>>>
>>> Please rebase on top of linux-next, this part was rewritten.
>> This one wasn't, but yeah it is based on some local changes
>> (see 6350/6375)..
>
> It was,
>
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c?h=next-20221208#n498
We're not looking at the same thing, this patch adds a compatible
for 8150's mdss so that it can be matched as a client, not a
compatible for the SMMU itself (which itself has been merged
dinosaur years ago).
Konrad
>
>>
>> Konrad
>>>
>>>
>>>>
>>>> Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
>>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>>> ---
>>>> Depends on the binding here:
>>>>
>>>> [1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
>>>>
>>>> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>>> index 6610f5d2877d..cd74c1efcfde 100644
>>>> --- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>>> +++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>>> @@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
>>>> { .compatible = "qcom,sdm845-mss-pil" },
>>>> { .compatible = "qcom,sm6350-mdss" },
>>>> { .compatible = "qcom,sm6375-mdss" },
>>>> + { .compatible = "qcom,sm8150-mdss" },
>>>> { .compatible = "qcom,sm8250-mdss" },
>>>> { }
>>>> };
>>>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
2022-12-12 11:00 ` Konrad Dybcio
2022-12-12 11:20 ` Dmitry Baryshkov
@ 2022-12-12 11:27 ` Dmitry Baryshkov
2022-12-12 11:33 ` Konrad Dybcio
1 sibling, 1 reply; 7+ messages in thread
From: Dmitry Baryshkov @ 2022-12-12 11:27 UTC (permalink / raw)
To: Konrad Dybcio, linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Will Deacon, Robin Murphy, Joerg Roedel,
Konrad Dybcio, Sai Prakash Ranjan, Rob Clark, Adam Skladowski,
linux-arm-kernel, iommu, linux-kernel
On 12 December 2022 14:00:10 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>
>
>On 12.12.2022 11:58, Dmitry Baryshkov wrote:
>>
>>
>> On 12 December 2022 13:06:16 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>> From: Marijn Suijten <marijn.suijten@somainline.org>
>>>
>>> Add the SM8150 DPU compatible to clients compatible list, as it also
>>> needs the workarounds.
>>
>> Please rebase on top of linux-next, this part was rewritten.
>This one wasn't, but yeah it is based on some local changes
>(see 6350/6375)..
Stupid me, you are right here. This part wasn't changed. Please ignore the comment.
>
>Konrad
>>
>>
>>>
>>> Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>> ---
>>> Depends on the binding here:
>>>
>>> [1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
>>>
>>> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>> index 6610f5d2877d..cd74c1efcfde 100644
>>> --- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>> +++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>> @@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
>>> { .compatible = "qcom,sdm845-mss-pil" },
>>> { .compatible = "qcom,sm6350-mdss" },
>>> { .compatible = "qcom,sm6375-mdss" },
>>> + { .compatible = "qcom,sm8150-mdss" },
>>> { .compatible = "qcom,sm8250-mdss" },
>>> { }
>>> };
>>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible
2022-12-12 11:27 ` Dmitry Baryshkov
@ 2022-12-12 11:33 ` Konrad Dybcio
0 siblings, 0 replies; 7+ messages in thread
From: Konrad Dybcio @ 2022-12-12 11:33 UTC (permalink / raw)
To: Dmitry Baryshkov, linux-arm-msm, andersson, agross, krzysztof.kozlowski
Cc: marijn.suijten, Will Deacon, Robin Murphy, Joerg Roedel,
Konrad Dybcio, Sai Prakash Ranjan, Rob Clark, Adam Skladowski,
linux-arm-kernel, iommu, linux-kernel
On 12.12.2022 12:27, Dmitry Baryshkov wrote:
>
>
> On 12 December 2022 14:00:10 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>
>>
>> On 12.12.2022 11:58, Dmitry Baryshkov wrote:
>>>
>>>
>>> On 12 December 2022 13:06:16 GMT+03:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>>> From: Marijn Suijten <marijn.suijten@somainline.org>
>>>>
>>>> Add the SM8150 DPU compatible to clients compatible list, as it also
>>>> needs the workarounds.
>>>
>>> Please rebase on top of linux-next, this part was rewritten.
>> This one wasn't, but yeah it is based on some local changes
>> (see 6350/6375)..
>
> Stupid me, you are right here. This part wasn't changed. Please ignore the comment.
As I mentioned, it was accidentally right, as this doesn't apply
on plain next ^^
Konrad
>
>>
>> Konrad
>>>
>>>
>>>>
>>>> Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org>
>>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>>> ---
>>>> Depends on the binding here:
>>>>
>>>> [1] https://lore.kernel.org/linux-arm-msm/20221212093315.11390-1-konrad.dybcio@linaro.org/T/#t
>>>>
>>>> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>>> index 6610f5d2877d..cd74c1efcfde 100644
>>>> --- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>>> +++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c
>>>> @@ -255,6 +255,7 @@ static const struct of_device_id qcom_smmu_client_of_match[] __maybe_unused = {
>>>> { .compatible = "qcom,sdm845-mss-pil" },
>>>> { .compatible = "qcom,sm6350-mdss" },
>>>> { .compatible = "qcom,sm6375-mdss" },
>>>> + { .compatible = "qcom,sm8150-mdss" },
>>>> { .compatible = "qcom,sm8250-mdss" },
>>>> { }
>>>> };
>>>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-12-12 11:39 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-12 10:06 [PATCH] iommu/arm-smmu-qcom: Add SM8150 DPU compatible Konrad Dybcio
2022-12-12 10:58 ` Dmitry Baryshkov
2022-12-12 11:00 ` Konrad Dybcio
2022-12-12 11:20 ` Dmitry Baryshkov
2022-12-12 11:21 ` Konrad Dybcio
2022-12-12 11:27 ` Dmitry Baryshkov
2022-12-12 11:33 ` Konrad Dybcio
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®