On Fri, 19 Nov 2010, Hans Verkuil wrote:

> On Friday 19 November 2010 14:26:42 Laurent Pinchart wrote:
> > Some buggy sensors generate corrupt frames when the stream is started.
> > This new operation returns the number of corrupt frames to skip when
> > starting the stream.
> 
> Looks OK, but perhaps the two should be combined to one function?
> 
> I also have my doubts about the sensor_ops in general. I expected originally
> to see a lot of ops here, but apparently there is little or no need for it.
> 
> Do we expect to see this grow, or would it make more sense to move the ops
> to video_ops? I'd be interested to hear what sensor specialists think.

I would keep the struct.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to