[4/6] dt-bindings: media: venus: describe sdm845 with optional pmdomains
diff mbox series

Message ID 20191209083824.806-5-stanimir.varbanov@linaro.org
State Changes Requested
Headers show
Series
  • Venus new features
Related show

Checks

Context Check Description
robh/checkpatch success

Commit Message

Stanimir Varbanov Dec. 9, 2019, 8:38 a.m. UTC
Describe an optional DT binding for sdm845 with power-domain-names
and clocks in Venus DT node. After that the binding for sdm845 will
have two options, the first will be to describe vcodec clocks and
pmdomains in (encoder and decoder) subnodes and the second will be
to describe vcodec clocks and pmdomains in the Venus DT node
(the parent). Both option will be handled in driver, and the second
one will give an opportunity to the driver to dynamically assign
vcodec pipelines for every given instance depending on its load.

Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
---
 .../devicetree/bindings/media/qcom,venus.txt  | 22 ++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

Comments

Stanimir Varbanov Dec. 10, 2019, 9:08 a.m. UTC | #1
Hi,

Please ignore this patch for now. I forgot to add description for
power-domain-names property for sdm845. Will fix that in next version.

On 12/9/19 10:38 AM, Stanimir Varbanov wrote:
> Describe an optional DT binding for sdm845 with power-domain-names
> and clocks in Venus DT node. After that the binding for sdm845 will
> have two options, the first will be to describe vcodec clocks and
> pmdomains in (encoder and decoder) subnodes and the second will be
> to describe vcodec clocks and pmdomains in the Venus DT node
> (the parent). Both option will be handled in driver, and the second
> one will give an opportunity to the driver to dynamically assign
> vcodec pipelines for every given instance depending on its load.
> 
> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
> ---
>  .../devicetree/bindings/media/qcom,venus.txt  | 22 ++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/qcom,venus.txt b/Documentation/devicetree/bindings/media/qcom,venus.txt
> index b602c4c025e7..69a104fcb8eb 100644
> --- a/Documentation/devicetree/bindings/media/qcom,venus.txt
> +++ b/Documentation/devicetree/bindings/media/qcom,venus.txt
> @@ -35,6 +35,21 @@
>  		- "iface"	Video accelerator AHB clock
>  		- "bus"		Video accelerator AXI clock
>  		- "mbus"	Video MAXI clock
> +- clock-names:
> +	Usage: required for sdm845
> +	Value type: <stringlist>
> +	Definition: Should contain the following entries:
> +		- "core"	Core video accelerator clock
> +		- "iface"	Video accelerator AHB clock
> +		- "bus"		Video accelerator AXI clock
> +- clock-names:
> +	Usage: optional for sdm845
> +	Value type: <stringlist>
> +	Definition: Should contain the following entries:
> +		- "vcodec0_core"	Codec0 core clock
> +		- "vcodec0_bus"		Codec0 AXI clock
> +		- "vcodec1_core"	Codec1 core clock
> +		- "vcoded1_bus"		Codec1 AXI clock
>  - power-domains:
>  	Usage: required
>  	Value type: <prop-encoded-array>
> @@ -72,7 +87,12 @@ Every of video-encoder or video-decoder subnode should have:
>  	Value type: <stringlist>
>  	Definition: Should contain the following entries:
>  		- "core"	Subcore video accelerator clock
> -
> +- clock-names:
> +	Usage: required for sdm845
> +	Value type: <stringlist>
> +	Definition: Should contain the following entries:
> +		- "core"	Video codec accelerator clock
> +		- "bus"		Video codec AXI clock
>  - power-domains:
>  	Usage: required for msm8996
>  	Value type: <prop-encoded-array>
>

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/media/qcom,venus.txt b/Documentation/devicetree/bindings/media/qcom,venus.txt
index b602c4c025e7..69a104fcb8eb 100644
--- a/Documentation/devicetree/bindings/media/qcom,venus.txt
+++ b/Documentation/devicetree/bindings/media/qcom,venus.txt
@@ -35,6 +35,21 @@ 
 		- "iface"	Video accelerator AHB clock
 		- "bus"		Video accelerator AXI clock
 		- "mbus"	Video MAXI clock
+- clock-names:
+	Usage: required for sdm845
+	Value type: <stringlist>
+	Definition: Should contain the following entries:
+		- "core"	Core video accelerator clock
+		- "iface"	Video accelerator AHB clock
+		- "bus"		Video accelerator AXI clock
+- clock-names:
+	Usage: optional for sdm845
+	Value type: <stringlist>
+	Definition: Should contain the following entries:
+		- "vcodec0_core"	Codec0 core clock
+		- "vcodec0_bus"		Codec0 AXI clock
+		- "vcodec1_core"	Codec1 core clock
+		- "vcoded1_bus"		Codec1 AXI clock
 - power-domains:
 	Usage: required
 	Value type: <prop-encoded-array>
@@ -72,7 +87,12 @@  Every of video-encoder or video-decoder subnode should have:
 	Value type: <stringlist>
 	Definition: Should contain the following entries:
 		- "core"	Subcore video accelerator clock
-
+- clock-names:
+	Usage: required for sdm845
+	Value type: <stringlist>
+	Definition: Should contain the following entries:
+		- "core"	Video codec accelerator clock
+		- "bus"		Video codec AXI clock
 - power-domains:
 	Usage: required for msm8996
 	Value type: <prop-encoded-array>