diff mbox series

dt-bindings: gpio: altera: correct interrupt-cells

Message ID 20220512142910.328995-1-dinguyen@kernel.org
State Accepted
Commit 3a21c3ac93aff7b4522b152399df8f6a041df56d
Headers show
Series dt-bindings: gpio: altera: correct interrupt-cells | expand

Commit Message

Dinh Nguyen May 12, 2022, 2:29 p.m. UTC
update documentation to correctly state the interrupt-cells to be 2.

Cc: stable@vger.kernel.org
Fixes: 4fd9bbc6e071 ("drivers/gpio: Altera soft IP GPIO driver devicetree binding")
Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
---
 Documentation/devicetree/bindings/gpio/gpio-altera.txt | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Linus Walleij May 13, 2022, 8:51 p.m. UTC | #1
On Thu, May 12, 2022 at 4:29 PM Dinh Nguyen <dinguyen@kernel.org> wrote:

> update documentation to correctly state the interrupt-cells to be 2.
>
> Cc: stable@vger.kernel.org
> Fixes: 4fd9bbc6e071 ("drivers/gpio: Altera soft IP GPIO driver devicetree binding")
> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
(...)
> -- #interrupt-cells : Should be 1. The interrupt type is fixed in the hardware.
> +- #interrupt-cells : Should be 2. The interrupt type is fixed in the hardware.
>    - The first cell is the GPIO offset number within the GPIO controller.
> +  - The second cell is the interrupt trigger type and level flags.

So now this says (A) that the interrupt type is fixed in hardware,
and (B) that you should specify it.

This is confusing, I think something is wrong?

Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/gpio/gpio-altera.txt b/Documentation/devicetree/bindings/gpio/gpio-altera.txt
index 146e554b3c67..2a80e272cd66 100644
--- a/Documentation/devicetree/bindings/gpio/gpio-altera.txt
+++ b/Documentation/devicetree/bindings/gpio/gpio-altera.txt
@@ -9,8 +9,9 @@  Required properties:
   - The second cell is reserved and is currently unused.
 - gpio-controller : Marks the device node as a GPIO controller.
 - interrupt-controller: Mark the device node as an interrupt controller
-- #interrupt-cells : Should be 1. The interrupt type is fixed in the hardware.
+- #interrupt-cells : Should be 2. The interrupt type is fixed in the hardware.
   - The first cell is the GPIO offset number within the GPIO controller.
+  - The second cell is the interrupt trigger type and level flags.
 - interrupts: Specify the interrupt.
 - altr,interrupt-type: Specifies the interrupt trigger type the GPIO
   hardware is synthesized. This field is required if the Altera GPIO controller
@@ -38,6 +39,6 @@  gpio_altr: gpio@ff200000 {
 	altr,interrupt-type = <IRQ_TYPE_EDGE_RISING>;
 	#gpio-cells = <2>;
 	gpio-controller;
-	#interrupt-cells = <1>;
+	#interrupt-cells = <2>;
 	interrupt-controller;
 };