On Mon, Mar 3, 2014 at 12:02 PM, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > Hi, > > On Thu, Feb 27, 2014 at 08:34:22PM +0100, Carlo Caione wrote: >> Added documentation for NMI irqchip. >> >> Signed-off-by: Carlo Caione <carlo@xxxxxxxxxx> >> --- >> .../allwinner,sun67i-sc-nmi.txt | 27 ++++++++++++++++++++++ >> 1 file changed, 27 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/allwinner,sun67i-sc-nmi.txt >> >> diff --git a/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun67i-sc-nmi.txt b/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun67i-sc-nmi.txt >> new file mode 100644 >> index 0000000..ac31487 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun67i-sc-nmi.txt >> @@ -0,0 +1,27 @@ >> +Allwinner Sunxi NMI Controller >> +============================== >> + >> +Required properties: >> + >> +- compatible : should be "allwinner,sun7i-a20-sc-nmi" or >> + "allwinner,sun6i-a31-sc-nmi" >> +- reg : Specifies base physical address and size of the registers. >> +- interrupt-controller : Identifies the node as an interrupt controller >> +- #interrupt-cells : Specifies the number of cells needed to encode an >> + interrupt source. The value shall be 2. The first cell is the IRQ number, the >> + second cell the trigger type. > > What values correspond to what trigger type? What are the accepted values? > >> +- interrupt-parent: Specifies the parent interrupt controller. >> +- interrupts: Specifies the list of interrupt lines which are handled by >> + the interrupt controller in the parent controller's notation. This value >> + shall be the NMI. > > Hmmm, there's only one interrupt, not a list of them. Fix for both in v5. Thank you, -- Carlo Caione -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
