[PATCH 04/10] dt-bindings: interrupt-controller: Add i.MX8qxp Display Controller interrupt controller

Krzysztof Kozlowski krzk at kernel.org
Sun Jul 7 07:01:27 PDT 2024


On 05/07/2024 11:09, Liu Ying wrote:
> i.MX8qxp Display Controller has a built-in interrupt controller to support
> Enable/Status/Preset/Clear interrupt bit.
> 
> Signed-off-by: Liu Ying <victor.liu at nxp.com>
> ---
>  .../fsl,imx8qxp-dc-intc.yaml                  | 321 ++++++++++++++++++
>  1 file changed, 321 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/interrupt-controller/fsl,imx8qxp-dc-intc.yaml
> 
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/fsl,imx8qxp-dc-intc.yaml b/Documentation/devicetree/bindings/interrupt-controller/fsl,imx8qxp-dc-intc.yaml
> new file mode 100644
> index 000000000000..3d0d11def505
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/interrupt-controller/fsl,imx8qxp-dc-intc.yaml
> @@ -0,0 +1,321 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/interrupt-controller/fsl,imx8qxp-dc-intc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Freescale i.MX8qxp Display Controller interrupt controller
> +
> +description: |
> +  The Display Controller has a built-in interrupt controller with the following
> +  features for all relevant HW events:
> +
> +  * Enable bit (mask)
> +  * Status bit (set by an HW event)
> +  * Preset bit (can be used by SW to set status)
> +  * Clear bit (used by SW to reset the status)

1. Why this is split from the main node?

2. Who can use this interrupt controller? Children of your display unit?
Then it is not really a separate device, is it?

> +properties:
> +  compatible:
> +    const: fsl,imx8qxp-dc-intc
> +
> +  reg:
> +    maxItems: 1
> +
> +  clocks:
> +    maxItems: 1
> +
> +  interrupt-controller: true
> +
> +  "#interrupt-cells":
> +    const: 1
> +
> +  interrupts:
> +    items:
> +      - description: |

Do not need '|' unless you need to preserve formatting.



Best regards,
Krzysztof




More information about the linux-arm-kernel mailing list