Help bringing up a sensor driver for isp omap34xx.c

Help bringing up a sensor driver for isp omap34xx.c

[John Sarman]

Hello,
   I am having a problem deciphering what is wrong with my sensor
driver.  It seems that everything operates on the driver but that I am
getting buffer overflows.  I have fully tested the image sensor and it
is set to operate in 640x480 mode. currently it is like 648x 487 for
the dummy pixels and lines.  I have enabled all the debugging #defines
in the latest code from the gitorious repository.  I also had to edit
a few debug statements because they cause the compile to fail. Those
failures were due to the resizer rewrite and since the #defines were
commented out that code was never compiled.  Anyways here is my dmesg
after I open and select the /dev/video0.

I have been banging my head against a wall for 2 weeks now.

Thanks,

OV5620: ioctl_g_priv
OV5620: ov5620_probe
OV5620: ioctl_g_priv
OV5620: ioctl_g_priv
OV5620: ioctl_g_priv
OV5620: ioctl_g_priv
ISPCTRL: isp_get: old 0
ISPCCDC: Restoring context
ISPHIST:  Restoring context
ISPH3A:  Restoring context
ISPPREV: Restoring context
ISPRESZ: Restoring context
ISPCTRL: isp_get: new 1
OV5620: ioctl_s_power
ISPCTRL: isp_set_xclk(): cam_xclka set to 24000000 Hz
BOARD_OVERO_CAMERA: Switching Power to 1
OV5620: POWER ON
OV5620: Sensor not detected, calling ioctl_dev_init(s)
OV5620: ioctl_dev_init
ov5620:detect
ov5620 3-0030: model id detected 0x5621 mfr 0x7fa2
ov5620 3-0030: Chip version 0x02 detected
OV5620: Sensor Detected, calling configure
ov5620:configure
OV5620: ioctl_g_fmt_cap
OV5620: ioctl_s_power
OV5620: POWER OFF
BOARD_OVERO_CAMERA: Switching Power to 0
OV5620: POWER OFF
ISPCTRL: isp_set_xclk(): cam_xclka set to 0 Hz
ISPCTRL: isp_put: old 1
ISPCCDC: Saving context
ISPHIST:  Saving context
ISPH3A:  Saving context
ISPPREV: Saving context
ISPRESZ: Saving context
ISPCCDC: ISP_ERR: CCDC Module already freed
ISPRESZ: ISP_ERR : Resizer Module already freed
ISPCTRL: isp_put: new 0
ISPCTRL: isp_get: old 0
ISPCCDC: Restoring context
ISPHIST:  Restoring context
ISPH3A:  Restoring context
ISPPREV: Restoring context
ISPRESZ: Restoring context
ISPCTRL: isp_get: new 1
OV5620: ioctl_s_power
ISPCTRL: isp_set_xclk(): cam_xclka set to 24000000 Hz
BOARD_OVERO_CAMERA: Switching Power to 1
OV5620: POWER ON
OV5620: Sensor Detected, calling configure
ov5620:configure
OV5620: ioctl_s_power
BOARD_OVERO_CAMERA: Switching Power to 2
OV5620: POWER STANDBY
ISPCTRL: isp_set_xclk(): cam_xclka set to 0 Hz
OV5620: ioctl_g_fmt_cap
OV5620: ioctl_enum_fmt_cap
OV5620: ioctl_enum_fmt_cap index 0 type 1
OV5620: ioctl_enum_framesizes
OV5620: ioctl_enum_frameintervals
OV5620:frmi->index = 0
OV5620: ioctl_enum_frameintervals
OV5620: ioctl_enum_framesizes
OV5620: ioctl_enum_frameintervals
OV5620:frmi->index = 0
OV5620: ioctl_enum_frameintervals
OV5620: ioctl_enum_framesizes
OV5620: ioctl_enum_fmt_cap
OV5620: ioctl_enum_fmt_cap index 1 type 1
ISPCCDC: ISP_ERR: CCDC Module already freed
ISPRESZ: ISP_ERR : Resizer Module already freed
ISPCCDC: Module in use =1
ISPCCDC: Accepted CCDC Input (width = 640,Height = 480)
ISPCCDC: Accepted CCDC Output (width = 640,Height = 479)
ISPCCDC: ###CCDC PCR=0x0
ISPCCDC: ISP_CTRL =0x39e150
ISPCCDC: ccdc input format is CCDC_RAW
ISPCCDC: ccdc output format is CCDC_OTHERS_VP
ISPCCDC: ###ISP_CTRL in ccdc =0x39e150
ISPCCDC: ###ISP_IRQ0ENABLE in ccdc =0x0
ISPCCDC: ###ISP_IRQ0STATUS in ccdc =0x82000000
ISPCCDC: ###CCDC SYN_MODE=0x10600
ISPCCDC: ###CCDC HORZ_INFO=0x27f
ISPCCDC: ###CCDC VERT_START=0x0
ISPCCDC: ###CCDC VERT_LINES=0x1df
ISPCCDC: ###CCDC CULLING=0xffff00ff
ISPCCDC: ###CCDC HSIZE_OFF=0x500
ISPCCDC: ###CCDC SDOFST=0x0
ISPCCDC: ###CCDC SDR_ADDR=0x266000
ISPCCDC: ###CCDC CLAMP=0x10
ISPCCDC: ###CCDC COLPTN=0xbb11bb11
ISPCCDC: ###CCDC CFG=0x8000
ISPCCDC: ###CCDC VP_OUT=0x77c5000
ISPCCDC: ###CCDC_SDR_ADDR= 0x266000
ISPCCDC: ###CCDC FMTCFG=0xe000
ISPCCDC: ###CCDC FMT_HORZ=0x500
ISPCCDC: ###CCDC FMT_VERT=0x3c0
ISPCCDC: ###CCDC LSC_CONFIG=0x6608
ISPCCDC: ###CCDC LSC_INIT=0x0
ISPCCDC: ###CCDC LSC_TABLE BASE=0x1000
ISPCCDC: ###CCDC LSC TABLE OFFSET=0x60
ISPCTRL: ###ISP_CTRL=0x39e150
ISPCTRL: ###ISP_TCTRL_CTRL=0x0
ISPCTRL: ###ISP_SYSCONFIG=0x2000
ISPCTRL: ###ISP_SYSSTATUS=0x1
ISPCTRL: ###ISP_IRQ0ENABLE=0x0
ISPCTRL: ###ISP_IRQ0STATUS=0x82000000
ISPPREV: 	Configuring brightness in ISP: 0
ISPRESZ: ispresizer_config_datapath()+
ISPRESZ: ispresizer_config_ycpos()+
ISPRESZ: ispresizer_config_ycpos()-
ISPRESZ: ispresizer_config_filter_coef()+
ISPRESZ: ispresizer_config_filter_coef()-
ISPRESZ: ispresizer_enable_cbilin()+
ISPRESZ: ispresizer_enable_cbilin()-
ISPRESZ: ispresizer_config_luma_enhance()+
ISPRESZ: ispresizer_config_luma_enhance()-
ISPRESZ: ispresizer_config_datapath()-
ISPRESZ: ispresizer_config_inlineoffset()+
ISPRESZ: ispresizer_config_inlineoffset()-
ISPRESZ: ispresizer_set_inaddr()+
ISPRESZ: ispresizer_set_inaddr()-
ISPRESZ: ispresizer_config_outlineoffset()+
ISPRESZ: ispresizer_config_outlineoffset()-
ISPRESZ: ispresizer_config_ycpos()+
ISPRESZ: ispresizer_config_ycpos()-
ISPRESZ: ispresizer_config_size()-
OV5620: ioctl_g_fmt_cap
OV5620: ioctl_s_fmt_cap
OV5620: ioctl_try_fmt_cap
OV5620: ioctl_try_fmt_cap before WIDTH = 640
OV5620: ioctl_try_fmt_cap before HEIGHT = 480
OV5620: ioctl_try_fmt_cap WIDTH = 640
OV5620: ioctl_try_fmt_cap HEIGHT = 480
OV5620: ioctl_s_parm
OV5620 desired_fps = 60
OV5620: ioctl_g_fmt_cap
ISPRESZ: ispresizer_config_datapath()+
ISPRESZ: ispresizer_config_ycpos()+
ISPRESZ: ispresizer_config_ycpos()-
ISPRESZ: ispresizer_config_filter_coef()+
ISPRESZ: ispresizer_config_filter_coef()-
ISPRESZ: ispresizer_enable_cbilin()+
ISPRESZ: ispresizer_enable_cbilin()-
ISPRESZ: ispresizer_config_luma_enhance()+
ISPRESZ: ispresizer_config_luma_enhance()-
ISPRESZ: ispresizer_config_datapath()-
ISPRESZ: ispresizer_config_inlineoffset()+
ISPRESZ: ispresizer_config_inlineoffset()-
ISPRESZ: ispresizer_set_inaddr()+
ISPRESZ: ispresizer_set_inaddr()-
ISPRESZ: ispresizer_config_outlineoffset()+
ISPRESZ: ispresizer_config_outlineoffset()-
ISPRESZ: ispresizer_config_ycpos()+
ISPRESZ: ispresizer_config_ycpos()-
ISPRESZ: ispresizer_config_size()-
OV5620: ioctl_enum_fmt_cap
OV5620: ioctl_enum_fmt_cap index 0 type 1
OV5620: ioctl_enum_framesizes
OV5620: ioctl_enum_frameintervals
OV5620:frmi->index = 0
OV5620: ioctl_enum_frameintervals
OV5620: ioctl_enum_framesizes
OV5620: ioctl_enum_frameintervals
OV5620:frmi->index = 0
OV5620: ioctl_enum_frameintervals
OV5620: ioctl_enum_framesizes
OV5620: ioctl_enum_fmt_cap
OV5620: ioctl_enum_fmt_cap index 1 type 1
ISPCCDC: Module in use =1
ISPCCDC: Accepted CCDC Input (width = 640,Height = 480)
ISPCCDC: Accepted CCDC Output (width = 640,Height = 479)
ISPCCDC: ###CCDC PCR=0x0
ISPCCDC: ISP_CTRL =0x29c150
ISPCCDC: ccdc input format is CCDC_RAW
ISPCCDC: ccdc output format is CCDC_OTHERS_VP
ISPCCDC: ###ISP_CTRL in ccdc =0x29c150
ISPCCDC: ###ISP_IRQ0ENABLE in ccdc =0x0
ISPCCDC: ###ISP_IRQ0STATUS in ccdc =0x82000000
ISPCCDC: ###CCDC SYN_MODE=0x10600
ISPCCDC: ###CCDC HORZ_INFO=0x27f
ISPCCDC: ###CCDC VERT_START=0x0
ISPCCDC: ###CCDC VERT_LINES=0x1df
ISPCCDC: ###CCDC CULLING=0xffff00ff
ISPCCDC: ###CCDC HSIZE_OFF=0x500
ISPCCDC: ###CCDC SDOFST=0x0
ISPCCDC: ###CCDC SDR_ADDR=0x266000
ISPCCDC: ###CCDC CLAMP=0x10
ISPCCDC: ###CCDC COLPTN=0xbb11bb11
ISPCCDC: ###CCDC CFG=0x8000
ISPCCDC: ###CCDC VP_OUT=0x3bc2800
ISPCCDC: ###CCDC_SDR_ADDR= 0x266000
ISPCCDC: ###CCDC FMTCFG=0xe000
ISPCCDC: ###CCDC FMT_HORZ=0x280
ISPCCDC: ###CCDC FMT_VERT=0x1e0
ISPCCDC: ###CCDC LSC_CONFIG=0x6608
ISPCCDC: ###CCDC LSC_INIT=0x0
ISPCCDC: ###CCDC LSC_TABLE BASE=0x1000
ISPCCDC: ###CCDC LSC TABLE OFFSET=0x60
ISPCTRL: ###ISP_CTRL=0x29c150
ISPCTRL: ###ISP_TCTRL_CTRL=0x0
ISPCTRL: ###ISP_SYSCONFIG=0x2000
ISPCTRL: ###ISP_SYSSTATUS=0x1
ISPCTRL: ###ISP_IRQ0ENABLE=0x0
ISPCTRL: ###ISP_IRQ0STATUS=0x82000000
ISPPREV: 	Configuring brightness in ISP: 0
ISPRESZ: ispresizer_config_datapath()+
ISPRESZ: ispresizer_config_ycpos()+
ISPRESZ: ispresizer_config_ycpos()-
ISPRESZ: ispresizer_config_filter_coef()+
ISPRESZ: ispresizer_config_filter_coef()-
ISPRESZ: ispresizer_enable_cbilin()+
ISPRESZ: ispresizer_enable_cbilin()-
ISPRESZ: ispresizer_config_luma_enhance()+
ISPRESZ: ispresizer_config_luma_enhance()-
ISPRESZ: ispresizer_config_datapath()-
ISPRESZ: ispresizer_config_inlineoffset()+
ISPRESZ: ispresizer_config_inlineoffset()-
ISPRESZ: ispresizer_set_inaddr()+
ISPRESZ: ispresizer_set_inaddr()-
ISPRESZ: ispresizer_config_outlineoffset()+
ISPRESZ: ispresizer_config_outlineoffset()-
ISPRESZ: ispresizer_config_ycpos()+
ISPRESZ: ispresizer_config_ycpos()-
ISPRESZ: ispresizer_config_size()-
OV5620: ioctl_g_fmt_cap
OV5620: ioctl_s_fmt_cap
OV5620: ioctl_try_fmt_cap
OV5620: ioctl_try_fmt_cap before WIDTH = 640
OV5620: ioctl_try_fmt_cap before HEIGHT = 480
OV5620: ioctl_try_fmt_cap WIDTH = 640
OV5620: ioctl_try_fmt_cap HEIGHT = 480
OV5620: ioctl_s_parm
OV5620 desired_fps = 60
ISPRESZ: ispresizer_set_inaddr()+
ISPRESZ: ispresizer_set_inaddr()-
OV5620: ioctl_s_power
ISPCTRL: isp_set_xclk(): cam_xclka set to 24000000 Hz
BOARD_OVERO_CAMERA: Switching Power to 1
OV5620: POWER ON
OV5620: Sensor Detected, calling configure
ov5620:configure
ISPCTRL: <1>isp_buf_queue: queue 0 vb 0, mmu 000a4000
ISPCTRL: <1>isp_buf_queue: queue 1 vb 1, mmu 0013a000
ISPCTRL: <1>isp_buf_queue: queue 2 vb 2, mmu 001d0000
ISPCTRL: <1>isp_buf_queue: queue 3 vb 3, mmu 00266000
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
ISPH3A:     H3A disabled
ISPHIST:    histogram disabled
ISPRESZ: +ispresizer_enable()+
ISPRESZ: +ispresizer_enable()-
omap3isp omap3isp: __isp_disable_modules: can't stop ccdc
ISPCCDC: Saving context
ISPHIST:  Saving context
ISPH3A:  Saving context
ISPPREV: Saving context
ISPRESZ: Saving context
ISPCCDC: Restoring context
ISPHIST:  Restoring context
ISPH3A:  Restoring context
ISPPREV: Restoring context
ISPRESZ: Restoring context
OV5620: ioctl_s_power
BOARD_OVERO_CAMERA: Switching Power to 2
OV5620: POWER STANDBY
ISPCTRL: isp_set_xclk(): cam_xclka set to 0 Hz
OV5620: ioctl_s_power
OV5620: POWER OFF
BOARD_OVERO_CAMERA: Switching Power to 0
OV5620: POWER OFF
ISPCTRL: isp_set_xclk(): cam_xclka set to 0 Hz
ISPCTRL: isp_put: old 1
ISPCCDC: Saving context
ISPHIST:  Saving context
ISPH3A:  Saving context
ISPPREV: Saving context
ISPRESZ: Saving context
ISPCTRL: isp_put: new 0

RE: Help bringing up a sensor driver for isp omap34xx.c

[Aguirre Rodriguez, Sergio Alberto]

(Unlooping Sameer and Mohit, as they don't longer maintain the driver)

Hi John,

> -----Original Message-----
> From: John Sarman [mailto:johnsarman@gmail.com]
> Sent: Wednesday, July 15, 2009 4:45 PM
> To: sakari.ailus@nokia.com; Venkatraman, Sameer; Mohit Jalori; Aguirre
> Rodriguez, Sergio Alberto; Tuukka Toivonen; linux-media
> Subject: Help bringing up a sensor driver for isp omap34xx.c
> 
> Hello,
>    I am having a problem deciphering what is wrong with my sensor
> driver.  It seems that everything operates on the driver but that I am
> getting buffer overflows.  I have fully tested the image sensor and it
> is set to operate in 640x480 mode. currently it is like 648x 487 for
> the dummy pixels and lines.  I have enabled all the debugging #defines
> in the latest code from the gitorious repository.

Can you specify the gitorious repository URL you're using?

  I also had to edit
> a few debug statements because they cause the compile to fail. Those
> failures were due to the resizer rewrite and since the #defines were
> commented out that code was never compiled.  Anyways here is my dmesg
> after I open and select the /dev/video0.
> 
> I have been banging my head against a wall for 2 weeks now.
> 
> Thanks,
> 

<snip>

> ISPCTRL: <1>isp_buf_queue: queue 0 vb 0, mmu 000a4000
> ISPCTRL: <1>isp_buf_queue: queue 1 vb 1, mmu 0013a000
> ISPCTRL: <1>isp_buf_queue: queue 2 vb 2, mmu 001d0000
> ISPCTRL: <1>isp_buf_queue: queue 3 vb 3, mmu 00266000
> ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
> ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
> ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:
> ISPCTRL: HS_VS_IRQ <6>ISPCTRL: OVF_IRQ <6>ISPCTRL:

Seems that you're getting an overflow in the SBL (Shared Buffer Logic) component, which is the one that manages to save/load the buffers from memory.

It could happen because the SBL is writing pretty slow to memory...

Is it possible that you share your patches to integrate this sensor driver + boardfile changes you did?

That way I can help you more.

Regards,
Sergio

Re: Help bringing up a sensor driver for isp omap34xx.c

[Sakari Ailus]

(Dropped Sameer and Mohit from Cc.)

John Sarman wrote:
> Hello,

Hi,

>    I am having a problem deciphering what is wrong with my sensor
> driver.  It seems that everything operates on the driver but that I am
> getting buffer overflows.  I have fully tested the image sensor and it
> is set to operate in 640x480 mode. currently it is like 648x 487 for
> the dummy pixels and lines.  I have enabled all the debugging #defines
> in the latest code from the gitorious repository.  I also had to edit
> a few debug statements because they cause the compile to fail. Those
> failures were due to the resizer rewrite and since the #defines were
> commented out that code was never compiled.  Anyways here is my dmesg
> after I open and select the /dev/video0.
> 
> I have been banging my head against a wall for 2 weeks now.
> 
> Thanks,

...

ISPSBL_PCR_CCDCPRV_2_RSZ_OVF very often without any ill effects (AFAIR) 
which consequently causes OVF_IRQ to be triggered. It can be ignored.

How are your images? :) Printing things that big from the interrupt 
handler might hamper with your image captuting efforts, too.

Cheers,

-- 
Sakari Ailus
sakari.ailus@maxwell.research.nokia.com

Re: Help bringing up a sensor driver for isp omap34xx.c

[John Sarman]

Sakari,
My images are not so pretty yet, but that is because I haven't set any
real configuration on the image sensor yet.  First I would like to get
the data streaming to the framebuffer. That will allow me to focus the
image, then I will start applying the i2c calls needed.   Also I have
an idea for creating a sensor driver, one to rule them all so to
speak.  What are your opinions on a driver that would read a firmware
file that basically contains the i2c calls that are sensor specific.
In the firmware block it would have the necessary code to first find
the sensor, then it would have blocks to do the various sensor related
features, such as init ,gain, exposure , etc, etc.  This model would
allow for anyone to use an image sensor with confidential datasheets
because they could place there firmware in say /lib/firmware/imaging
and not be forced to have that GPL'ed.  Plus everyone benefits from a
highly tested GPL'ed driver.  I even image a tool that allows
designers to generate the firmware files while viewing the data on the
framebuffer, but thats a future discussion.

Thanks,
John Sarman

On Mon, Jul 20, 2009 at 5:08 AM, Sakari Ailus<sakari.ailus@nokia.com> wrote:
> (Dropped Sameer and Mohit from Cc.)
>
> John Sarman wrote:
>>
>> Hello,
>
> Hi,
>
>>   I am having a problem deciphering what is wrong with my sensor
>> driver.  It seems that everything operates on the driver but that I am
>> getting buffer overflows.  I have fully tested the image sensor and it
>> is set to operate in 640x480 mode. currently it is like 648x 487 for
>> the dummy pixels and lines.  I have enabled all the debugging #defines
>> in the latest code from the gitorious repository.  I also had to edit
>> a few debug statements because they cause the compile to fail. Those
>> failures were due to the resizer rewrite and since the #defines were
>> commented out that code was never compiled.  Anyways here is my dmesg
>> after I open and select the /dev/video0.
>>
>> I have been banging my head against a wall for 2 weeks now.
>>
>> Thanks,
>
> ...
>
> ISPSBL_PCR_CCDCPRV_2_RSZ_OVF very often without any ill effects (AFAIR)
> which consequently causes OVF_IRQ to be triggered. It can be ignored.
>
> How are your images? :) Printing things that big from the interrupt handler
> might hamper with your image captuting efforts, too.
>
> Cheers,
>
> --
> Sakari Ailus
> sakari.ailus@maxwell.research.nokia.com
>