On Mon, Jul 30, 2012 at 3:04 PM, DebBarma, Tarun Kanti <tarun.ka...@ti.com> wrote: > Sourav, > > On Mon, Jul 30, 2012 at 2:13 PM, Poddar, Sourav <sourav.pod...@ti.com> wrote: >> Hi All, >> >> I tried using gpio as an interrupt line for my driver >> (drivers/staging/iio/light/tsl2x7x_core.c) for omap5. >> The interrupt line number was directly passed to the driver using >> device tree. But what I observed >> is the following crash.. >> >> >> [ 1.599273] mousedev: PS/2 mouse device common for all mice >> [ 1.607513] i2c /dev entries driver >> [ 1.613739] Driver for 1-wire Dallas network protocol. >> [ 1.622650] usbcore: registered new interface driver usbhid >> [ 1.628540] usbhid: USB HID core driver >> [ 1.633728] Unhandled fault: imprecise external abort (0x1406) at >> 0x00000000 >> [ 1.641113] Internal error: : 1406 [#1] SMP ARM >> [ 1.645874] Modules linked in: >> [ 1.649078] CPU: 0 Not tainted (3.5.0-02045-g0b474d6-dirty #415) >> [ 1.655761] PC is at _set_gpio_triggering+0x44/0x264 >> [ 1.660980] LR is at gpio_irq_type+0xb8/0x160 >> [ 1.665527] pc : [<c02dbb68>] lr : [<c02dc2dc>] psr: 60000093 >> [ 1.665527] sp : dc851df0 ip : c07a8f00 fp : 00000000 >> [ 1.677581] r10: 00000081 r9 : 00000000 r8 : dc8ffc10 >> [ 1.683074] r7 : 20000093 r6 : 00000000 r5 : 00000001 r4 : fa05b000 >> [ 1.689910] r3 : dc8ffc10 r2 : 00000002 r1 : 00000002 r0 : 00000140 >> [ 1.696746] Flags: nZCv IRQs off FIQs on Mode SVC_32 ISA ARM >> Segment kernel >> [ 1.704528] Control: 10c53c7d Table: 8000406a DAC: 00000017 >> [ 1.710540] Process swapper/0 (pid: 1, stack limit = 0xdc8502f8) >> [ 1.716857] Stack: (0xdc851df0 to 0xdc852000) >> [ 1.721405] 1de0: 00000002 >> c0731280 dc8ffc6c 20000093 >> [ 1.730010] 1e00: c02dc224 c0731280 dc8d4980 00000000 c07a8e50 >> 00000141 00000002 c00a37b8 >> [ 1.738586] 1e20: 00002002 c0731280 dc8d4980 00000141 c07312d4 >> c07312b4 40000013 c00a3d7c >> [ 1.747161] 1e40: 00000141 c06728c0 00000000 c0090578 c247f000 >> c00a2d5c c0412258 00000141 >> [ 1.755737] 1e60: c0731280 c0537818 dc8d4980 c00a3fe4 c07bf6a0 >> 00002002 c071bed8 c247f000 >> [ 1.764312] 1e80: dc9df000 00000004 c07bf6a0 dc9df020 c247f3c0 >> c04ea344 c06728c0 c247f000 >> [ 1.772888] 1ea0: 00000000 09090578 c07bf6a0 dc9df004 dc9df000 >> c04ea194 00000091 c071bed8 >> [ 1.781494] 1ec0: c06f89f4 c03e13c0 c03e1314 dc9df020 c0d1afd8 >> dc9df054 c07bf60c c03272ac >> [ 1.790069] 1ee0: dc9df020 c07bf60c dc9df054 00000000 00000091 >> c03274c0 c07bf60c dc851f08 >> [ 1.798645] 1f00: c032742c c0325b10 dc8f26a8 dc9ef790 00000000 >> c07bf60c c07bcea8 c243d6c0 >> [ 1.807220] 1f20: 00000000 c0326240 c0672958 c02bce08 c07bf5e4 >> c07bf60c c03e12a0 00000000 >> [ 1.815795] 1f40: 00000091 c071bed8 c06f89f4 c0327ab0 c07bf5e4 >> 00000007 c07c8c40 00000000 >> [ 1.824401] 1f60: 00000091 c03e1738 00000000 dc850000 00000007 >> c0008648 00000000 c112c9f0 >> [ 1.832977] 1f80: c06a6d58 c06f89f4 00000001 60000013 c06694c8 >> 00000000 00000006 00000006 >> [ 1.841552] 1fa0: 60000013 c0700a20 00000007 c07c8c40 c06d020c >> 00000091 c071bed8 c0700a28 >> [ 1.850128] 1fc0: 00000000 c06d0380 00000006 00000006 c06d020c >> 00000000 00000000 c06d028c >> [ 1.858703] 1fe0: c001548c 00000013 00000000 00000000 00000000 >> c001548c 3f3f3f3f 3f3f3f3f >> [ 1.867309] [<c02dbb68>] (_set_gpio_triggering+0x44/0x264) from >> [<c02dc224>] (gpio_irq_type+0x0/0x160) >> [ 1.877075] [<c02dc224>] (gpio_irq_type+0x0/0x160) from [<00002002>] >> (0x2002) >> [ 1.884552] Code: e3120008 11816006 01c66001 e7846000 (e593c0fc) >> [ 1.890960] ------------[ cut here ]------------ >> [ 1.895812] WARNING: at arch/arm/mach-omap2/omap_l3_noc.c:113 >> l3_interrupt_handler+0x184/0x1bc() >> [ 1.905029] L3 custom error: MASTER:MPU TARGET:L4 PER2 >> [ 1.910430] Modules linked in: >> [ 1.913635] [<c001bd5c>] (unwind_backtrace+0x0/0xf4) from >> [<c00414c4>] (warn_slowpath_common+0x4c/0x64) >> [ 1.923492] [<c00414c4>] (warn_slowpath_common+0x4c/0x64) from >> [<c0041570>] (warn_slowpath_fmt+0x30/0x40) >> [ 1.933563] [<c0041570>] (warn_slowpath_fmt+0x30/0x40) from >> [<c0035de4>] (l3_interrupt_handler+0x184/0x1bc) >> [ 1.943786] [<c0035de4>] (l3_interrupt_handler+0x184/0x1bc) from >> [<c00a28b8>] (handle_irq_event_percpu+0x64/0x24c) >> [ 1.954650] [<c00a28b8>] (handle_irq_event_percpu+0x64/0x24c) from >> [<c00a2adc>] (handle_irq_event+0x3c/0x5c) >> [ 1.964965] [<c00a2adc>] (handle_irq_event+0x3c/0x5c) from >> [<c00a56a8>] (handle_fasteoi_irq+0x98/0x13c) >> [ 1.974822] [<c00a56a8>] (handle_fasteoi_irq+0x98/0x13c) from >> [<c00a235c>] (generic_handle_irq+0x34/0x44) >> [ 1.984863] [<c00a235c>] (generic_handle_irq+0x34/0x44) from >> [<c00153a0>] (handle_IRQ+0x4c/0xac) >> [ 1.994110] [<c00153a0>] (handle_IRQ+0x4c/0xac) from [<c0008480>] >> (gic_handle_irq+0x2c/0x60) >> [ 2.002960] [<c0008480>] (gic_handle_irq+0x2c/0x60) from >> [<c04f26e4>] (__irq_svc+0x44/0x60) >> [ 2.011718] Exception stack(0xdc851c48 to 0xdc851c90) >> [ 2.017028] 1c40: c07a87a0 00000000 dc851c80 >> 00000000 c0743f80 dc851da8 >> [ 2.025604] 1c60: 00000001 00000001 dc851cd7 00000000 c02dbb6a >> c02dbb6c ffffffff dc851c90 >> [ 2.034179] 1c80: c04f2440 c04f2444 60000113 ffffffff >> [ 2.039489] [<c04f26e4>] (__irq_svc+0x44/0x60) from [<c04f2444>] >> (_raw_spin_unlock_irq+0x28/0x2c) >> [ 2.048797] [<c04f2444>] (_raw_spin_unlock_irq+0x28/0x2c) from >> [<c001871c>] (die+0xe0/0x324) >> [ 2.057647] [<c001871c>] (die+0xe0/0x324) from [<c0008444>] >> (do_DataAbort+0x88/0x98) >> [ 2.065765] [<c0008444>] (do_DataAbort+0x88/0x98) from [<c04f2664>] >> (__dabt_svc+0x44/0x80) >> [ 2.074462] Exception stack(0xdc851da8 to 0xdc851df0) >> [ 2.079742] 1da0: 00000140 00000002 00000002 >> dc8ffc10 fa05b000 00000001 >> [ 2.088317] 1dc0: 00000000 20000093 dc8ffc10 00000000 00000081 >> 00000000 c07a8f00 dc851df0 >> [ 2.096893] 1de0: c02dc2dc c02dbb68 60000093 ffffffff >> [ 2.102203] [<c04f2664>] (__dabt_svc+0x44/0x80) from [<c02dbb68>] >> (_set_gpio_triggering+0x44/0x264) >> [ 2.111724] [<c02dbb68>] (_set_gpio_triggering+0x44/0x264) from >> [<c02dc224>] (gpio_irq_type+0x0/0x160) >> [ 2.121490] [<c02dc224>] (gpio_irq_type+0x0/0x160) from [<00002002>] >> (0x2002) >> [ 2.128967] ---[ end trace 8b4b88305e6b2aff ]--- >> [ 2.133911] ---[ end trace 8b4b88305e6b2b00 ]--- >> [ 2.138763] Kernel panic - not syncing: Attempted to kill init! >> exitcode=0x0000000b >> [ 2.138763] >> [ 2.148376] CPU1: stopping >> [ 2.151214] [<c001bd5c>] (unwind_backtrace+0x0/0xf4) from >> [<c0019c74>] (handle_IPI+0x130/0x15c) >> [ 2.160339] [<c0019c74>] (handle_IPI+0x130/0x15c) from [<c00084ac>] >> (gic_handle_irq+0x58/0x60) >> [ 2.169372] [<c00084ac>] (gic_handle_irq+0x58/0x60) from >> [<c04f26e4>] (__irq_svc+0x44/0x60) >> [ 2.178131] Exception stack(0xdc869e10 to 0xdc869e58) >> [ 2.183410] 9e00: 00000000 >> 00000000 00000002 60000113 >> [ 2.191986] 9e20: 00000000 dc868000 c07970e4 00000000 00000000 >> 00000001 00000002 00000000 >> [ 2.200561] 9e40: 00000001 dc869e58 c00936f4 c0093704 60000113 ffffffff >> [ 2.207489] [<c04f26e4>] (__irq_svc+0x44/0x60) from [<c0093704>] >> (lock_acquire+0xac/0x104) >> [ 2.216156] [<c0093704>] (lock_acquire+0xac/0x104) from >> [<c007a8bc>] (rebalance_domains+0x5c/0x1a8) >> [ 2.225646] [<c007a8bc>] (rebalance_domains+0x5c/0x1a8) from >> [<c007ae84>] (run_rebalance_domains+0x44/0x14c) >> [ 2.235961] [<c007ae84>] (run_rebalance_domains+0x44/0x14c) from >> [<c004a864>] (__do_softirq+0xb4/0x214) >> [ 2.245819] [<c004a864>] (__do_softirq+0xb4/0x214) from >> [<c004aba0>] (irq_exit+0x90/0x98) >> [ 2.254394] [<c004aba0>] (irq_exit+0x90/0x98) from [<c0019c48>] >> (handle_IPI+0x104/0x15c) >> [ 2.262878] [<c0019c48>] (handle_IPI+0x104/0x15c) from [<c00084ac>] >> (gic_handle_irq+0x58/0x60) >> [ 2.271911] [<c00084ac>] (gic_handle_irq+0x58/0x60) from >> [<c04f26e4>] (__irq_svc+0x44/0x60) >> [ 2.280670] Exception stack(0xdc869f88 to 0xdc869fd0) >> [ 2.285949] 9f80: c07a87a0 c0022a40 00000000 >> 00000000 dc868000 c07c8d08 >> [ 2.294525] 9fa0: c04fe078 c0743d70 00000000 410fc0f0 c0743fa0 >> 00000000 00000000 dc869fd0 >> >> >> The device tree data for acquiring the above GPIO interrupt line looks >> like this. >> >> +++ linux-omap-storage/arch/arm/boot/dts/omap5-evm.dts 2012-07-30 >> 14:11:08.931694001 +0530 >> @@ -42,7 +42,8 @@ >> tsl2771@39 { >> compatible = "taos,tsl2771"; >> reg = <0x39>; >> + interrupt-parent = <&gpio5>; >> + interrupts = <21>; /* gpio line 149 */ >> }; >> }; >> >> Note: using "gpio_request_one" in the driver solves the issue. >> Is using this api in the driver required? >> Any pointer on the above crash? > Hi Tarun, > Any user/client driver of GPIO is supposed to go through > gpio_request() API so that module clock > is enabled correctly. Overriding of APIs would put the power > management state machine in jeopardy. > -- I tried putting "pm_runtime_get_sync" in gpio_irq_type api where the kernel is crashing and the crash is no longer observed. So indeed, its about enabling clocks.
One doubt: Can't we put runtime apis in "gpio_irq_type" and eliminate the use of "gpio_request_one"?? > Tarun > >> >> ~Sourav -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html