Re: [PATCH 5/5] LDP: Add support for built-in camera

["stanley.miao" <stanley.miao@xxxxxxxxxxxxx>]

Aguirre Rodriguez, Sergio Alberto wrote:
>   
> > -----Original Message-----
> > From: stanley.miao [mailto:stanley.miao@xxxxxxxxxxxxx]
> > Sent: Tuesday, March 10, 2009 3:04 AM
> > To: Aguirre Rodriguez, Sergio Alberto
> > Cc: linux-media@xxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx; Sakari Ailus;
> > Tuukka.O Toivonen; Hiroshi DOYU; DongSoo(Nathaniel) Kim; MiaoStanley;
> > Nagalla, Hari; Hiremath, Vaibhav; Lakhani, Amish; Menon, Nishanth
> > Subject: RE: [PATCH 5/5] LDP: Add support for built-in camera
> >
> > On Mon, 2009-03-09 at 15:24 -0500, Aguirre Rodriguez, Sergio Alberto
> > wrote:
> >     
> > <snip>
> > out
> >     
> > > > when I run my test program.
> > > >
> > > > <3>CSI2: ComplexIO Error IRQ 80
> > > > CSI2: ComplexIO Error IRQ 80
> > > > <3>CSI2: ComplexIO Error IRQ c2
> > > > CSI2: ComplexIO Error IRQ c2
> > > > <3>CSI2: ComplexIO Error IRQ c2
> > > > CSI2: ComplexIO Error IRQ c2
> > > > <3>CSI2: ComplexIO Error IRQ c6
> > > > CSI2: ComplexIO Error IRQ c6
> > > > <3>CSI2: ECC correction failed
> > > > CSI2: ECC correction failed
> > > > <3>CSI2: ComplexIO Error IRQ 6
> > > > CSI2: ComplexIO Error IRQ 6
> > > > <3>CSI2: ComplexIO Error IRQ 6
> > > > CSI2: ComplexIO Error IRQ 6
> > > > <3>CSI2: ComplexIO Error IRQ 6
> > > > CSI2: ComplexIO Error IRQ 6
> > > > <3>CSI2: ComplexIO Error IRQ 6
> > > > CSI2: ComplexIO Error IRQ 6
> > > >
> > > >         
> > > Oops, my mistake. Missed to add that struct there... Fixed now.
> > >
> > > About the CSI2 errors you're receiving... Which version of LDP are you
> > >       
> > using? Which Silicon revision has (ES2.1 or ES3.0)?
> >
> > ZOOM1 board(LDP3430-VG1.0.0-1), omap3430 ES2.1.
> >
> > When I use your old version patch, sometimes the test succeed, sometimes
> > failed(no data was generated and no error). This version, always failed.
> >     
>
> Stanley,
>
> I'm working on some CSI2 fixes that could help you with this..
>
> I'll keep you updated on this.
>
> Also, about your board version, it is possible that you'll need a HW modfix because of a redundant resistor in CSI2 datalanes. I'm in talks with some people at TI to prepare a ready to publish document about this rework.
>
> Zoom1 with ES3 silicon doesn't need this HW fix.
>   
Hi, Sergio,

When I used your version in omapzoom tree, the camera worked sometimes. 
So I think it's not HW's problem.

My test program is capture.c that I got from v4l2 spec and did some 
modifications.
This is my test log:

root@localhost:/root> ./capture -d /dev/video4
select timeout
read_frame: Resource temporarily unavailable
select timeout
read_frame: Resource temporarily unavailable
^C

root@localhost:/root> ./capture -d /dev/video4
Got 20 frames data, success.

root@localhost:/root> ./capture -d /dev/video4
CSI2: ECC correction failed
CSI2: Short packet receive error
CSI2: ECC correction failed
CSI2: Short packet receive error
CSI2: ECC correction failed
CSI2: Short packet receive error
CSI2: ECC correction failed
CSI2: Short packet receive error
CSI2: ECC correction failed
CSI2: Short packet receive error
CSI2: ECC correction failed
CSI2: Short packet receive error
CSI2: ECC correction failed
select timeout
read_frame: Resource temporarily unavailable
select timeout
read_frame: Resource temporarily unavailable
select timeout
read_frame: Resource temporarily unavailable
select timeout
read_frame: Resource temporarily unavailable
^C

"select timeout" means that no data was generated.

Stanley.

> I'll hope to get back to you about this next week.
>
> Regards,
> Sergio
>
>   
> > Thanks.
> > Stanley.
> >
> >     
> > > Regards,
> > > Sergio
> > >       
> > > > Stanley.
> > > >         
>
>
>   

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html