On Friday 23 of August 2013 16:11:18 Stephen Warren wrote: > On 08/22/2013 05:22 PM, Tomasz Figa wrote: > > This patch extends vic_of_init to parse valid interrupt sources > > and resume sources masks from device tree. > > > > If mask values are not specified in device tree, all sources > > are assumed to be valid, as before this patch. > > Can you explain further why the VIC needs this information up-front? > Presumably it can accumulate it as devices request interrupts. It does not need this information just for operation, but this makes the hardware description more detailed and allows better sanity checking of interrupts being requested. To clarify, this is a mask of valid interrupt sources of the VIC, where set bit indicates that given signal is wired and clear bit that it is not. > > diff --git a/Documentation/devicetree/bindings/arm/vic.txt > > b/Documentation/devicetree/bindings/arm/vic.txt > > > > +- valid-mask : Bit mask of valid interrupt sources (defaults to all > > valid) > Until a device requests an interrupt, it can be left disabled. Once > something does request it, it gets enabled. If nothing requests it, it > never gets enabled. Doesn't the result of that logic end up being the > same thing as valid-mask represents? The only difference would be if > some device requests an invalid interrupt source, but then why not just > fix the interrupt client instead of adding this property to reject the > request? This property does not have anything to do with enabling or disabling (aka unmasking/masking) of interrupts. It just lists valid interrupt signals of given VIC. > > +- wakeup-mask : Bit mask of interrupt sources that can wake up the > > system + (defaults to all allowed) > > Shouldn't drivers for end-devices request interrupts, and then set wake > enable on those interrupts, which would then trickle up the IRQ chain to > tell the VIC which interrupts to enable wakeup on? This is the same as previous property. It does not have anything to do with requesting particular signal to wake up the system, but rather indicating that particular signal _can_ be requested to do so. Again, this is not strictly needed for correct operation, but this way you wouldn't get any sanity check over wake up signals being requested. Best regards, Tomasz -- 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
