> -----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: > > Hi, > > > > > -----Original Message----- > > > From: stanley.miao [mailto:stanley.miao@xxxxxxxxxxxxx] > > > > <snip> > > > > > > +static struct i2c_board_info __initdata ldp_i2c_boardinfo_2[] = { > > > > +#if defined(CONFIG_VIDEO_OV3640) || > defined(CONFIG_VIDEO_OV3640_MODULE) > > > > + { > > > > + I2C_BOARD_INFO("ov3640", OV3640_I2C_ADDR), > > > > + .platform_data = &ldp_ov3640_platform_data, > > > > + }, > > > > +#endif > > > > +}; > > > > > > > This new added i2c_board_info was not registered. > > > After I added this line, > > > > > > omap_register_i2c_bus(2, 400, ldp_i2c_boardinfo_2, > > > ARRAY_SIZE(ldp_i2c_boardinfo_2)); > > > > > > the sensor was found. but the test still failed. A lot of error came > 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. 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-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
