[PATCH v3] ARM: i.MX27 pcm970: Add camera support

[Mark Brown]

On 18 Aug 2010, at 22:13, Guennadi Liakhovetski <g.liakhovetski at gmx.de> wrote:

> On Wed, 18 Aug 2010, Mark Brown wrote:
> 
>> Again, this looks like you need to add regulator support to the camera
>> driver.
> 
> No, don't think so. This is a generic camera sensor, that can be attached 
> to any video-capable SoC or built into a USB webcam. It knows nothing 
> about where it's getting its power from.

With the regulator API it doesn't need to - that's all done externally by the machine layer. At the very least the regulator stuff should be going somewhere in the core driver where a machine only needs an assignment statement to use it since the code is going to be identical for all machines with regulators supplying the camera.