On 5/26/26 3:24 PM, Erikas Bitovtas wrote:
> From: AndrĂ© Apitzsch <[email protected]>
> 
> Add DT entries for the msm8939 venus encoder/decoder.
> 
> Signed-off-by: AndrĂ© Apitzsch <[email protected]>
> Reviewed-by: Bryan O'Donoghue <[email protected]>
> Signed-off-by: Erikas Bitovtas <[email protected]>
> ---
>  arch/arm64/boot/dts/qcom/msm8939.dtsi | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8939.dtsi 
> b/arch/arm64/boot/dts/qcom/msm8939.dtsi
> index d4d7b0c9206c..71e22584a46b 100644
> --- a/arch/arm64/boot/dts/qcom/msm8939.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8939.dtsi
> @@ -1627,6 +1627,29 @@ opp-19200000 {
>                       };
>               };
>  
> +             venus: video-codec@1d00000 {
> +                     compatible = "qcom,msm8939-venus";
> +                     reg = <0x01d00000 0xff000>;
> +                     interrupts = <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>;
> +                     clocks = <&gcc GCC_VENUS0_VCODEC0_CLK>,
> +                              <&gcc GCC_VENUS0_AHB_CLK>,
> +                              <&gcc GCC_VENUS0_AXI_CLK>,
> +                              <&gcc GCC_VENUS0_CORE0_VCODEC0_CLK>,
> +                              <&gcc GCC_VENUS0_CORE1_VCODEC0_CLK>;
> +                     clock-names = "core",
> +                                   "iface",
> +                                   "bus",
> +                                   "vcodec0_core",
> +                                   "vcodec1_core";
> +                     power-domains = <&gcc VENUS_GDSC>,
> +                                     <&gcc VENUS_CORE0_GDSC>,
> +                                     <&gcc VENUS_CORE1_GDSC>;
> +                     power-domain-names = "venus", "vcodec0", "vcodec1";
> +                     iommus = <&apps_iommu 5>;
> +                     memory-region = <&venus_mem>;
> +                     status = "disabled";
> +             };

Reviewed-by: Konrad Dybcio <[email protected]>

in case you're going to resend, please add a \n before 'status'
and make power-domain-names a vertical list (like clock-names)

Konrad

Reply via email to