[PATCH v7 1/2] Add driver for Aptina (Micron) mt9p031 sensor.

This version fixes some cosmetic issues pointed out
by Guennadi.

Signed-off-by: Javier Martin <javier.martin@vista-silicon.com>

Fixes some cosmetic issues pointed out by Guennadi.

Signed-off-by: Javier Martin <javier.martin@vista-silicon.com>

Hi Javier,

I'm testing your patch on a 2.6.39 kernel. Here's what I get when loading the
omap3-isp module.

root@arago:~# modprobe omap3-isp
[ 159.523681] omap3isp omap3isp: Revision 15.0 found
[ 159.528991] omap-iommu omap-iommu.0: isp: version 1.1
[ 159.875701] omap_i2c omap_i2c.2: Arbitration lost
[ 159.881622] mt9p031 2-0048: Failed to reset the camera
[ 159.887054] omap3isp omap3isp: Failed to power on: -5
[ 159.892425] mt9p031 2-0048: Failed to power on device: -5
[ 159.898956] isp_register_subdev_group: Unable to register subdev mt9p031

Have you (or anyone else) seen that issue ?

I build in both statically to avoid that problem.

regards,

Koen

I used modules and it worked for me.

Thanks
Guennadi

Maybe u-boot version Laurent uses does not enable internal pull-up
resistors for i2c2 interface.
You could either use a different u-boot version or attach external
pull-up resistors to that interface.

http://dominion.thruhere.net/koen/angstrom/beagleboard/2.6.39/MLO
http://dominion.thruhere.net/koen/angstrom/beagleboard/2.6.39/u-boot.bin

The above MLO and uboot enable the pullups and work will all released versions of the beagleboard.

regards,

Koen

Hi Koen,

So, how is it going?

Are you finally accepting the patches for mainline?

Thank you,

Laurent,
have you been able to successfully test the driver?

I've found some issues and I don't know whether I should send a new
version or just wait for you to mainline the last one and send a patch
later.

Hi Javier,