On 23/01/15 10:21, Arnd Bergmann wrote:
On Friday 23 January 2015 10:07:01 Stathis Voukelatos wrote:+- interrupts : sniffer interrupt specifier +- clocks : specify the system clock for the peripheral +- clock-names : must contain the "sys" entry +- fifo-block-words : number of words in one data FIFO entry + +Example: + +sniffer@1814a000 { + compatible = "linn,eth-sniffer"; + reg = <0x1814a000 0x100>, <0x1814a400 0x400>, <0x1814a800 0x400>; + reg-names = "regs", "tx-ram", "rx-ram"; + interrupts = <GIC_SHARED 58 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "eth-sniffer-irq"; + clocks = <&system_clk>; + clock-names = "sys"; + fifo-block-words = <4>;The example contains an interrupt-names property that is not documented. If you want to name interrupts, the exact name strings need to be mandated by the binding. Alternatively just drop the name. I notice that the driver requests the first interrupt without giving a name anyway, and the description above suggests that there can only be one interrupt. Arnd
Hi Arnd, Yes, we can probably just drop the interrupt name field. It will be done in the next version of the patch set. Thank you, Stathis -- 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
