Hi Janusz
On Sun, 18 Jul 2010, Janusz Krzysztofik wrote:
This series consists of the following patches:
1/6 SoC Camera: add driver for OMAP1 camera interface 2/6 OMAP1: Add support for SoC camera interface 3/6 SoC Camera: add driver for OV6650 sensor 4/6 SoC Camera: add support for g_parm / s_parm operations 5/6 OMAP1: Amstrad Delta: add support for on-board camera 6/6 OMAP1: Amstrad Delta: add camera controlled LEDS trigger
It is an interesting decision to use soc-camera for an OMAP SoC, as you most probably know OMAP3 and OMAP2 camera drivers do not use soc-camera. I certainly do not want to discourage you from using soc-camera, just don't want you to go the wrong way and then regret it or spend time re-designing your driver. Have you had specific reasons for this design? Is OMAP1 so different from 2 (and 3)? In any case - thanks for the patches, if you do insist on going this path (;)) I'll review them and get back to you after that. Beware, it might be difficult to finish the review process in time for 2.6.36...
Thanks Guennadi
arch/arm/mach-omap1/board-ams-delta.c | 69 + arch/arm/mach-omap1/devices.c | 43 arch/arm/mach-omap1/include/mach/camera.h | 8 drivers/media/video/Kconfig | 20 drivers/media/video/Makefile | 2 drivers/media/video/omap1_camera.c | 1656 ++++++++++++++++++++++++++++++ drivers/media/video/ov6650.c | 1336 ++++++++++++++++++++++++ drivers/media/video/soc_camera.c | 18 include/media/omap1_camera.h | 16 include/media/v4l2-chip-ident.h | 1 10 files changed, 3169 insertions(+)
Thanks, Janusz
--- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/