Re: [PATCH] soc-camera: ov7670 merged multiple drivers and moved over to v4l2-subdev

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 15 Jun 2009, Jonathan Cameron wrote:

> From: Jonathan Cameron <jic23@xxxxxxxxx>
> 
> OV7670 soc-camera driver. Merge of drivers from Jonathan Corbet,
> Darius Augulis and Jonathan Cameron

Could you please, describe in more detail how you merged them?

However, I am not sure this is the best way to go. I think, a better 
approach would be to take a driver currently in the mainline, perhaps, 
the most feature-complete one if there are several of them there, convert 
it and its user(s) to v4l2-subdev, extend it with any features missing in 
it and present in other drivers, then switch users of all other ov7670 
drivers over to this one, and finally make it work with soc-camera. This 
way you get a series of smaller and reviewable patches, instead of a 
completely new driver, that reproduces a lot of existing code but has to 
be reviewed anew. How does this sound?

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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux