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