Hi Steve,
I don't know what your plan is, but when you want to mainline this it is
the gpu subsystem that needs to review it. I noticed it wasn't cross-posted
to the dri-devel mailinglist.
I am a bit worried about the amount of v4l2-specific stuff that is going
into drivers/gpu/ipu-v3. Do things like csc and csi really belong there
instead of under drivers/media?
Let me know if this was just preliminary code, or if this was intended to
be the final code. I suspect the former.
Regards,
Hans
On 06/26/2014 03:05 AM, Steve Longerbeam wrote:
> Hi Philip, Sascha,
>
> Here is a rebased set of IPU patches that prepares for video capture
> support. Video capture is not included in this set. I've addressed
> all your IPU-specific concerns from the previous patch set, the
> major ones being:
>
> - the IOMUXC control for CSI input selection has been removed. This
> should be part of a future CSI media entity driver.
>
> - the ipu-irt unit has been removed. Enabling the IRT module is
> folded into ipu-ic unit. The ipu-ic unit is also cleaned up a bit.
>
> - the ipu-csi APIs are consolidated/simplified.
>
> - added CSI and IC base offsets for i.MX51/i.MX53.
>
>
> Steve Longerbeam (28):
> ARM: dts: imx6qdl: Add ipu aliases
> gpu: ipu-v3: Add ipu_get_num()
> gpu: ipu-v3: Add functions to set CSI/IC source muxes
> gpu: ipu-v3: Rename and add IDMAC channels
> gpu: ipu-v3: Add units required for video capture
> gpu: ipu-v3: smfc: Move enable/disable to ipu-smfc.c
> gpu: ipu-v3: smfc: Convert to per-channel
> gpu: ipu-v3: smfc: Add ipu_smfc_set_watermark()
> gpu: ipu-v3: Add ipu_mbus_code_to_colorspace()
> gpu: ipu-v3: Add rotation mode conversion utilities
> gpu: ipu-v3: Add helper function checking if pixfmt is planar
> gpu: ipu-v3: Move IDMAC channel names to imx-ipu-v3.h
> gpu: ipu-v3: Add ipu_idmac_buffer_is_ready()
> gpu: ipu-v3: Add ipu_idmac_clear_buffer()
> gpu: ipu-v3: Add __ipu_idmac_reset_current_buffer()
> gpu: ipu-v3: Add ipu_stride_to_bytes()
> gpu: ipu-v3: Add ipu_idmac_enable_watermark()
> gpu: ipu-v3: Add ipu_idmac_lock_enable()
> gpu: ipu-v3: Add idmac channel linking support
> gpu: ipu-v3: Add ipu-cpmem unit
> staging: imx-drm: Convert to new ipu_cpmem API
> gpu: ipu-cpmem: Add ipu_cpmem_set_block_mode()
> gpu: ipu-cpmem: Add ipu_cpmem_set_axi_id()
> gpu: ipu-cpmem: Add ipu_cpmem_set_rotation()
> gpu: ipu-cpmem: Add second buffer support to ipu_cpmem_set_image()
> gpu: ipu-v3: Add more planar formats support
> gpu: ipu-cpmem: Add ipu_cpmem_dump()
> gpu: ipu-v3: Add ipu_dump()
>
> arch/arm/boot/dts/imx6q.dtsi | 1 +
> arch/arm/boot/dts/imx6qdl.dtsi | 1 +
> drivers/gpu/ipu-v3/Makefile | 3 +-
> drivers/gpu/ipu-v3/ipu-common.c | 1077
> +++++++++++++++++++--------------
> drivers/gpu/ipu-v3/ipu-cpmem.c | 817 +++++++++++++++++++++++++
> drivers/gpu/ipu-v3/ipu-csi.c | 701 +++++++++++++++++++++
> drivers/gpu/ipu-v3/ipu-ic.c | 812 +++++++++++++++++++++++++
> drivers/gpu/ipu-v3/ipu-prv.h | 103 +++-
> drivers/gpu/ipu-v3/ipu-smfc.c | 156 ++++-
> drivers/staging/imx-drm/ipuv3-plane.c | 16 +-
> include/video/imx-ipu-v3.h | 371 +++++++-----
> 11 files changed, 3389 insertions(+), 669 deletions(-)
> create mode 100644 drivers/gpu/ipu-v3/ipu-cpmem.c
> create mode 100644 drivers/gpu/ipu-v3/ipu-csi.c
> create mode 100644 drivers/gpu/ipu-v3/ipu-ic.c
>
--
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
