On 01/03/2017 02:24 PM, Rob Herring wrote: > Correcting Grant's email. > > On Tue, Jan 3, 2017 at 1:24 PM, Florian Fainelli <f.fainelli@xxxxxxxxx> wrote: >> The interrupts-extended property is a common property used when >> interrupt generating devices having interrupt lines in several interrupt >> controllers with possibly distinct interrupt specifiers. >> >> Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx> > > I assume this is mostly copied from the kernel documentation. This > needs Grant's ack as the kernel documentation license is ambiguous > (default GPL) and the spec is Apache. Sort of, I did look at the changes submitted to the documentation, but this was kind of rewritten from scratch, nonetheless, having some clarification on the documentation would indeed be a good thing. > >> --- >> source/devicetree-basics.rst | 34 ++++++++++++++++++++++++++++++++++ >> 1 file changed, 34 insertions(+) >> >> diff --git a/source/devicetree-basics.rst b/source/devicetree-basics.rst >> index be80af3d825b..1bda0fa54e7a 100644 >> --- a/source/devicetree-basics.rst >> +++ b/source/devicetree-basics.rst >> @@ -955,6 +955,40 @@ Description: >> to the interrupt parent. If this property is missing from a device, its >> interrupt parent is assumed to be its devicetree parent. >> >> +interrupts-extended >> +^^^^^^^^^^^^^^^^^^^ >> + >> +Property: ``interrupts-extended`` >> + >> +Value type: ``<phandle> <prop-encoded-array>`` >> + >> +Description: >> + >> + The *interrupts-extended* property of a device node defines the interrupt or >> + the interrupts that are generated by the device. In case where an interrupt > > In the case... > >> + generating devices has several interrupt lines, some of them having distinct > > s/devices/device/ Whoops, thanks, anything else that needs fixing in the wording used below? > >> + interrupt parents, the *interrupts-extended* property should be used to >> + fully describe the interrupts of this device, relative to the interrupt >> + controller(s), which is encoded by the phandle part of the property. The non >> + phandle part of the property encodes the interrupt specifiers and obeys to >> + the *interrupts* property description. >> + >> + If both *interrupts-extended* and *interrupts* properties are present, the >> + *interrupts-extended* properties takes precedence, and the *interrupts* >> + property may be provided to a client program for compatibility purposes. >> + >> +Example: >> + >> + A device with several interrupt lines in distinct interrupt controllers, >> + having different interrupt specifiers is illustrated below. In this example >> + ``pic`` is an interrupt controller with an *#interrupt-cells* specifier >> + of 2, while ``gic`` is an interrupt controller with an *#interrupts-cells* >> + specifier of 1. >> + >> + >> + ``interrupts-extended = <&pic 0xA 8>, <&gic 0xda>;`` >> + >> + >> Properties for Interrupt Controllers >> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >> >> -- >> 2.9.3 >> -- Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree-spec" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html