Re: [PATCH] net: Linn Ethernet Packet Sniffer driver

[Stathis Voukelatos]

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 linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/