[E3-hacking] [RFC] [PATCH 0/6] Add camera support to the OMAP1 Amstrad Delta videophone
Guennadi Liakhovetski
g.liakhovetski at gmx.de
Tue Jul 20 10:49:54 BST 2010
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/
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
More information about the e3-hacking
mailing list