| Message ID | 20231206112337.2234849-1-festevam@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [v2] dt-bindings: lcdif: Properly describe the i.MX23 interrupts | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 32 lines checked |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Wed, Dec 06, 2023 at 08:23:37AM -0300, Fabio Estevam wrote: > From: Fabio Estevam <festevam@denx.de> > > i.MX23 has two LCDIF interrupts instead of a single one like other > i.MX devices. > > Take this into account for properly describing the i.MX23 LCDIF > interrupts. > > This fixes the following dt-schema warning: > > imx23-olinuxino.dtb: lcdif@80030000: interrupts: [[46], [45]] is too long > from schema $id: http://devicetree.org/schemas/display/fsl,lcdif.yaml# > > Signed-off-by: Fabio Estevam <festevam@denx.de> > Reviewed-by: Marek Vasut <marex@denx.de> Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor.
On Wed, 06 Dec 2023 08:23:37 -0300, Fabio Estevam wrote: > From: Fabio Estevam <festevam@denx.de> > > i.MX23 has two LCDIF interrupts instead of a single one like other > i.MX devices. > > Take this into account for properly describing the i.MX23 LCDIF > interrupts. > > This fixes the following dt-schema warning: > > imx23-olinuxino.dtb: lcdif@80030000: interrupts: [[46], [45]] is too long > from schema $id: http://devicetree.org/schemas/display/fsl,lcdif.yaml# > > Signed-off-by: Fabio Estevam <festevam@denx.de> > Reviewed-by: Marek Vasut <marex@denx.de> > --- > Changes since v1: > - Add missing else to limit the number of irqs for the other variants. (Krzysztof) > > .../bindings/display/fsl,lcdif.yaml | 20 ++++++++++++++++++- > 1 file changed, 19 insertions(+), 1 deletion(-) > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml index fc11ab5fc465..1c2be8d6f633 100644 --- a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml +++ b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml @@ -51,7 +51,10 @@ properties: minItems: 1 interrupts: - maxItems: 1 + items: + - description: LCDIF DMA interrupt + - description: LCDIF Error interrupt + minItems: 1 power-domains: maxItems: 1 @@ -131,6 +134,21 @@ allOf: then: required: - power-domains + - if: + properties: + compatible: + contains: + enum: + - fsl,imx23-lcdif + then: + properties: + interrupts: + minItems: 2 + maxItems: 2 + else: + properties: + interrupts: + maxItems: 1 examples: - |