| Message ID | 20221213132006.6446-1-marex@denx.de |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [v5] dt-bindings: mxsfb: Document i.MX8M/i.MX6SX/i.MX6SL power-domains property | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | fail | build log |
On 13/12/2022 14:20, Marek Vasut wrote: > The power-domains property is mandatory on i.MX8M Mini, Nano, Plus > and i.MX6SX, i.MX6SL. Document the property and mark it as required > on the aforementioned variants of the IP, present in those SoCs. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > Cc: Liu Ying <victor.liu@nxp.com> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: linux-arm-kernel@lists.infradead.org > To: devicetree@vger.kernel.org > --- > V2: - Add AB from Krzysztof > - Add mx6sx power domain into the list > V3: - Update commit message > - Add i.MX6SL > - Update example > V4: - Drop ack from Krzysztof > - Rebase on next 20221213 instead of previous 20221208, which now > contains power-domains: part of this patch > V5: - Drop power-domains from MX8MP/MX6SX entry Thanks. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Tue, 2022-12-13 at 14:20 +0100, Marek Vasut wrote: > The power-domains property is mandatory on i.MX8M Mini, Nano, Plus > and i.MX6SX, i.MX6SL. Document the property and mark it as required > on the aforementioned variants of the IP, present in those SoCs. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > Cc: Liu Ying <victor.liu@nxp.com> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: linux-arm-kernel@lists.infradead.org > To: devicetree@vger.kernel.org > --- > V2: - Add AB from Krzysztof > - Add mx6sx power domain into the list > V3: - Update commit message > - Add i.MX6SL > - Update example > V4: - Drop ack from Krzysztof > - Rebase on next 20221213 instead of previous 20221208, which now > contains power-domains: part of this patch > V5: - Drop power-domains from MX8MP/MX6SX entry > --- > .../devicetree/bindings/display/fsl,lcdif.yaml | 15 > ++++++++++++++- > 1 file changed, 14 insertions(+), 1 deletion(-) Reviewed-by: Liu Ying <victor.liu@nxp.com>
On Tue, 13 Dec 2022 14:20:06 +0100, Marek Vasut wrote: > The power-domains property is mandatory on i.MX8M Mini, Nano, Plus > and i.MX6SX, i.MX6SL. Document the property and mark it as required > on the aforementioned variants of the IP, present in those SoCs. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > Cc: Liu Ying <victor.liu@nxp.com> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: linux-arm-kernel@lists.infradead.org > To: devicetree@vger.kernel.org > --- > V2: - Add AB from Krzysztof > - Add mx6sx power domain into the list > V3: - Update commit message > - Add i.MX6SL > - Update example > V4: - Drop ack from Krzysztof > - Rebase on next 20221213 instead of previous 20221208, which now > contains power-domains: part of this patch > V5: - Drop power-domains from MX8MP/MX6SX entry > --- > .../devicetree/bindings/display/fsl,lcdif.yaml | 15 ++++++++++++++- > 1 file changed, 14 insertions(+), 1 deletion(-) > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml index f449cfc767899..75b4efd70ba85 100644 --- a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml +++ b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml @@ -99,7 +99,6 @@ allOf: maxItems: 3 required: - clock-names - - power-domains - if: not: properties: @@ -114,6 +113,19 @@ allOf: maxItems: 1 clock-names: maxItems: 1 + - if: + properties: + compatible: + contains: + enum: + - fsl,imx6sl-lcdif + - fsl,imx6sx-lcdif + - fsl,imx8mm-lcdif + - fsl,imx8mn-lcdif + - fsl,imx8mp-lcdif + then: + required: + - power-domains examples: - | @@ -128,6 +140,7 @@ examples: <&clks IMX6SX_CLK_LCDIF_APB>, <&clks IMX6SX_CLK_DISPLAY_AXI>; clock-names = "pix", "axi", "disp_axi"; + power-domains = <&pd_disp>; port { endpoint {
The power-domains property is mandatory on i.MX8M Mini, Nano, Plus and i.MX6SX, i.MX6SL. Document the property and mark it as required on the aforementioned variants of the IP, present in those SoCs. Signed-off-by: Marek Vasut <marex@denx.de> --- Cc: Fabio Estevam <festevam@gmail.com> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> Cc: Liu Ying <victor.liu@nxp.com> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: Rob Herring <robh+dt@kernel.org> Cc: Shawn Guo <shawnguo@kernel.org> Cc: linux-arm-kernel@lists.infradead.org To: devicetree@vger.kernel.org --- V2: - Add AB from Krzysztof - Add mx6sx power domain into the list V3: - Update commit message - Add i.MX6SL - Update example V4: - Drop ack from Krzysztof - Rebase on next 20221213 instead of previous 20221208, which now contains power-domains: part of this patch V5: - Drop power-domains from MX8MP/MX6SX entry --- .../devicetree/bindings/display/fsl,lcdif.yaml | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)