| Message ID | 1629453791-10580-1-git-send-email-shengjiu.wang@nxp.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/2] dt-bindings: remoteproc: Add fsl,imx-dsp-rproc binding document | expand |
On Fri, Aug 20, 2021 at 06:03:10PM +0800, Shengjiu Wang wrote: > Define the compatible string and properties needed by imx_dsp_rproc > driver. > > Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> > --- > .../remoteproc/fsl,imx-dsp-rproc.yaml | 123 ++++++++++++++++++ > 1 file changed, 123 insertions(+) > create mode 100644 Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml > > diff --git a/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml > new file mode 100644 > index 000000000000..7eba49aa959e > --- /dev/null > +++ b/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml > @@ -0,0 +1,123 @@ > +# SPDX-License-Identifier: (GPL-2.0-only or BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/remoteproc/fsl,imx-dsp-rproc.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: i.MX DSP Remoteproc Devices > + > +maintainers: > + - Shengjiu Wang <shengjiu.wang@nxp.com> > + > +description: > + This binding provides support for DSP processors found on i.mX family of SoCs > + > +properties: > + compatible: > + enum: > + - fsl,imx8qxp-hifi4 > + - fsl,imx8qm-hifi4 > + - fsl,imx8mp-hifi4 > + - fsl,imx8ulp-hifi4 > + > + clocks: > + description: | > + Main functional clock for the remote processor Don't need '|' if no formatting. > + minItems: 1 > + maxItems: 32 > + > + clock-names: > + description: > + List of clock names for the remote processor. > + dsp_clkx for clocks of dsp itself. > + per_clkx for clocks of peripherals used by dsp. Sounds like constraints that should be a schema. But this looks like some random collection of clocks, not clocks for the DSP h/w. > + minItems: 1 > + maxItems: 32 > + > + syscon: Vendor prefix and name it based on what it points to or is used for. > + $ref: /schemas/types.yaml#/definitions/phandle > + description: > + Phandle to syscon block which provide access for processor enablement > + > + mbox-names: > + items: > + - const: tx > + - const: rx > + - const: rxdb > + > + mboxes: > + description: > + This property is required only if the rpmsg/virtio functionality is used. > + List of <&phandle type channel> - 1 channel for TX, 1 channel for RX, 1 channel for RXDB. > + (see mailbox/fsl,mu.yaml) > + minItems: 1 > + maxItems: 3 > + > + firmware-name: > + description: | > + Default name of the firmware to load to the remote processor. > + > + memory-region: > + description: > + If present, a phandle for a reserved memory area that used for vdev buffer, > + resource table, vring region and others used by remote processor. > + minItems: 1 > + maxItems: 32 > + > + reg: > + description: | > + Address space for any remoteproc memories present on the SoC. > + > + power-domains: > + minItems: 1 > + maxItems: 32 > + > +required: > + - compatible > + - reg > + - mboxes > + - mbox-names > + - clocks > + - clock-names > + - firmware-name > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/imx8mp-clock.h> > + dsp_reserved: dsp@92400000 { > + reg = <0x92400000 0x1000000>; > + no-map; > + }; > + dsp_vdev0vring0: vdev0vring0@942f0000 { > + reg = <0x942f0000 0x8000>; > + no-map; > + }; > + dsp_vdev0vring1: vdev0vring1@942f8000 { > + reg = <0x942f8000 0x8000>; > + no-map; > + }; > + dsp_vdev0buffer: vdev0buffer@94300000 { > + compatible = "shared-dma-pool"; > + reg = <0x94300000 0x100000>; > + no-map; > + }; > + > + dsp: dsp@3b6e8000 { > + compatible = "fsl,imx8mp-hifi4"; > + reg = <0x3B6E8000 0x88000>; > + clocks = <&audio_blk_ctrl IMX8MP_CLK_AUDIOMIX_OCRAMA_IPG>, > + <&audio_blk_ctrl IMX8MP_CLK_AUDIOMIX_DSP_ROOT>, > + <&audio_blk_ctrl IMX8MP_CLK_AUDIOMIX_DSPDBG_ROOT>; > + clock-names = "dsp_clk1", "dsp_clk2", "dsp_clk3"; > + firmware-name = "imx/dsp/hifi4.bin"; > + power-domains = <&audiomix_pd>; > + mbox-names = "tx", "rx", "rxdb"; > + mboxes = <&mu2 0 0>, > + <&mu2 1 0>, > + <&mu2 3 0>; > + memory-region = <&dsp_vdev0buffer>, <&dsp_vdev0vring0>, > + <&dsp_vdev0vring1>, <&dsp_reserved>; > + syscon = <&audio_blk_ctrl>; > + }; > -- > 2.17.1 > >
Hi Rob On Sat, Aug 21, 2021 at 4:28 AM Rob Herring <robh@kernel.org> wrote: > > On Fri, Aug 20, 2021 at 06:03:10PM +0800, Shengjiu Wang wrote: > > Define the compatible string and properties needed by imx_dsp_rproc > > driver. > > > > Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> > > --- > > .../remoteproc/fsl,imx-dsp-rproc.yaml | 123 ++++++++++++++++++ > > 1 file changed, 123 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml > > > > diff --git a/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml > > new file mode 100644 > > index 000000000000..7eba49aa959e > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml > > @@ -0,0 +1,123 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only or BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/remoteproc/fsl,imx-dsp-rproc.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: i.MX DSP Remoteproc Devices > > + > > +maintainers: > > + - Shengjiu Wang <shengjiu.wang@nxp.com> > > + > > +description: > > + This binding provides support for DSP processors found on i.mX family of SoCs > > + > > +properties: > > + compatible: > > + enum: > > + - fsl,imx8qxp-hifi4 > > + - fsl,imx8qm-hifi4 > > + - fsl,imx8mp-hifi4 > > + - fsl,imx8ulp-hifi4 > > + > > + clocks: > > + description: | > > + Main functional clock for the remote processor > > Don't need '|' if no formatting. Ok, will update it. > > > + minItems: 1 > > + maxItems: 32 > > + > > + clock-names: > > + description: > > + List of clock names for the remote processor. > > + dsp_clkx for clocks of dsp itself. > > + per_clkx for clocks of peripherals used by dsp. > > Sounds like constraints that should be a schema. But this looks like > some random collection of clocks, not clocks for the DSP h/w. The DSP may use some peripheral devices, then we need to enable the clock in this driver. Using which peripherals depends on the DSP firmware, so we didn't specify the clock name here. I will add below items for this. items: - const: dsp_clk1 - const: dsp_clk2 - const: dsp_clk3 - const: dsp_clk4 - const: dsp_clk5 - const: dsp_clk6 - const: dsp_clk7 - const: dsp_clk8 - const: per_clk1 - const: per_clk2 - const: per_clk3 - const: per_clk4 - const: per_clk5 - const: per_clk6 - const: per_clk7 - const: per_clk8 - const: per_clk9 - const: per_clk10 - const: per_clk11 - const: per_clk12 - const: per_clk13 - const: per_clk14 - const: per_clk15 - const: per_clk16 - const: per_clk17 - const: per_clk18 > > > + minItems: 1 > > + maxItems: 32 > > + > > + syscon: > > Vendor prefix and name it based on what it points to or is used for. Ok, will update it. best regards Wang shengjiu
diff --git a/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml new file mode 100644 index 000000000000..7eba49aa959e --- /dev/null +++ b/Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml @@ -0,0 +1,123 @@ +# SPDX-License-Identifier: (GPL-2.0-only or BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/remoteproc/fsl,imx-dsp-rproc.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: i.MX DSP Remoteproc Devices + +maintainers: + - Shengjiu Wang <shengjiu.wang@nxp.com> + +description: + This binding provides support for DSP processors found on i.mX family of SoCs + +properties: + compatible: + enum: + - fsl,imx8qxp-hifi4 + - fsl,imx8qm-hifi4 + - fsl,imx8mp-hifi4 + - fsl,imx8ulp-hifi4 + + clocks: + description: | + Main functional clock for the remote processor + minItems: 1 + maxItems: 32 + + clock-names: + description: + List of clock names for the remote processor. + dsp_clkx for clocks of dsp itself. + per_clkx for clocks of peripherals used by dsp. + minItems: 1 + maxItems: 32 + + syscon: + $ref: /schemas/types.yaml#/definitions/phandle + description: + Phandle to syscon block which provide access for processor enablement + + mbox-names: + items: + - const: tx + - const: rx + - const: rxdb + + mboxes: + description: + This property is required only if the rpmsg/virtio functionality is used. + List of <&phandle type channel> - 1 channel for TX, 1 channel for RX, 1 channel for RXDB. + (see mailbox/fsl,mu.yaml) + minItems: 1 + maxItems: 3 + + firmware-name: + description: | + Default name of the firmware to load to the remote processor. + + memory-region: + description: + If present, a phandle for a reserved memory area that used for vdev buffer, + resource table, vring region and others used by remote processor. + minItems: 1 + maxItems: 32 + + reg: + description: | + Address space for any remoteproc memories present on the SoC. + + power-domains: + minItems: 1 + maxItems: 32 + +required: + - compatible + - reg + - mboxes + - mbox-names + - clocks + - clock-names + - firmware-name + +additionalProperties: false + +examples: + - | + #include <dt-bindings/clock/imx8mp-clock.h> + dsp_reserved: dsp@92400000 { + reg = <0x92400000 0x1000000>; + no-map; + }; + dsp_vdev0vring0: vdev0vring0@942f0000 { + reg = <0x942f0000 0x8000>; + no-map; + }; + dsp_vdev0vring1: vdev0vring1@942f8000 { + reg = <0x942f8000 0x8000>; + no-map; + }; + dsp_vdev0buffer: vdev0buffer@94300000 { + compatible = "shared-dma-pool"; + reg = <0x94300000 0x100000>; + no-map; + }; + + dsp: dsp@3b6e8000 { + compatible = "fsl,imx8mp-hifi4"; + reg = <0x3B6E8000 0x88000>; + clocks = <&audio_blk_ctrl IMX8MP_CLK_AUDIOMIX_OCRAMA_IPG>, + <&audio_blk_ctrl IMX8MP_CLK_AUDIOMIX_DSP_ROOT>, + <&audio_blk_ctrl IMX8MP_CLK_AUDIOMIX_DSPDBG_ROOT>; + clock-names = "dsp_clk1", "dsp_clk2", "dsp_clk3"; + firmware-name = "imx/dsp/hifi4.bin"; + power-domains = <&audiomix_pd>; + mbox-names = "tx", "rx", "rxdb"; + mboxes = <&mu2 0 0>, + <&mu2 1 0>, + <&mu2 3 0>; + memory-region = <&dsp_vdev0buffer>, <&dsp_vdev0vring0>, + <&dsp_vdev0vring1>, <&dsp_reserved>; + syscon = <&audio_blk_ctrl>; + };
Define the compatible string and properties needed by imx_dsp_rproc driver. Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> --- .../remoteproc/fsl,imx-dsp-rproc.yaml | 123 ++++++++++++++++++ 1 file changed, 123 insertions(+) create mode 100644 Documentation/devicetree/bindings/remoteproc/fsl,imx-dsp-rproc.yaml -- 2.17.1