Re: [PATCH 0/6] OMAP4 ISS driver
Hi Hans, On Thursday 03 October 2013 09:00:07 Hans Verkuil wrote: > On 10/03/2013 01:55 AM, Laurent Pinchart wrote: > > Hello, > > > > 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. > > > > 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. > > Thanks for the patch series! Much appreciated. > > For this patch series: > > Acked-by: Hans Verkuil > > I've posted a few comments for the first two patches (nothing jumped out to > me for the other patches) that you may want to address in a follow-up > patch, particularly the missing timestamp_type for vb2_queue will give a > lot of WARN_ON messages in the kernel log. Sure, I'll fix those. I wanted to send the code in with as little modifications as possible (I've only made sure that it would compile, as that's the main staging criteria nowadays), I'll tell send follow-up patches to clean the driver and fix problems before moving it to drivers/media/ > > Laurent Pinchart (1): > > v4l: omap4iss: Add support for OMAP4 camera interface - Build system > > > > 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 | 63 ++ > > 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 | 1477 + > > 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 | 1129 > > drivers/staging/media/omap4iss/iss_video.h | 201 > > include/media/omap4iss.h | 65 ++ > > 22 files changed, 8416 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 -- R
Re: [PATCH 0/6] OMAP4 ISS driver
Hi Laurent, On 10/03/2013 01:55 AM, Laurent Pinchart wrote: > Hello, > > 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. > > 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. Thanks for the patch series! Much appreciated. For this patch series: Acked-by: Hans Verkuil I've posted a few comments for the first two patches (nothing jumped out to me for the other patches) that you may want to address in a follow-up patch, particularly the missing timestamp_type for vb2_queue will give a lot of WARN_ON messages in the kernel log. Regards, Hans > > Laurent Pinchart (1): > v4l: omap4iss: Add support for OMAP4 camera interface - Build system > > 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 | 63 ++ > 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 | 1477 > ++ > 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 | 1129 > drivers/staging/media/omap4iss/iss_video.h | 201 > include/media/omap4iss.h | 65 ++ > 22 files changed, 8416 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 > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 0/6] OMAP4 ISS driver
Hello, 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. 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. Laurent Pinchart (1): v4l: omap4iss: Add support for OMAP4 camera interface - Build system 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 | 63 ++ 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 | 1477 ++ 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 | 1129 drivers/staging/media/omap4iss/iss_video.h | 201 include/media/omap4iss.h | 65 ++ 22 files changed, 8416 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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
