Re: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform
Hi Sergio, On Friday 22 January 2010 19:36:06 Aguirre, Sergio wrote: [snip] Ok, I was able to work around the kernel panic with the attached patch. I have the feeling that all your development is dependant on loading all camera/sensors as modules in the filesystem. Have you done validation with built-in option in kernel's menuconfig? You're right. I haven't done any validation, and I've been to reproduce the crash when compiling everything directly into the kernel image. I'm on holidays this week so I won't be able to look into the problem before February the 1st. If nobody beats me to it, I'll see how we can fix it properly then. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform
Hi Sergio, On Friday 22 January 2010 10:49:17 Aguirre, Sergio wrote: Laurent, Sakari, While I was trying to adapt my Zoom2/3 sensor drivers into latest 'devel' branch with latest commit: commit 2e7d09ec5e09ee80462a611c9958e99866ee337c Author: Laurent Pinchart laurent.pinch...@ideasonboard.com Date: Wed Jan 20 13:49:31 2010 +0100 omap3isp: Work around sg_alloc_table BUG_ON Work in progress Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com That last patch shouldn't have been applied to the linux-omap tree. The patch itself is correct, but the commit message isn't. I'll check that with Sakari. And applying the attached patches. I see the following kernel panic on bootup: omap3isp omap3isp: Revision 2.0 found omap-iommu omap-iommu.0: isp: version 1.1 Unable to handle kernel NULL pointer dereference at virtual address 0048 pgd = c0004000 [0048] *pgd= Internal error: Oops: 5 [#1] last sysfs file: Modules linked in: CPU: 0Not tainted (2.6.32-07583-gd4ae425-dirty #7) PC is at get_device_parent+0x68/0x114 LR is at device_add+0x7c/0x474 pc : [c019345c]lr : [c0194264]psr: 6013 sp : cf823dc8 ip : cf95f680 fp : cf91b178 r10: c03f37e0 r9 : c0367b06 r8 : c03dc770 r7 : ffea r6 : cf91b170 r5 : c03b7748 r4 : r3 : c03dc770 r2 : r1 : c03b7740 r0 : c03b7748 Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment kernel Control: 10c5387d Table: 80004019 DAC: 0017 Process swapper (pid: 1, stack limit = 0xcf8222e8) Stack: (0xcf823dc8 to 0xcf824000) 3dc0: cf91b170 cf91b170 c0194264 cf823e10 c03b7740 3de0: cf91b178 c01670a4 cf91b178 cf91b168 cf91b170 3e00: c03f33dc c03dc770 c0367b06 c03f37e0 c03b787c c0200744 c03b7740 cf91b160 3e20: c03b7740 cf91b000 c03d3cc0 c01fdcbc c03b8ae0 c03b8fc0 3e40: c03b7738 98f7cec1 cf820a30 c03b8f68 c0067f78 cf82c01c cf821a40 3e60: 0017 cf820a00 cf821bdc cf823eac 0001 fffd 3e80: cf823ebc cf803740 0001 cf803744 c03ed394 3ea0: cf823ef4 c016630c cf80d7a8 c016630c cf82dfa4 cf80d7a8 3ec0: c00e5cdc cf80e608 c02a8c58 cf823f10 cf95e7b8 c00e6474 3ee0: cf95e7b8 c00e60ac cf95e818 cf95e7b8 cf823f10 c00e6184 cf823f10 3f00: cf95e7b8 cf80e608 0001 c00e6eec cf80e608 c03b7740 3f20: c03b7740 c03dc440 cf95dbc0 c03d3cc0 c01971e4 3f40: c03b7740 c01963e0 c03b7740 c03b7774 c03dc440 cf95dbc0 c03d3cc0 c01964ec 3f60: c019648c c03dc440 c0195cb4 cf803af8 cf8459f0 c00222c8 c03dc440 3f80: c03dc440 c0195614 c03361b1 c03361b1 0006 c00222c8 c03dc440 3fa0: c01967bc c00222c8 c001c060 3fc0: c0027334 0031 0192 c00222c8 3fe0: c0008578 c0028dbc 00dbda20 24ffdc02 [c019345c] (get_device_parent+0x68/0x114) from [c0194264] (device_add+0x7c/0x474) [c0194264] (device_add+0x7c/0x474) from [c0200744] (media_devnode_register+0x1d0/0x29c) [c0200744] (media_devnode_register+0x1d0/0x29c) from [c01fdcbc] (omap34xxcam_probe+0x64/0x428) [c01fdcbc] (omap34xxcam_probe+0x64/0x428) from [c01971e4] (platform_drv_probe+0x18/0x1c) [c01971e4] (platform_drv_probe+0x18/0x1c) from [c01963e0] (driver_probe_device+0xa0/0x14c) [c01963e0] (driver_probe_device+0xa0/0x14c) from [c01964ec] (__driver_attach+0x60/0x84) [c01964ec] (__driver_attach+0x60/0x84) from [c0195cb4] (bus_for_each_dev+0x44/0x74) [c0195cb4] (bus_for_each_dev+0x44/0x74) from [c0195614] (bus_add_driver+0x9c/0x220) [c0195614] (bus_add_driver+0x9c/0x220) from [c01967bc] (driver_register+0xa8/0x130) [c01967bc] (driver_register+0xa8/0x130) from [c0027334] (do_one_initcall+0x5c/0x1b4) [c0027334] (do_one_initcall+0x5c/0x1b4) from [c0008578] (kernel_init+0x90/0x10c) [c0008578] (kernel_init+0x90/0x10c) from [c0028dbc] (kernel_thread_exit+0x0/0x8) Code: 18bd8070 e1a05000 e596309c e5932030 (e5b23048) ---[ end trace 31cd15fa922e3123 ]--- Kernel panic - not syncing: Attempted to kill init! Do you have any idea what I could be missing? Yes, I think you're missing the equivalent to commit 38f5cbed7710ab373252b64e3fb0701ab5a7b828 rx51: Add omap3isp_device pointer to platform data. Signed-off-by: Sakari Ailus sakari.ai...@maxwell.research.nokia.com -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform
Laurent Pinchart wrote: Hi Sergio, On Friday 22 January 2010 10:49:17 Aguirre, Sergio wrote: Laurent, Sakari, While I was trying to adapt my Zoom2/3 sensor drivers into latest 'devel' branch with latest commit: commit 2e7d09ec5e09ee80462a611c9958e99866ee337c Author: Laurent Pinchart laurent.pinch...@ideasonboard.com Date: Wed Jan 20 13:49:31 2010 +0100 omap3isp: Work around sg_alloc_table BUG_ON Work in progress Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com That last patch shouldn't have been applied to the linux-omap tree. The patch itself is correct, but the commit message isn't. I'll check that with Sakari. Killed the commit... -- Sakari Ailus sakari.ai...@maxwell.research.nokia.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform
Hi Laurent, -Original Message- From: Laurent Pinchart [mailto:laurent.pinch...@ideasonboard.com] Sent: Friday, January 22, 2010 5:46 AM To: Aguirre, Sergio Cc: Sakari Ailus; linux-media@vger.kernel.org Subject: Re: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform Hi Sergio, On Friday 22 January 2010 10:49:17 Aguirre, Sergio wrote: Laurent, Sakari, While I was trying to adapt my Zoom2/3 sensor drivers into latest 'devel' branch with latest commit: commit 2e7d09ec5e09ee80462a611c9958e99866ee337c Author: Laurent Pinchart laurent.pinch...@ideasonboard.com Date: Wed Jan 20 13:49:31 2010 +0100 omap3isp: Work around sg_alloc_table BUG_ON Work in progress Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com That last patch shouldn't have been applied to the linux-omap tree. The patch itself is correct, but the commit message isn't. I'll check that with Sakari. And applying the attached patches. I see the following kernel panic on bootup: omap3isp omap3isp: Revision 2.0 found omap-iommu omap-iommu.0: isp: version 1.1 Unable to handle kernel NULL pointer dereference at virtual address 0048 pgd = c0004000 [0048] *pgd= Internal error: Oops: 5 [#1] last sysfs file: Modules linked in: CPU: 0Not tainted (2.6.32-07583-gd4ae425-dirty #7) PC is at get_device_parent+0x68/0x114 LR is at device_add+0x7c/0x474 pc : [c019345c]lr : [c0194264]psr: 6013 sp : cf823dc8 ip : cf95f680 fp : cf91b178 r10: c03f37e0 r9 : c0367b06 r8 : c03dc770 r7 : ffea r6 : cf91b170 r5 : c03b7748 r4 : r3 : c03dc770 r2 : r1 : c03b7740 r0 : c03b7748 Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment kernel Control: 10c5387d Table: 80004019 DAC: 0017 Process swapper (pid: 1, stack limit = 0xcf8222e8) Stack: (0xcf823dc8 to 0xcf824000) 3dc0: cf91b170 cf91b170 c0194264 cf823e10 c03b7740 3de0: cf91b178 c01670a4 cf91b178 cf91b168 cf91b170 3e00: c03f33dc c03dc770 c0367b06 c03f37e0 c03b787c c0200744 c03b7740 cf91b160 3e20: c03b7740 cf91b000 c03d3cc0 c01fdcbc c03b8ae0 c03b8fc0 3e40: c03b7738 98f7cec1 cf820a30 c03b8f68 c0067f78 cf82c01c cf821a40 3e60: 0017 cf820a00 cf821bdc cf823eac 0001 fffd 3e80: cf823ebc cf803740 0001 cf803744 c03ed394 3ea0: cf823ef4 c016630c cf80d7a8 c016630c cf82dfa4 cf80d7a8 3ec0: c00e5cdc cf80e608 c02a8c58 cf823f10 cf95e7b8 c00e6474 3ee0: cf95e7b8 c00e60ac cf95e818 cf95e7b8 cf823f10 c00e6184 cf823f10 3f00: cf95e7b8 cf80e608 0001 c00e6eec cf80e608 c03b7740 3f20: c03b7740 c03dc440 cf95dbc0 c03d3cc0 c01971e4 3f40: c03b7740 c01963e0 c03b7740 c03b7774 c03dc440 cf95dbc0 c03d3cc0 c01964ec 3f60: c019648c c03dc440 c0195cb4 cf803af8 cf8459f0 c00222c8 c03dc440 3f80: c03dc440 c0195614 c03361b1 c03361b1 0006 c00222c8 c03dc440 3fa0: c01967bc c00222c8 c001c060 3fc0: c0027334 0031 0192 c00222c8 3fe0: c0008578 c0028dbc 00dbda20 24ffdc02 [c019345c] (get_device_parent+0x68/0x114) from [c0194264] (device_add+0x7c/0x474) [c0194264] (device_add+0x7c/0x474) from [c0200744] (media_devnode_register+0x1d0/0x29c) [c0200744] (media_devnode_register+0x1d0/0x29c) from [c01fdcbc] (omap34xxcam_probe+0x64/0x428) [c01fdcbc] (omap34xxcam_probe+0x64/0x428) from [c01971e4] (platform_drv_probe+0x18/0x1c) [c01971e4] (platform_drv_probe+0x18/0x1c) from [c01963e0] (driver_probe_device+0xa0/0x14c) [c01963e0] (driver_probe_device+0xa0/0x14c) from [c01964ec] (__driver_attach+0x60/0x84) [c01964ec] (__driver_attach+0x60/0x84) from [c0195cb4] (bus_for_each_dev+0x44/0x74) [c0195cb4] (bus_for_each_dev+0x44/0x74) from [c0195614] (bus_add_driver+0x9c/0x220) [c0195614] (bus_add_driver+0x9c/0x220) from [c01967bc] (driver_register+0xa8/0x130) [c01967bc] (driver_register+0xa8/0x130) from [c0027334] (do_one_initcall+0x5c/0x1b4) [c0027334] (do_one_initcall+0x5c/0x1b4) from [c0008578] (kernel_init+0x90/0x10c) [c0008578] (kernel_init+0x90/0x10c) from [c0028dbc] (kernel_thread_exit+0x0/0x8) Code: 18bd8070 e1a05000 e596309c e5932030 (e5b23048) ---[ end trace 31cd15fa922e3123 ]--- Kernel panic - not syncing: Attempted to kill init! Do you have any idea what I could be missing? Yes, I think you're missing the equivalent to commit 38f5cbed7710ab373252b64e3fb0701ab5a7b828 rx51: Add omap3isp_device pointer to platform data. Signed-off-by: Sakari Ailus sakari.ai...@maxwell.research.nokia.com I do have that change. In the patch 0003, I
RE: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform
Laurent, From: Aguirre, Sergio Sent: Friday, January 22, 2010 11:46 AM Hi Laurent, From: Laurent Pinchart [mailto:laurent.pinch...@ideasonboard.com] Sent: Friday, January 22, 2010 5:46 AM Hi Sergio, On Friday 22 January 2010 10:49:17 Aguirre, Sergio wrote: Laurent, Sakari, While I was trying to adapt my Zoom2/3 sensor drivers into latest 'devel' branch with latest commit: commit 2e7d09ec5e09ee80462a611c9958e99866ee337c Author: Laurent Pinchart laurent.pinch...@ideasonboard.com Date: Wed Jan 20 13:49:31 2010 +0100 omap3isp: Work around sg_alloc_table BUG_ON Work in progress Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com That last patch shouldn't have been applied to the linux-omap tree. The patch itself is correct, but the commit message isn't. I'll check that with Sakari. And applying the attached patches. I see the following kernel panic on bootup: omap3isp omap3isp: Revision 2.0 found omap-iommu omap-iommu.0: isp: version 1.1 Unable to handle kernel NULL pointer dereference at virtual address 0048 pgd = c0004000 [0048] *pgd= Internal error: Oops: 5 [#1] last sysfs file: Modules linked in: CPU: 0Not tainted (2.6.32-07583-gd4ae425-dirty #7) PC is at get_device_parent+0x68/0x114 LR is at device_add+0x7c/0x474 pc : [c019345c]lr : [c0194264]psr: 6013 sp : cf823dc8 ip : cf95f680 fp : cf91b178 r10: c03f37e0 r9 : c0367b06 r8 : c03dc770 r7 : ffea r6 : cf91b170 r5 : c03b7748 r4 : r3 : c03dc770 r2 : r1 : c03b7740 r0 : c03b7748 Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment kernel Control: 10c5387d Table: 80004019 DAC: 0017 Process swapper (pid: 1, stack limit = 0xcf8222e8) Stack: (0xcf823dc8 to 0xcf824000) 3dc0: cf91b170 cf91b170 c0194264 cf823e10 c03b7740 3de0: cf91b178 c01670a4 cf91b178 cf91b168 cf91b170 3e00: c03f33dc c03dc770 c0367b06 c03f37e0 c03b787c c0200744 c03b7740 cf91b160 3e20: c03b7740 cf91b000 c03d3cc0 c01fdcbc c03b8ae0 c03b8fc0 3e40: c03b7738 98f7cec1 cf820a30 c03b8f68 c0067f78 cf82c01c cf821a40 3e60: 0017 cf820a00 cf821bdc cf823eac 0001 fffd 3e80: cf823ebc cf803740 0001 cf803744 c03ed394 3ea0: cf823ef4 c016630c cf80d7a8 c016630c cf82dfa4 cf80d7a8 3ec0: c00e5cdc cf80e608 c02a8c58 cf823f10 cf95e7b8 c00e6474 3ee0: cf95e7b8 c00e60ac cf95e818 cf95e7b8 cf823f10 c00e6184 cf823f10 3f00: cf95e7b8 cf80e608 0001 c00e6eec cf80e608 c03b7740 3f20: c03b7740 c03dc440 cf95dbc0 c03d3cc0 c01971e4 3f40: c03b7740 c01963e0 c03b7740 c03b7774 c03dc440 cf95dbc0 c03d3cc0 c01964ec 3f60: c019648c c03dc440 c0195cb4 cf803af8 cf8459f0 c00222c8 c03dc440 3f80: c03dc440 c0195614 c03361b1 c03361b1 0006 c00222c8 c03dc440 3fa0: c01967bc c00222c8 c001c060 3fc0: c0027334 0031 0192 c00222c8 3fe0: c0008578 c0028dbc 00dbda20 24ffdc02 [c019345c] (get_device_parent+0x68/0x114) from [c0194264] (device_add+0x7c/0x474) [c0194264] (device_add+0x7c/0x474) from [c0200744] (media_devnode_register+0x1d0/0x29c) [c0200744] (media_devnode_register+0x1d0/0x29c) from [c01fdcbc] (omap34xxcam_probe+0x64/0x428) [c01fdcbc] (omap34xxcam_probe+0x64/0x428) from [c01971e4] (platform_drv_probe+0x18/0x1c) [c01971e4] (platform_drv_probe+0x18/0x1c) from [c01963e0] (driver_probe_device+0xa0/0x14c) [c01963e0] (driver_probe_device+0xa0/0x14c) from [c01964ec] (__driver_attach+0x60/0x84) [c01964ec] (__driver_attach+0x60/0x84) from [c0195cb4] (bus_for_each_dev+0x44/0x74) [c0195cb4] (bus_for_each_dev+0x44/0x74) from [c0195614] (bus_add_driver+0x9c/0x220) [c0195614] (bus_add_driver+0x9c/0x220) from [c01967bc] (driver_register+0xa8/0x130) [c01967bc] (driver_register+0xa8/0x130) from [c0027334] (do_one_initcall+0x5c/0x1b4) [c0027334] (do_one_initcall+0x5c/0x1b4) from [c0008578] (kernel_init+0x90/0x10c) [c0008578] (kernel_init+0x90/0x10c) from [c0028dbc] (kernel_thread_exit+0x0/0x8) Code: 18bd8070 e1a05000 e596309c e5932030 (e5b23048) ---[ end trace 31cd15fa922e3123 ]--- Kernel panic - not syncing: Attempted to kill init! Do you have any idea what I could be missing? Yes, I think you're missing the equivalent to commit 38f5cbed7710ab373252b64e3fb0701ab5a7b828 rx51: Add omap3isp_device pointer to platform data. Signed-off-by: Sakari Ailus sakari.ai...@maxwell.research.nokia.com I do have that change. In the patch 0003, I added devices.h, and