From: V Sujith Kumar Reddy <[email protected]>
Update iommu property in lpass cpu node for supporting
simultaneous playback on headset and speaker.
Signed-off-by: V Sujith Kumar Reddy <[email protected]>
Signed-off-by: Srinivasa Rao Mandadapu <[email protected]>
---
Changes since v1:
-- Commit messge header change
arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index a6da78d31fdd..6228ba2d8513 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -3566,7 +3566,8 @@ lpass_cpu: lpass@62f00000 {
reg = <0 0x62f00000 0 0x29000>;
reg-names = "lpass-lpaif";
- iommus = <&apps_smmu 0x1020 0>;
+ iommus = <&apps_smmu 0x1020 0>,
+ <&apps_smmu 0x1021 0>;
power-domains = <&lpass_hm LPASS_CORE_HM_GDSCR>;
--
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
Quoting Srinivasa Rao Mandadapu (2021-04-06 09:33:30)
> From: V Sujith Kumar Reddy <[email protected]>
>
> Update iommu property in lpass cpu node for supporting
> simultaneous playback on headset and speaker.
>
> Signed-off-by: V Sujith Kumar Reddy <[email protected]>
> Signed-off-by: Srinivasa Rao Mandadapu <[email protected]>
> ---
> Changes since v1:
> -- Commit messge header change
>
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> index a6da78d31fdd..6228ba2d8513 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -3566,7 +3566,8 @@ lpass_cpu: lpass@62f00000 {
> reg = <0 0x62f00000 0 0x29000>;
> reg-names = "lpass-lpaif";
>
> - iommus = <&apps_smmu 0x1020 0>;
> + iommus = <&apps_smmu 0x1020 0>,
> + <&apps_smmu 0x1021 0>;
The stream ID 0x1032 was also dropped in this version but there's no
mention of that in the changelog. Why?
>
> power-domains = <&lpass_hm LPASS_CORE_HM_GDSCR>;
>
Hi Stephen.
Thanks for your time!!!
On 4/9/2021 10:31 PM, Stephen Boyd wrote:
> Quoting Srinivasa Rao Mandadapu (2021-04-06 09:33:30)
>> From: V Sujith Kumar Reddy <[email protected]>
>>
>> Update iommu property in lpass cpu node for supporting
>> simultaneous playback on headset and speaker.
>>
>> Signed-off-by: V Sujith Kumar Reddy <[email protected]>
>> Signed-off-by: Srinivasa Rao Mandadapu <[email protected]>
>> ---
>> Changes since v1:
>> -- Commit messge header change
>>
>> arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> index a6da78d31fdd..6228ba2d8513 100644
>> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> @@ -3566,7 +3566,8 @@ lpass_cpu: lpass@62f00000 {
>> reg = <0 0x62f00000 0 0x29000>;
>> reg-names = "lpass-lpaif";
>>
>> - iommus = <&apps_smmu 0x1020 0>;
>> + iommus = <&apps_smmu 0x1020 0>,
>> + <&apps_smmu 0x1021 0>;
> The stream ID 0x1032 was also dropped in this version but there's no
> mention of that in the changelog. Why?
That is ID is for HDMI Stream, so as part of DP patches that will be added.
>
>>
>> power-domains = <&lpass_hm LPASS_CORE_HM_GDSCR>;
>>
--
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
Quoting Srinivasa Rao Mandadapu (2021-04-09 22:17:07)
> Hi Stephen.
>
> Thanks for your time!!!
>
>
> On 4/9/2021 10:31 PM, Stephen Boyd wrote:
> > Quoting Srinivasa Rao Mandadapu (2021-04-06 09:33:30)
> >> From: V Sujith Kumar Reddy <[email protected]>
> >>
> >> Update iommu property in lpass cpu node for supporting
> >> simultaneous playback on headset and speaker.
> >>
> >> Signed-off-by: V Sujith Kumar Reddy <[email protected]>
> >> Signed-off-by: Srinivasa Rao Mandadapu <[email protected]>
> >> ---
> >> Changes since v1:
> >> -- Commit messge header change
> >>
> >> arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 ++-
> >> 1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> >> index a6da78d31fdd..6228ba2d8513 100644
> >> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> >> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> >> @@ -3566,7 +3566,8 @@ lpass_cpu: lpass@62f00000 {
> >> reg = <0 0x62f00000 0 0x29000>;
> >> reg-names = "lpass-lpaif";
> >>
> >> - iommus = <&apps_smmu 0x1020 0>;
> >> + iommus = <&apps_smmu 0x1020 0>,
> >> + <&apps_smmu 0x1021 0>;
> > The stream ID 0x1032 was also dropped in this version but there's no
> > mention of that in the changelog. Why?
> That is ID is for HDMI Stream, so as part of DP patches that will be added.
Ok, got it.
Quoting Srinivasa Rao Mandadapu (2021-04-06 09:33:30)
> From: V Sujith Kumar Reddy <[email protected]>
>
> Update iommu property in lpass cpu node for supporting
> simultaneous playback on headset and speaker.
>
> Signed-off-by: V Sujith Kumar Reddy <[email protected]>
> Signed-off-by: Srinivasa Rao Mandadapu <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>