Re: gitorious.org/omap3camera: Falied attempt to migrate sensor driver to Zoom2/3 platform

2010-01-24 Thread Laurent Pinchart
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

2010-01-22 Thread Laurent Pinchart
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

2010-01-22 Thread Sakari Ailus
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

2010-01-22 Thread Aguirre, Sergio
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

2010-01-22 Thread Aguirre, Sergio
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