Hi Javier, On Friday 08 April 2011 17:30:54 javier Martin wrote: > On 8 April 2011 17:07, Laurent Pinchart wrote: > > On Friday 08 April 2011 17:02:48 javier Martin wrote: > >> Hi, > >> I've just received a LI-LBCM3M1 camera module from Leopard Imaging and > >> I want to test it with my Beagleboard xM. This module has a mt9t111 > >> sensor. > >> > >> At first glance, this driver > >> (http://lxr.linux.no/#linux+v2.6.38/drivers/media/video/mt9t112.c) > >> supports mt9t111 sensor and uses both soc-camera and v4l2-subdev > >> frameworks. > >> I am trying to somehow connect this sensor with the omap3isp driver > >> recently merged (I'm working with latest mainline kernel), however, I > >> found an issue when trying to pass "mt9t112_camera_info" data to the > >> sensor driver in my board specific file. > >> > >> It seems that this data is passed through soc-camera but omap3isp > >> doesn't use soc-camera. Do you know what kind of changes are required > >> to adapt this driver so that it can be used with omap3isp? > > > > The OMAP3 ISP driver isn't compatible with the soc-camera framework, as > > you correctly noticed. You will need to port the MT9T111 driver to > > pad-level subdev operations. > > > > You can find a sensor driver (MT9V032) implementing pad-level subdev > > operations at > > http://git.linuxtv.org/pinchartl/media.git?a=commit;h=940b87a5cb7ea3f3cff > > 16454e9085e33ab340064 > > Hi Laurent, > thank you for your quick answer. > > Does the fact of adding pad-level subdev operations for the sensor > break old way of doing things? Adding pad-level operations will not break any existing driver, as long as you keep the existing operations functional. > I mean, if I port MT9T111 driver to pad-level subdev operations would > it be accepted for mainline or would it be rejected since it breaks > something older? The patch will be accepted if you don't break anything :-) We first need to add pad-level operations to subdev drivers. The next step will be to convert bridge drivers to pad-level operations, at which point legacy operations will be removed from subdevs. -- Regards, Laurent Pinchart -- 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
