dt-binding: irq: imx-irqsteer: Use irq number instead of group number
authorAisheng Dong <aisheng.dong@nxp.com>
Wed, 20 Feb 2019 11:40:40 +0000 (11:40 +0000)
committerMarc Zyngier <marc.zyngier@arm.com>
Fri, 22 Feb 2019 09:22:22 +0000 (09:22 +0000)
Not all 64 interrupts may be used in one group. e.g. most irqsteer in
imx8qxp and imx8qm subsystems supports only 32 interrupts.

As the IP integration parameters are Channel number and interrupts number,
let's use fsl,irqs-num to represents how many interrupts supported
by this irqsteer channel.

Note this will break the compatibility of old binding. As the original
fsl,irq-groups was born out of a misunderstanding of the HW config
options and we are not aware of any users of the current binding.
And the old binding was just published in recent months, so it's
worth to change now to avoid confusing in the future.

Cc: Rob Herring <robh+dt@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: devicetree@vger.kernel.org
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.txt

index 45790ce6f5b9be2b2910fe44fa16377b8c316580..6d0a41b549430cf8169ddfbe677a8d6ff68b0648 100644 (file)
@@ -16,8 +16,8 @@ Required properties:
 - #interrupt-cells: Specifies the number of cells needed to encode an
   interrupt source. The value must be 1.
 - fsl,channel: The output channel that all input IRQs should be steered into.
-- fsl,irq-groups: Number of IRQ groups managed by this controller instance.
-  Each group manages 64 input interrupts.
+- fsl,num-irqs: Number of input interrupts of this channel.
+  Should be multiple of 32 input interrupts and up to 512 interrupts.
 
 Example:
 
@@ -28,7 +28,7 @@ Example:
                clocks = <&clk IMX8MQ_CLK_DISP_APB_ROOT>;
                clock-names = "ipg";
                fsl,channel = <0>;
-               fsl,irq-groups = <1>;
+               fsl,num-irqs = <64>;
                interrupt-controller;
                #interrupt-cells = <1>;
        };