Message ID | 20220523113029.842753-3-peng.fan@oss.nxp.com |
---|---|
State | Changes Requested, archived |
Headers | show |
Series | imx: support i.MX93 SRC and mediamix blk ctrl | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | success | |
robh/patch-applied | success | |
robh/dtbs-check | warning | build log |
robh/dt-meta-schema | success |
On 23/05/2022 13:30, Peng Fan (OSS) wrote: > From: Peng Fan <peng.fan@nxp.com> > > Add DT bindings for i.MX93 MEDIAMIX BLK CTRL. In the subject - you duplicate "bindings", so just "dt-bindings: soc: add i.MX93 mediamix blk ctrl" > > Signed-off-by: Peng Fan <peng.fan@nxp.com> > --- > .../soc/imx/fsl,imx93-media-blk-ctrl.yaml | 81 +++++++++++++++++++ > include/dt-bindings/power/imx93-power.h | 6 ++ > 2 files changed, 87 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml > > diff --git a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml > new file mode 100644 > index 000000000000..775c08d763b8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml > @@ -0,0 +1,81 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/soc/imx/fsl,imx93-media-blk-ctrl.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: NXP i.MX93 Media blk-ctrl > + > +maintainers: > + - Peng Fan <peng.fan@nxp.com> > + > +description: > + The i.MX93 MEDIAMIX domain contains control and status registers known > + as MEDIAMIX Block Control (MEDIAMIX BLK_CTRL). These registers include > + clocking, reset, and miscellaneous top-level controls for peripherals > + within the MEDIAMIX domain > + > +properties: > + compatible: > + items: > + - const: fsl,imx93-media-blk-ctrl > + - const: syscon > + > + reg: > + maxItems: 1 > + > + '#power-domain-cells': > + const: 1 Missing ref to power domains. > + > + power-domains: > + maxItems: 1 > + > + clocks: > + minItems: 10 minItems is not needed in such case. > + maxItems: 10 > + > + clock-names: > + items: > + - const: apb > + - const: axi > + - const: nic > + - const: disp > + - const: cam > + - const: pxp > + - const: lcdif > + - const: isi > + - const: csi > + - const: dsi > + > +required: > + - compatible > + - reg > + - power-domains > + - clocks > + - clock-names > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/imx93-clock.h> > + #include <dt-bindings/power/imx93-power.h> > + > + media_blk_ctrl: blk_ctrl@4ac10000 { No underscores in node names. Generic node name, so "system-controller" or "power-controller"? Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml new file mode 100644 index 000000000000..775c08d763b8 --- /dev/null +++ b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml @@ -0,0 +1,81 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/soc/imx/fsl,imx93-media-blk-ctrl.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: NXP i.MX93 Media blk-ctrl + +maintainers: + - Peng Fan <peng.fan@nxp.com> + +description: + The i.MX93 MEDIAMIX domain contains control and status registers known + as MEDIAMIX Block Control (MEDIAMIX BLK_CTRL). These registers include + clocking, reset, and miscellaneous top-level controls for peripherals + within the MEDIAMIX domain + +properties: + compatible: + items: + - const: fsl,imx93-media-blk-ctrl + - const: syscon + + reg: + maxItems: 1 + + '#power-domain-cells': + const: 1 + + power-domains: + maxItems: 1 + + clocks: + minItems: 10 + maxItems: 10 + + clock-names: + items: + - const: apb + - const: axi + - const: nic + - const: disp + - const: cam + - const: pxp + - const: lcdif + - const: isi + - const: csi + - const: dsi + +required: + - compatible + - reg + - power-domains + - clocks + - clock-names + +additionalProperties: false + +examples: + - | + #include <dt-bindings/clock/imx93-clock.h> + #include <dt-bindings/power/imx93-power.h> + + media_blk_ctrl: blk_ctrl@4ac10000 { + compatible = "fsl,imx93-media-blk-ctrl", "syscon"; + reg = <0x4ac10000 0x10000>; + power-domains = <&mediamix>; + clocks = <&clk IMX93_CLK_MEDIA_APB>, + <&clk IMX93_CLK_MEDIA_AXI>, + <&clk IMX93_CLK_NIC_MEDIA_GATE>, + <&clk IMX93_CLK_MEDIA_DISP_PIX>, + <&clk IMX93_CLK_CAM_PIX>, + <&clk IMX93_CLK_PXP_GATE>, + <&clk IMX93_CLK_LCDIF_GATE>, + <&clk IMX93_CLK_ISI_GATE>, + <&clk IMX93_CLK_MIPI_CSI_GATE>, + <&clk IMX93_CLK_MIPI_DSI_GATE>; + clock-names = "apb", "axi", "nic", "disp", "cam", + "pxp", "lcdif", "isi", "csi", "dsi"; + #power-domain-cells = <1>; + }; diff --git a/include/dt-bindings/power/imx93-power.h b/include/dt-bindings/power/imx93-power.h index 27fb7df80f93..b3d2d9619d3a 100644 --- a/include/dt-bindings/power/imx93-power.h +++ b/include/dt-bindings/power/imx93-power.h @@ -8,4 +8,10 @@ #define IMX93_POWER_DOMAIN_MEDIAMIX 0 +#define IMX93_MEDIABLK_PD_MIPI_DSI 0 +#define IMX93_MEDIABLK_PD_MIPI_CSI 1 +#define IMX93_MEDIABLK_PD_PXP 2 +#define IMX93_MEDIABLK_PD_LCDIF 3 +#define IMX93_MEDIABLK_PD_ISI 4 + #endif