Message ID | 1473248229-5540-2-git-send-email-stanimir.varbanov@linaro.org |
---|---|
State | Changes Requested, archived |
Headers | show |
On Wed, Sep 07, 2016 at 02:37:02PM +0300, Stanimir Varbanov wrote: > Adds binding document for vidc video encoder/decoder driver > > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: devicetree@vger.kernel.org > Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> > --- > .../devicetree/bindings/media/qcom,vidc.txt | 61 ++++++++++++++++++++++ > 1 file changed, 61 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/qcom,vidc.txt > > diff --git a/Documentation/devicetree/bindings/media/qcom,vidc.txt b/Documentation/devicetree/bindings/media/qcom,vidc.txt > new file mode 100644 > index 000000000000..0d50a7b2e3ed > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/qcom,vidc.txt > @@ -0,0 +1,61 @@ > +* Qualcomm video encoder/decoder accelerator > + > +- compatible: > + Usage: required > + Value type: <stringlist> > + Definition: Value should contain ... one of: > + - "qcom,vidc-msm8916" > + - "qcom,vidc-msm8996" > +- reg: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: Register ranges as listed in the reg-names property > + > +- interrupts: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: How many interrupts? > + > +- power-domains: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: A phandle and power domain specifier pairs to the > + power domain which is responsible for collapsing > + and restoring power to the peripheral How many power domains? > + > +- clocks: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: List of phandle and clock specifier pairs as listed > + in clock-names property > +- clock-names: > + Usage: required > + Value type: <stringlist> > + Definition: Should contain the following entries > + - "core" Core video accelerator clock > + - "iface" Video accelerator AHB clock > + - "bus" Video accelerator AXI clock > +- rproc: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: A phandle to remote processor responsible for > + firmware loading > + > +- iommus: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: A list of phandle and IOMMU specifier pairs > + > +* An Example > + qcom,vidc@1d00000 { node names should be generic: video-codec@ > + compatible = "qcom,vidc-msm8916"; > + reg = <0x01d00000 0xff000>; > + clocks = <&gcc GCC_VENUS0_VCODEC0_CLK>, > + <&gcc GCC_VENUS0_AHB_CLK>, > + <&gcc GCC_VENUS0_AXI_CLK>; > + clock-names = "core", "iface", "bus"; > + interrupts = <GIC_SPI 44 0>; > + power-domains = <&gcc VENUS_GDSC>; > + rproc = <&vidc_rproc>; > + iommus = <&apps_iommu 5>; > + }; > -- > 2.7.4 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Rob, Thanks for the review! On 09/16/2016 05:19 PM, Rob Herring wrote: > On Wed, Sep 07, 2016 at 02:37:02PM +0300, Stanimir Varbanov wrote: >> Adds binding document for vidc video encoder/decoder driver >> >> Cc: Rob Herring <robh+dt@kernel.org> >> Cc: Mark Rutland <mark.rutland@arm.com> >> Cc: devicetree@vger.kernel.org >> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> >> --- >> .../devicetree/bindings/media/qcom,vidc.txt | 61 ++++++++++++++++++++++ >> 1 file changed, 61 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/media/qcom,vidc.txt >> >> diff --git a/Documentation/devicetree/bindings/media/qcom,vidc.txt b/Documentation/devicetree/bindings/media/qcom,vidc.txt >> new file mode 100644 >> index 000000000000..0d50a7b2e3ed >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/qcom,vidc.txt >> @@ -0,0 +1,61 @@ >> +* Qualcomm video encoder/decoder accelerator >> + >> +- compatible: >> + Usage: required >> + Value type: <stringlist> >> + Definition: Value should contain > > ... one of: > >> + - "qcom,vidc-msm8916" >> + - "qcom,vidc-msm8996" >> +- reg: >> + Usage: required >> + Value type: <prop-encoded-array> >> + Definition: Register ranges as listed in the reg-names property >> + >> +- interrupts: >> + Usage: required >> + Value type: <prop-encoded-array> >> + Definition: > > How many interrupts? It is one, thanks for the catch. > >> + >> +- power-domains: >> + Usage: required >> + Value type: <prop-encoded-array> >> + Definition: A phandle and power domain specifier pairs to the >> + power domain which is responsible for collapsing >> + and restoring power to the peripheral > > How many power domains? Good question, for vidc-msm8916 it is one power-domain, for vidc-msm8996 the power domains should be 3. Here the problem is that the genpd doesn't permit more than one power-domain per device. > >> + >> +- clocks: >> + Usage: required >> + Value type: <prop-encoded-array> >> + Definition: List of phandle and clock specifier pairs as listed >> + in clock-names property >> +- clock-names: >> + Usage: required >> + Value type: <stringlist> >> + Definition: Should contain the following entries >> + - "core" Core video accelerator clock >> + - "iface" Video accelerator AHB clock >> + - "bus" Video accelerator AXI clock >> +- rproc: >> + Usage: required >> + Value type: <prop-encoded-array> >> + Definition: A phandle to remote processor responsible for >> + firmware loading >> + >> +- iommus: >> + Usage: required >> + Value type: <prop-encoded-array> >> + Definition: A list of phandle and IOMMU specifier pairs >> + >> +* An Example >> + qcom,vidc@1d00000 { > > node names should be generic: video-codec@ correct, will update it in next version.
diff --git a/Documentation/devicetree/bindings/media/qcom,vidc.txt b/Documentation/devicetree/bindings/media/qcom,vidc.txt new file mode 100644 index 000000000000..0d50a7b2e3ed --- /dev/null +++ b/Documentation/devicetree/bindings/media/qcom,vidc.txt @@ -0,0 +1,61 @@ +* Qualcomm video encoder/decoder accelerator + +- compatible: + Usage: required + Value type: <stringlist> + Definition: Value should contain + - "qcom,vidc-msm8916" + - "qcom,vidc-msm8996" +- reg: + Usage: required + Value type: <prop-encoded-array> + Definition: Register ranges as listed in the reg-names property + +- interrupts: + Usage: required + Value type: <prop-encoded-array> + Definition: + +- power-domains: + Usage: required + Value type: <prop-encoded-array> + Definition: A phandle and power domain specifier pairs to the + power domain which is responsible for collapsing + and restoring power to the peripheral + +- clocks: + Usage: required + Value type: <prop-encoded-array> + Definition: List of phandle and clock specifier pairs as listed + in clock-names property +- clock-names: + Usage: required + Value type: <stringlist> + Definition: Should contain the following entries + - "core" Core video accelerator clock + - "iface" Video accelerator AHB clock + - "bus" Video accelerator AXI clock +- rproc: + Usage: required + Value type: <prop-encoded-array> + Definition: A phandle to remote processor responsible for + firmware loading + +- iommus: + Usage: required + Value type: <prop-encoded-array> + Definition: A list of phandle and IOMMU specifier pairs + +* An Example + qcom,vidc@1d00000 { + compatible = "qcom,vidc-msm8916"; + reg = <0x01d00000 0xff000>; + clocks = <&gcc GCC_VENUS0_VCODEC0_CLK>, + <&gcc GCC_VENUS0_AHB_CLK>, + <&gcc GCC_VENUS0_AXI_CLK>; + clock-names = "core", "iface", "bus"; + interrupts = <GIC_SPI 44 0>; + power-domains = <&gcc VENUS_GDSC>; + rproc = <&vidc_rproc>; + iommus = <&apps_iommu 5>; + };
Adds binding document for vidc video encoder/decoder driver Cc: Rob Herring <robh+dt@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: devicetree@vger.kernel.org Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> --- .../devicetree/bindings/media/qcom,vidc.txt | 61 ++++++++++++++++++++++ 1 file changed, 61 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/qcom,vidc.txt