Re: [RFC 0/5] Sensor drivers for OMAP3430SDP and LDP camera

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

 



On Tuesday 03 March 2009 21:44:12 Aguirre Rodriguez, Sergio Alberto wrote:
> This patch series depends on the following patches:
>
>  - "Add TWL4030 registers", posted by Tuukka Toivonen on March 2nd.
>  - "OMAP3 ISP and camera drivers" patch series, posted by Sakari Ailus on
>    March 3rd. (Please follow his instructions to pull from gitorious.org
> server)

Sergio, Sakari,

I'm feeling quite uncomfortable about this series with regards to the use of 
the v4l2-int API instead of v4l2_subdev. I know that it is on the TODO 
list, but every driver that is merged that uses v4l2-int will later mean 
extra work.

I and others have been working very hard to get the existing ioctl-based i2c 
modules converted in time for the 2.6.30 merge window. It looks like I'll 
be able to have it done in time (fingers crossed :-) ). So it is rather sad 
to see new modules that do not yet use it.

Right now the v4l2_device and v4l2_subdev framework is pretty basic and so 
the amount of work to do the conversion is still limited, but once I've 
finished my initial conversion I'll be adding lots more features, do 
cleanups, and generally improve the framework substantially. Any existing 
modules that use v4l2_device and v4l2_subdev will be updated by me. But I'm 
not going to do that for modules using v4l2-int, that will be the 
responsibility of the module's author when he converts it to v4l2_subdev. 
So the longer you wait, the more work that will be.

I *strongly* recommend that the conversion to the new framework is done 
first. I know it might delay inclusion of some drivers, but my expectation 
based on all the other conversions I've done until now is that it will 
actually simplify the drivers.

My experiences with it have been uniformly positive and it should be 
possible to use it as well with the ISP module or other logical 
sub-devices. There are lots of interesting possibilities there that you do 
not have with v4l2-int.

Best regards,

	Hans

> This has been tested with:
>  - SDP3430-VG5.0.1 with OMAP3430-ES3.1 daughter board upgrade.
>  - Camkit V3.0.1 with MT9P012 and OV3640 sensors
>  - LDP with OV3640 sensor
>
> Sergio Aguirre (5):
>   MT9P012: Add driver
>   DW9710: Add driver
>   OV3640: Add driver
>   OMAP3430SDP: Add support for Camera Kit v3
>   LDP: Add support for built-in camera
>
>  arch/arm/mach-omap2/Makefile                    |    6 +-
>  arch/arm/mach-omap2/board-3430sdp-camera.c      |  490 +++++
>  arch/arm/mach-omap2/board-3430sdp.c             |   42 +-
>  arch/arm/mach-omap2/board-ldp-camera.c          |  203 +++
>  arch/arm/mach-omap2/board-ldp.c                 |   17 +
>  arch/arm/plat-omap/include/mach/board-3430sdp.h |    1 +
>  arch/arm/plat-omap/include/mach/board-ldp.h     |    1 +
>  drivers/media/video/Kconfig                     |   31 +
>  drivers/media/video/Makefile                    |    3 +
>  drivers/media/video/dw9710.c                    |  548 ++++++
>  drivers/media/video/dw9710_priv.h               |   57 +
>  drivers/media/video/mt9p012.c                   | 1890
> +++++++++++++++++++ drivers/media/video/mt9p012_regs.h              |  
> 74 +
>  drivers/media/video/ov3640.c                    | 2202
> +++++++++++++++++++++++ drivers/media/video/ov3640_regs.h               |
>  600 ++++++
>  include/media/dw9710.h                          |   35 +
>  include/media/mt9p012.h                         |   37 +
>  include/media/ov3640.h                          |   31 +
>  18 files changed, 6265 insertions(+), 3 deletions(-)
>  create mode 100644 arch/arm/mach-omap2/board-3430sdp-camera.c
>  create mode 100644 arch/arm/mach-omap2/board-ldp-camera.c
>  create mode 100644 drivers/media/video/dw9710.c
>  create mode 100644 drivers/media/video/dw9710_priv.h
>  create mode 100644 drivers/media/video/mt9p012.c
>  create mode 100644 drivers/media/video/mt9p012_regs.h
>  create mode 100644 drivers/media/video/ov3640.c
>  create mode 100644 drivers/media/video/ov3640_regs.h
>  create mode 100644 include/media/dw9710.h
>  create mode 100644 include/media/mt9p012.h
>  create mode 100644 include/media/ov3640.h
>
> --
> 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



-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux