On Sat, Oct 26, 2013 at 01:37:57AM -0500, Kumar Gala wrote: > > On Oct 25, 2013, at 5:18 PM, Sebastian Reichel wrote: > > > On Fri, Oct 25, 2013 at 04:41:20PM -0500, Kumar Gala wrote: > >> On Oct 24, 2013, at 9:48 AM, Sebastian Reichel wrote: > >>> +- interrupt: should be one of the following > >>> + - <8>: For controllers compatible with twl4030 > >> > >> Just checking, but the interrupt is always 8 for this device? > > > > Yes. It's currently hardcoded in drivers/mfd/twl-core.c. > > The fact that is hard coded in the driver does not imply that it > should be in the device tree binding. Is there an interrupt > controller as part of the TWL4030?
Hardware looks like this:
&twl4030 {
compatible = "ti,twl4030";
interrupt-controller;
#interrupt-cells = <1>;
twl_pwrbutton: pwrbutton {
compatible = "ti,twl4030-pwrbutton";
interrupts = <8>; /* 8th interrupt from the twl4030 */
};
};
Simplified the initialization of twl4030 stuff works
like this for non DT boot:
twl4030_init(...) {
init_subdev(...);
init_subdev("twl4030-pwrbutton", ..., irq=8, ...);
init_subdev(...);
};
-- Sebastian
signature.asc
Description: Digital signature
