Hi Steve!
On 06/07/2014 11:56 PM, Steve Longerbeam wrote:
> Hi all,
>
> This patch set adds video capture support for the Freescale i.MX6 SOC.
>
> It is a from-scratch standardized driver that works with community
> v4l2 utilities, such as v4l2-ctl, v4l2-cap, and the v4l2src gstreamer
> plugin. It uses the latest v4l2 interfaces (subdev, videobuf2).
> Please see Documentation/video4linux/mx6_camera.txt for it's full list
> of features!
>
> The first 38 patches:
>
> - prepare the ipu-v3 driver for video capture support. The current driver
> contains only video display functionality to support the imx DRM drivers.
> At some point ipu-v3 should be moved out from under staging/imx-drm since
> it will no longer only support DRM.
>
> - Adds the device tree nodes and OF graph bindings for video capture support
> on sabrelite, sabresd, and sabreauto reference platforms.
>
> The new i.MX6 capture host interface driver is at patch 39.
>
> To support the sensors found on the sabrelite, sabresd, and sabreauto,
> three patches add sensor subdev's for parallel OV5642, MIPI CSI-2 OV5640,
> and the ADV7180 decoder chip, beginning at patch 40.
>
> There is an existing adv7180 subdev driver under drivers/media/i2c, but
> it needs some extra functionality to work on the sabreauto. It will need
> OF graph bindings support and gpio for a power-on pin on the sabreauto.
> It would also need to send a new subdev notification to take advantage
> of decoder status change handling provided by the host driver. This
> feature makes it possible to correctly handle "hot" (while streaming)
> signal lock/unlock and autodetected video standard changes.
A new V4L2_EVENT_SOURCE_CHANGE event has just been added for that.
> Usage notes are found in Documentation/video4linux/mx6_camera.txt for the
> above three reference platforms.
>
> The driver source is under drivers/staging/media/imx6/capture/.
Thank you for this patch series! Much appreciated that this hardware is
finally going to supported with a proper driver.
I did a quick scan of the driver and I noticed a few things that need
to be fixed: instead of implementing g/s_crop, implement g/s_selection:
new drivers should implement the selection API and they will get the crop
API for free.
You should use the vb2 helper functions (vb2_fop_*, vb2_ioctl_*) unless
there is a good reason not to do it. Those functions should simplify the
code and they give you proper 'streaming ownership'. See also the example
code Documentation/video4linux/v4l2-pci-skeleton.c.
Finally you should run the v4l2-compliance test tool and fix any failures.
That tool is part of git://linuxtv.org/v4l-utils.git. Always compile from
that repository to be sure you use the latest code.
Test first with 'v4l2-compliance'. When all issues are fixed, then test
with 'v4l2-compliance -s' to test actual streaming behavior as well.
When you post v2 of this patch series I want to see the output of
'v4l2-compliance -s'! New drivers should pass v4l2-compliance. However,
this is a staging driver so I won't be that strict, but it seems to be
the intention that this driver will become a mainline driver, so I would
recommend to fix any issues now rather than later.
If you have questions about v4l2-compliance it it might be easiest to
set up an irc session where we go through them. In that case mail me
so we can set up a time and date to do that. I'm in timezone UTC+2.
Regards,
Hans
>
>
> Steve Longerbeam (43):
> imx-drm: ipu-v3: Move imx-ipu-v3.h to include/linux/platform_data/
> ARM: dts: imx6qdl: Add ipu aliases
> imx-drm: ipu-v3: Add ipu_get_num()
> imx-drm: ipu-v3: Add solo/dual-lite IPU device type
> imx-drm: ipu-v3: Map IOMUXC registers
> imx-drm: ipu-v3: Add functions to set CSI/IC source muxes
> imx-drm: ipu-v3: Rename and add IDMAC channels
> imx-drm: ipu-v3: Add units required for video capture
> imx-drm: ipu-v3: Add ipu_mbus_code_to_colorspace()
> imx-drm: ipu-v3: Add rotation mode conversion utilities
> imx-drm: ipu-v3: Add helper function checking if pixfmt is planar
> imx-drm: ipu-v3: Move IDMAC channel names to imx-ipu-v3.h
> imx-drm: ipu-v3: Add ipu_idmac_buffer_is_ready()
> imx-drm: ipu-v3: Add ipu_idmac_clear_buffer()
> imx-drm: ipu-v3: Add ipu_idmac_current_buffer()
> imx-drm: ipu-v3: Add __ipu_idmac_reset_current_buffer()
> imx-drm: ipu-v3: Add ipu_stride_to_bytes()
> imx-drm: ipu-v3: Add ipu_idmac_enable_watermark()
> imx-drm: ipu-v3: Add ipu_idmac_lock_enable()
> imx-drm: ipu-v3: Add idmac channel linking support
> imx-drm: ipu-v3: Add ipu_bits_per_pixel()
> imx-drm: ipu-v3: Add ipu-cpmem unit
> imx-drm: ipu-cpmem: Add ipu_cpmem_set_block_mode()
> imx-drm: ipu-cpmem: Add ipu_cpmem_set_axi_id()
> imx-drm: ipu-cpmem: Add ipu_cpmem_set_rotation()
> imx-drm: ipu-cpmem: Add second buffer support to ipu_cpmem_set_image()
> imx-drm: ipu-v3: Add more planar formats support
> imx-drm: ipu-cpmem: Add ipu_cpmem_dump()
> imx-drm: ipu-v3: Add ipu_dump()
> ARM: dts: imx6: add pin groups for imx6q/dl for IPU1 CSI0
> ARM: dts: imx6qdl: Flesh out MIPI CSI2 receiver node
> ARM: dts: imx: sabrelite: add video capture ports and endpoints
> ARM: dts: imx6-sabresd: add video capture ports and endpoints
> ARM: dts: imx6-sabreauto: add video capture ports and endpoints
> ARM: dts: imx6qdl: Add simple-bus to ipu compatibility
> gpio: pca953x: Add reset-gpios property
> ARM: imx6q: clk: Add video 27m clock
> media: imx6: Add device tree binding documentation
> media: Add new camera interface driver for i.MX6
> media: imx6: Add support for MIPI CSI-2 OV5640
> media: imx6: Add support for Parallel OV5642
> media: imx6: Add support for ADV7180 Video Decoder
> ARM: imx_v6_v7_defconfig: Enable video4linux drivers
>
> .../devicetree/bindings/clock/imx6q-clock.txt | 1 +
> Documentation/devicetree/bindings/media/imx6.txt | 433 ++
> .../bindings/staging/imx-drm/fsl-imx-drm.txt | 6 +-
> .../devicetree/bindings/vendor-prefixes.txt | 1 +
> Documentation/video4linux/mx6_camera.txt | 188 +
> arch/arm/boot/dts/imx6q.dtsi | 3 +-
> arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 149 +
> arch/arm/boot/dts/imx6qdl-sabrelite.dtsi | 91 +
> arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 116 +
> arch/arm/boot/dts/imx6qdl.dtsi | 62 +-
> arch/arm/configs/imx_v6_v7_defconfig | 4 +
> arch/arm/mach-imx/clk-imx6q.c | 3 +-
> drivers/gpio/gpio-pca953x.c | 26 +
> drivers/staging/imx-drm/imx-hdmi.c | 2 +-
> drivers/staging/imx-drm/imx-tve.c | 2 +-
> drivers/staging/imx-drm/ipu-v3/Makefile | 3 +-
> drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h | 326 --
> drivers/staging/imx-drm/ipu-v3/ipu-common.c | 1151 ++++--
> drivers/staging/imx-drm/ipu-v3/ipu-cpmem.c | 814 ++++
> drivers/staging/imx-drm/ipu-v3/ipu-csi.c | 821 ++++
> drivers/staging/imx-drm/ipu-v3/ipu-dc.c | 2 +-
> drivers/staging/imx-drm/ipu-v3/ipu-di.c | 2 +-
> drivers/staging/imx-drm/ipu-v3/ipu-dmfc.c | 2 +-
> drivers/staging/imx-drm/ipu-v3/ipu-dp.c | 2 +-
> drivers/staging/imx-drm/ipu-v3/ipu-ic.c | 835 ++++
> drivers/staging/imx-drm/ipu-v3/ipu-irt.c | 103 +
> drivers/staging/imx-drm/ipu-v3/ipu-prv.h | 126 +-
> drivers/staging/imx-drm/ipu-v3/ipu-smfc.c | 348 ++
> drivers/staging/imx-drm/ipuv3-crtc.c | 2 +-
> drivers/staging/imx-drm/ipuv3-plane.c | 18 +-
> drivers/staging/media/Kconfig | 2 +
> drivers/staging/media/Makefile | 1 +
> drivers/staging/media/imx6/Kconfig | 25 +
> drivers/staging/media/imx6/Makefile | 1 +
> drivers/staging/media/imx6/capture/Kconfig | 33 +
> drivers/staging/media/imx6/capture/Makefile | 7 +
> drivers/staging/media/imx6/capture/adv7180.c | 1298 ++++++
> drivers/staging/media/imx6/capture/mipi-csi2.c | 322 ++
> drivers/staging/media/imx6/capture/mx6-camif.c | 2235 ++++++++++
> drivers/staging/media/imx6/capture/mx6-camif.h | 197 +
> drivers/staging/media/imx6/capture/mx6-encode.c | 775 ++++
> drivers/staging/media/imx6/capture/mx6-preview.c | 748 ++++
> drivers/staging/media/imx6/capture/ov5640-mipi.c | 2158 ++++++++++
> drivers/staging/media/imx6/capture/ov5642.c | 4258
> ++++++++++++++++++++
> include/linux/platform_data/imx-ipu-v3.h | 425 ++
> include/media/imx6.h | 18 +
> 46 files changed, 17340 insertions(+), 805 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/media/imx6.txt
> create mode 100644 Documentation/video4linux/mx6_camera.txt
> delete mode 100644 drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h
> create mode 100644 drivers/staging/imx-drm/ipu-v3/ipu-cpmem.c
> create mode 100644 drivers/staging/imx-drm/ipu-v3/ipu-csi.c
> create mode 100644 drivers/staging/imx-drm/ipu-v3/ipu-ic.c
> create mode 100644 drivers/staging/imx-drm/ipu-v3/ipu-irt.c
> create mode 100644 drivers/staging/imx-drm/ipu-v3/ipu-smfc.c
> create mode 100644 drivers/staging/media/imx6/Kconfig
> create mode 100644 drivers/staging/media/imx6/Makefile
> create mode 100644 drivers/staging/media/imx6/capture/Kconfig
> create mode 100644 drivers/staging/media/imx6/capture/Makefile
> create mode 100644 drivers/staging/media/imx6/capture/adv7180.c
> create mode 100644 drivers/staging/media/imx6/capture/mipi-csi2.c
> create mode 100644 drivers/staging/media/imx6/capture/mx6-camif.c
> create mode 100644 drivers/staging/media/imx6/capture/mx6-camif.h
> create mode 100644 drivers/staging/media/imx6/capture/mx6-encode.c
> create mode 100644 drivers/staging/media/imx6/capture/mx6-preview.c
> create mode 100644 drivers/staging/media/imx6/capture/ov5640-mipi.c
> create mode 100644 drivers/staging/media/imx6/capture/ov5642.c
> create mode 100644 include/linux/platform_data/imx-ipu-v3.h
> create mode 100644 include/media/imx6.h
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
