On Monday, September 13, 2010 17:27:52 Sakari Ailus wrote: > Aguirre, Sergio wrote: > > Hi, > > Hi Sergio, > > > I was wondering if there exists a current standard way to query a > > Imaging sensor driver for knowing things like the signal vert/horz blanking time. > > > > In an old TI custom driver, we used to have a private IOCTL in the sensor > > Driver we interfaced with the omap3 ISP, which was basically reporting: > > > > - Active resolution (Actual image size) > > - Full resolution (Above size + dummy pixel columns/rows representing blanking times) > > > > However I resist to keep importing that custom interface, since I think its > > Something that could be already part of an standard API. > > The N900 sensor drivers currently use private controls for this purpose. > That is an issue which should be resolved. I agree there should be a > uniform, standard way to access this information. > > What we currently have is this, not in upstream: > > --- > /* SMIA-type sensor information */ > #define V4L2_CID_MODE_CLASS_BASE (V4L2_CTRL_CLASS_MODE | > 0x900) > #define V4L2_CID_MODE_CLASS (V4L2_CTRL_CLASS_MODE | 1) > #define V4L2_CID_MODE_FRAME_WIDTH (V4L2_CID_MODE_CLASS_BASE+1) > #define V4L2_CID_MODE_FRAME_HEIGHT (V4L2_CID_MODE_CLASS_BASE+2) > #define V4L2_CID_MODE_VISIBLE_WIDTH (V4L2_CID_MODE_CLASS_BASE+3) > #define V4L2_CID_MODE_VISIBLE_HEIGHT (V4L2_CID_MODE_CLASS_BASE+4) > #define V4L2_CID_MODE_PIXELCLOCK (V4L2_CID_MODE_CLASS_BASE+5) > #define V4L2_CID_MODE_SENSITIVITY (V4L2_CID_MODE_CLASS_BASE+6) > --- > > The pixel clock is read-only but some of the others should likely be > changeable. It is very similar to the VIDIOC_G/S_DV_TIMINGS ioctls. I think we should look into adding an e.g. V4L2_DV_SMIA_SENSOR type or something along those lines. I'm no sensor expert, so I don't know what sort of timing information is needed for the various sensor types. But I'm sure there are other people who have this knowledge. It would be useful if someone can list the information that you need from the various sensor types. Based on that we can see if this ioctl is a good fit. Regards, Hans > > Regards, > > -- Hans Verkuil - video4linux developer - sponsored by TANDBERG, part of Cisco -- 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
