On Mon, Jan 02, 2012 at 12:53:16PM +0000, Jamie Iles wrote: > The Synopsys DesignWare block is used in some ARM devices (picoxcell) > and can be configured to provide multiple banks of GPIO pins. > > v3: - depend on rather than select IRQ_DOMAIN > - split IRQ support into a separate patch > v2: - use Rob Herring's irqdomain in generic irq chip patches > - use reg property to indicate bank index > - support irqs on both edges based on LinusW's u300 driver
Put stuff like this after the ---, it shouldn't end up in git history. > +- #interrupt-cells : Specifies the number of cells needed to encode an > +interrupt. Shall be set to 2. The first cell defines the interrupt number, > +the second encodes the triger flags encoded as: > + bits[3:0] trigger type and level flags. > + 1 = low-to-high edge triggered > + 2 = high-to-low edge triggered > + 4 = active high level-sensitive > + 8 = active low level-sensitive This looks like a straight mapping of the Linux constants to device tree. This seems sensible and reasonable and since we're forced to use magic numbers by the binding it'd be really good if we could standardise on using this for new drivers to reduce the pain for people writing and reading device tree bindings. To help with that could you factor this out into a separate document that other drivers can reference? _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
