Re: vpif_display.c bug

2009-08-11 Thread Hans Verkuil
On Monday 10 August 2009 17:07:51 Karicheri, Muralidharan wrote: > Hans, > > I have already changed v4l2_i2c_new_probed_subdev() to > v4l2_i2c_new_subdev_board() in my latest patch set for adding vpif capture > driver for DM6467 that you had reviewed. I think this change is not needed > once tha

RE: vpif_display.c bug

2009-08-11 Thread chaithrika
---Original Message- > >From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > >ow...@vger.kernel.org] On Behalf Of Subrahmanya, Chaithrika > >Sent: Monday, August 10, 2009 8:51 AM > >To: 'Hans Verkuil' > >Cc: linux-media@vger.kernel.org > &

RE: vpif_display.c bug

2009-08-10 Thread Karicheri, Muralidharan
il' >Cc: linux-media@vger.kernel.org >Subject: RE: vpif_display.c bug > >On Mon, Aug 10, 2009 at 11:37:23, Hans Verkuil wrote: >> Hi Chaithrika, >> >> This code in vpif_display.c is not correct: >> >> for (i = 0; i < subdev_count; i++) { >>

RE: vpif_display.c bug

2009-08-10 Thread Karicheri, Muralidharan
Hans, I have already changed v4l2_i2c_new_probed_subdev() to v4l2_i2c_new_subdev_board() in my latest patch set for adding vpif capture driver for DM6467 that you had reviewed. I think this change is not needed once that patch is applied. Murali Karicheri Software Design Engineer Texas Instrume

RE: vpif_display.c bug

2009-08-10 Thread chaithrika
On Mon, Aug 10, 2009 at 11:37:23, Hans Verkuil wrote: > Hi Chaithrika, > > This code in vpif_display.c is not correct: > > for (i = 0; i < subdev_count; i++) { > vpif_obj.sd[i] = > v4l2_i2c_new_probed_subdev(&vpif_obj.v4l2_dev, >