Re: [PATCH v2 00/18] OMAP4 ISS driver

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Mauro,

On Monday 04 November 2013 01:06:25 Laurent Pinchart wrote:
> Hello,
> 
> This is the second version of the OMAP4 ISS driver patches for inclusion in
> the mainline kernel. I've addressed most of the comments received on the
> first version (some of them are still being discussed) in additional
> patches, except for the file path updates in the documentation that have
> been squashed with patch 01/18.
> 
> The OMAP4 ISS driver has lived out of tree for more than two years now. This
> situation is both sad and resource-wasting, as the driver has been used
> (and thus) hacked since then with nowhere to send patches to. Time has come
> to fix the problem.
> 
> As the code is mostly, but not quite ready for prime time, I'd like to
> request its addition to drivers/staging/. I've added a (pretty small) TODO
> file and I commit to cleaning up the code and get it to drivers/media/
> where it belongs.
> 
> I've split the driver in six patches to avoid getting caught in vger's size
> and to make review slightly easier. Sergio Aguirre is the driver author
> (huge thanks for that!), I've thus kept his authorship on patches 1/6 to
> 5/6. Beside minimal changes to make the code compile on v3.12 and updates
> the file paths in the documentation I've kept Sergio's code unmodified.
> 
> I don't have much else to add here, let's get this beast to mainline and
> allow other developers to use the driver and contribute patches. Given that
> v3.12 has just been released I'm fine with pushing this series back to
> v3.13.

This should of course have read v3.14.

> Laurent Pinchart (13):
>   v4l: omap4iss: Add support for OMAP4 camera interface - Build system
>   v4l: omap4iss: Don't use v4l2_g_ext_ctrls() internally
>   v4l: omap4iss: Move common code out of switch...case
>   v4l: omap4iss: Report device caps in response to VIDIOC_QUERYCAP
>   v4l: omap4iss: Remove iss_video streaming field
>   v4l: omap4iss: Set the vb2 timestamp type
>   v4l: omap4iss: Remove duplicate video_is_registered() check
>   v4l: omap4iss: Remove unneeded status variable
>   v4l: omap4iss: Replace udelay/msleep with usleep_range
>   v4l: omap4iss: Make omap4iss_isp_subclk_(en|dis)able() functions void
>   v4l: omap4iss: Make loop counters unsigned where appropriate
>   v4l: omap4iss: Don't initialize fields to 0 manually
>   v4l: omap4iss: Simplify error paths
> 
> Sergio Aguirre (5):
>   v4l: omap4iss: Add support for OMAP4 camera interface - Core
>   v4l: omap4iss: Add support for OMAP4 camera interface - Video devices
>   v4l: omap4iss: Add support for OMAP4 camera interface - CSI receivers
>   v4l: omap4iss: Add support for OMAP4 camera interface - IPIPE(IF)
>   v4l: omap4iss: Add support for OMAP4 camera interface - Resizer
> 
>  Documentation/video4linux/omap4_camera.txt   |   60 ++
>  drivers/staging/media/Kconfig                |    2 +
>  drivers/staging/media/Makefile               |    1 +
>  drivers/staging/media/omap4iss/Kconfig       |   12 +
>  drivers/staging/media/omap4iss/Makefile      |    6 +
>  drivers/staging/media/omap4iss/TODO          |    4 +
>  drivers/staging/media/omap4iss/iss.c         | 1462 +++++++++++++++++++++++
>  drivers/staging/media/omap4iss/iss.h         |  153 +++
>  drivers/staging/media/omap4iss/iss_csi2.c    | 1368 +++++++++++++++++++++++
>  drivers/staging/media/omap4iss/iss_csi2.h    |  156 +++
>  drivers/staging/media/omap4iss/iss_csiphy.c  |  278 +++++
>  drivers/staging/media/omap4iss/iss_csiphy.h  |   51 +
>  drivers/staging/media/omap4iss/iss_ipipe.c   |  581 ++++++++++
>  drivers/staging/media/omap4iss/iss_ipipe.h   |   67 ++
>  drivers/staging/media/omap4iss/iss_ipipeif.c |  847 +++++++++++++++
>  drivers/staging/media/omap4iss/iss_ipipeif.h |   92 ++
>  drivers/staging/media/omap4iss/iss_regs.h    |  883 ++++++++++++++++
>  drivers/staging/media/omap4iss/iss_resizer.c |  905 ++++++++++++++++
>  drivers/staging/media/omap4iss/iss_resizer.h |   75 ++
>  drivers/staging/media/omap4iss/iss_video.c   | 1124 ++++++++++++++++++++
>  drivers/staging/media/omap4iss/iss_video.h   |  198 ++++
>  include/media/omap4iss.h                     |   65 ++
>  22 files changed, 8390 insertions(+)
>  create mode 100644 Documentation/video4linux/omap4_camera.txt
>  create mode 100644 drivers/staging/media/omap4iss/Kconfig
>  create mode 100644 drivers/staging/media/omap4iss/Makefile
>  create mode 100644 drivers/staging/media/omap4iss/TODO
>  create mode 100644 drivers/staging/media/omap4iss/iss.c
>  create mode 100644 drivers/staging/media/omap4iss/iss.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_csi2.c
>  create mode 100644 drivers/staging/media/omap4iss/iss_csi2.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_csiphy.c
>  create mode 100644 drivers/staging/media/omap4iss/iss_csiphy.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_ipipe.c
>  create mode 100644 drivers/staging/media/omap4iss/iss_ipipe.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_ipipeif.c
>  create mode 100644 drivers/staging/media/omap4iss/iss_ipipeif.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_regs.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_resizer.c
>  create mode 100644 drivers/staging/media/omap4iss/iss_resizer.h
>  create mode 100644 drivers/staging/media/omap4iss/iss_video.c
>  create mode 100644 drivers/staging/media/omap4iss/iss_video.h
>  create mode 100644 include/media/omap4iss.h
-- 
Regards,

Laurent Pinchart