* [PATCH] dt-bindings: lcdif: Properly describe the i.MX23 interrupts
@ 2023-12-05 17:49 Fabio Estevam
2023-12-06 11:03 ` Krzysztof Kozlowski
0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2023-12-05 17:49 UTC (permalink / raw)
To: marex
Cc: devicetree, conor+dt, Fabio Estevam, dri-devel, robh+dt,
krzysztof.kozlowski+dt
From: Fabio Estevam <festevam@denx.de>
i.MX23 has two i.MX23 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>
---
.../devicetree/bindings/display/fsl,lcdif.yaml | 16 +++++++++++++++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml
index fc11ab5fc465..c4228b893766 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,17 @@ allOf:
then:
required:
- power-domains
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - fsl,imx23-lcdif
+ then:
+ properties:
+ interrupts:
+ minItems: 2
+ maxItems: 2
examples:
- |
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] dt-bindings: lcdif: Properly describe the i.MX23 interrupts
2023-12-05 17:49 [PATCH] dt-bindings: lcdif: Properly describe the i.MX23 interrupts Fabio Estevam
@ 2023-12-06 11:03 ` Krzysztof Kozlowski
0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-06 11:03 UTC (permalink / raw)
To: Fabio Estevam, marex
Cc: devicetree, conor+dt, Fabio Estevam, dri-devel, robh+dt,
krzysztof.kozlowski+dt
On 05/12/2023 18:49, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
>
> i.MX23 has two i.MX23 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>
> ---
> .../devicetree/bindings/display/fsl,lcdif.yaml | 16 +++++++++++++++-
> 1 file changed, 15 insertions(+), 1 deletion(-)
Same problems as other patches. You should narrow to 1 existing variants.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-12-06 11:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-05 17:49 [PATCH] dt-bindings: lcdif: Properly describe the i.MX23 interrupts Fabio Estevam
2023-12-06 11:03 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).