Hans, I see it. Thanks. Murali Karicheri Software Design Engineer Texas Instruments Inc. Germantown, MD 20874 Phone : 301-515-3736 email: m-karicheri2@xxxxxx >-----Original Message----- >From: Hans Verkuil [mailto:hverkuil@xxxxxxxxx] >Sent: Friday, June 26, 2009 2:53 AM >To: Karicheri, Muralidharan >Cc: linux-media@xxxxxxxxxxxxxxx; Mauro Carvalho Chehab >Subject: Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-subdev2 > >On Friday 26 June 2009 08:17:05 Hans Verkuil wrote: >> On Thursday 25 June 2009 20:18:06 Karicheri, Muralidharan wrote: >> > Hans, >> > >> > I have tried to pull the latest from > git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-2.6.git >> > >> > I can't see it part of this. Which GIT tree can I use to see the sub >dev api changes or latest that went into 2.6.31? Is vpfe capture part of >2.6.31? >> >> Ah, it is not yet pulled into the git tree. Mauro did send a pull request >for >> this to Linus, though. So I hope it will appear soon. > >Oops, I didn't look carefully enough. These API changes *are* merged in >2.6.31. > >I'm using Linus' git tree. > >Regards, > > Hans > >> >> Regards, >> >> Hans >> >> > >> > Thanks. >> > >> > Murali Karicheri >> > Software Design Engineer >> > Texas Instruments Inc. >> > Germantown, MD 20874 >> > email: m-karicheri2@xxxxxx >> > >> > >-----Original Message----- >> > >From: Hans Verkuil [mailto:hverkuil@xxxxxxxxx] >> > >Sent: Thursday, June 25, 2009 11:18 AM >> > >To: Karicheri, Muralidharan >> > >Cc: linux-media@xxxxxxxxxxxxxxx; Mauro Carvalho Chehab >> > >Subject: RE: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb- >subdev2 >> > > >> > > >> > >> Hi, Mauro, >> > >> >> > >> I am using the v4l2_i2c_new_subdev_board() API for the next set of >vpfe >> > >> capture driver patches. So when do you think this will be merged? >> > > >> > >This has already been merged and is also in the 2.6.31 git tree. >> > > >> > >I'm very pleased that this is in as that will make life easier for >several >> > >embedded system developments. >> > > >> > >Regards, >> > > >> > > Hans >> > > >> > >> >> > >> Murali Karicheri >> > >> Software Design Engineer >> > >> Texas Instruments Inc. >> > >> Germantown, MD 20874 >> > >> email: m-karicheri2@xxxxxx >> > >> >> > >>>-----Original Message----- >> > >>>From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media- >> > >>>owner@xxxxxxxxxxxxxxx] On Behalf Of Hans Verkuil >> > >>>Sent: Saturday, June 20, 2009 9:11 AM >> > >>>To: linux-media@xxxxxxxxxxxxxxx >> > >>>Cc: Mauro Carvalho Chehab >> > >>>Subject: Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb- >subdev2 >> > >>> >> > >>>On Tuesday 09 June 2009 22:40:35 Hans Verkuil wrote: >> > >>>> Hi Mauro, >> > >>>> >> > >>>> Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb- >subdev2 >> > >>>> for >> > >>>> the following: >> > >>>> >> > >>>> - v4l2: add new s_config subdev ops and >v4l2_i2c_new_subdev_cfg/board >> > >>>> calls - v4l2-device: fix incorrect kernel check >> > >>>> - v4l-compat: add I2C_ADDRS macro. >> > >>>> - v4l2: update framework documentation. >> > >>>> - v4l2-subdev: remove unnecessary check >> > >>>> >> > >>>> This time I've only added new functions and left the existing ones >in >> > >>>> place. I did add a bit of code to the existing >> > >>>> v4l2_i2c_new_(probed_)subdev functions to call the new s_config op >if >> > >>>> it >> > >>>> is available. Existing subdev drivers never set this new op, so >this >> > >>>> code >> > >>>> will not effect current behavior. But for new drivers that do set >> > >>>> s_config it is important that it is called no matter what flavor >of >> > >>>> these >> > >>>> functions is used. >> > >>>> >> > >>>> At the end of the 2.6.31 cycle we can replace the current >> > >>>> v4l2_i2c_new_(probed_)subdev calls with the new one I had in my >earlier >> > >>>> patches. >> > >>> >> > >>>Hi Mauro, >> > >>> >> > >>>I've posted these changes as an RFC more than a week ago, but since >there >> > >>>were no comments I hope you can pull from this tree for 2.6.31. >> > >>> >> > >>>I would really, really like to get this into 2.6.31. It will help >anyone >> > >>>who >> > >>>is working with subdevs on embedded platforms. >> > >>> >> > >>>Regards, >> > >>> >> > >>> Hans >> > >>> >> > >>>> >> > >>>> Thanks, >> > >>>> >> > >>>> Hans >> > >>>> >> > >>>> diffstat: >> > >>>> linux/Documentation/video4linux/v4l2-framework.txt | 24 +++ >> > >>>> linux/drivers/media/video/v4l2-common.c | 166 >> > >>>> +++++++++++++++++++++ >> > >>>> linux/drivers/media/video/v4l2-device.c | 2 >> > >>>> linux/include/media/v4l2-common.h | 18 ++ >> > >>>> linux/include/media/v4l2-subdev.h | 9 - >> > >>>> v4l/compat.h | 6 >> > >>>> 6 files changed, 222 insertions(+), 3 deletions(-) >> > >>> >> > >>> >> > >>> >> > >>>-- >> > >>>Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom >> > >>>-- >> > >>>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 >> > >> >> > >> >> > >> >> > > >> > > >> > >-- >> > >Hans Verkuil - video4linux developer - sponsored by TANDBERG >> > > >> > >> > >> > >> > >> >> >> > > > >-- >Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom -- 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
