2020-11-02 11:39:42

by Stanimir Varbanov

Subject: [PATCH 15/17] arm64: dts: sdm845: Add interconnect properties for Venus

Populate Venus DT node with interconnect properties.

Signed-off-by: Stanimir Varbanov <[email protected]>
---
arch/arm64/boot/dts/qcom/sdm845.dtsi | 3 +++
1 file changed, 3 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index 40e8c11f23ab..aca7e9c954e0 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -3661,6 +3661,9 @@
iommus = <&apps_smmu 0x10a0 0x8>,
<&apps_smmu 0x10b0 0x0>;
memory-region = <&venus_mem>;
+ interconnects = <&mmss_noc MASTER_VIDEO_P0 0 &mem_noc SLAVE_EBI1 0>,
+ <&gladiator_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_VENUS_CFG 0>;
+ interconnect-names = "video-mem", "cpu-cfg";

video-core0 {
compatible = "venus-decoder";
--
2.17.1

2020-11-02 12:09:39

by Georgi Djakov

Subject: Re: [PATCH 15/17] arm64: dts: sdm845: Add interconnect properties for Venus

Hi Stan,

On 11/2/20 13:35, Stanimir Varbanov wrote:
> Populate Venus DT node with interconnect properties.
>
> Signed-off-by: Stanimir Varbanov <[email protected]>

Reviewed-by: Georgi Djakov <[email protected]>

Thanks!
Georgi

> ---
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> index 40e8c11f23ab..aca7e9c954e0 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> @@ -3661,6 +3661,9 @@
> iommus = <&apps_smmu 0x10a0 0x8>,
> <&apps_smmu 0x10b0 0x0>;
> memory-region = <&venus_mem>;
> + interconnects = <&mmss_noc MASTER_VIDEO_P0 0 &mem_noc SLAVE_EBI1 0>,
> + <&gladiator_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_VENUS_CFG 0>;
> + interconnect-names = "video-mem", "cpu-cfg";
>
> video-core0 {
> compatible = "venus-decoder";
>

2020-11-30 10:44:32

by Stanimir Varbanov

Subject: Re: [PATCH 15/17] arm64: dts: sdm845: Add interconnect properties for Venus

Bjorn, could you take this for v5.11.

On 11/2/20 2:07 PM, Georgi Djakov wrote:
> Hi Stan,
>
> On 11/2/20 13:35, Stanimir Varbanov wrote:
>> Populate Venus DT node with interconnect properties.
>>
>> Signed-off-by: Stanimir Varbanov <[email protected]>
>
> Reviewed-by: Georgi Djakov <[email protected]>
>
> Thanks!
> Georgi
>
>> ---
>> arch/arm64/boot/dts/qcom/sdm845.dtsi | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
>> index 40e8c11f23ab..aca7e9c954e0 100644
>> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
>> @@ -3661,6 +3661,9 @@
>> iommus = <&apps_smmu 0x10a0 0x8>,
>> <&apps_smmu 0x10b0 0x0>;
>> memory-region = <&venus_mem>;
>> + interconnects = <&mmss_noc MASTER_VIDEO_P0 0 &mem_noc SLAVE_EBI1 0>,
>> + <&gladiator_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_VENUS_CFG 0>;
>> + interconnect-names = "video-mem", "cpu-cfg";
>>
>> video-core0 {
>> compatible = "venus-decoder";
>>

--
regards,
Stan

2020-11-30 17:01:01

by Bjorn Andersson

Subject: Re: [PATCH 15/17] arm64: dts: sdm845: Add interconnect properties for Venus

On Mon 30 Nov 04:34 CST 2020, Stanimir Varbanov wrote:

> Bjorn, could you take this for v5.11.
>

I did apply it a few weeks back, but seems I forgot to reply (and it was
before I asked patchwork-bot to do so for me).

Regards,
Bjorn

> On 11/2/20 2:07 PM, Georgi Djakov wrote:
> > Hi Stan,
> >
> > On 11/2/20 13:35, Stanimir Varbanov wrote:
> >> Populate Venus DT node with interconnect properties.
> >>
> >> Signed-off-by: Stanimir Varbanov <[email protected]>
> >
> > Reviewed-by: Georgi Djakov <[email protected]>
> >
> > Thanks!
> > Georgi
> >
> >> ---
> >> arch/arm64/boot/dts/qcom/sdm845.dtsi | 3 +++
> >> 1 file changed, 3 insertions(+)
> >>
> >> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> >> index 40e8c11f23ab..aca7e9c954e0 100644
> >> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> >> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> >> @@ -3661,6 +3661,9 @@
> >> iommus = <&apps_smmu 0x10a0 0x8>,
> >> <&apps_smmu 0x10b0 0x0>;
> >> memory-region = <&venus_mem>;
> >> + interconnects = <&mmss_noc MASTER_VIDEO_P0 0 &mem_noc SLAVE_EBI1 0>,
> >> + <&gladiator_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_VENUS_CFG 0>;
> >> + interconnect-names = "video-mem", "cpu-cfg";
> >>
> >> video-core0 {
> >> compatible = "venus-decoder";
> >>
>
> --
> regards,
> Stan