Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751273AbdGPETp (ORCPT ); Sun, 16 Jul 2017 00:19:45 -0400 Received: from mail-qt0-f193.google.com ([209.85.216.193]:36088 "EHLO mail-qt0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750829AbdGPETm (ORCPT ); Sun, 16 Jul 2017 00:19:42 -0400 MIME-Version: 1.0 In-Reply-To: <1500137353.2353.1.camel@ndufresne.ca> References: <1500101920-24039-1-git-send-email-jacob-chen@iotwrt.com> <1500101920-24039-3-git-send-email-jacob-chen@iotwrt.com> <2363665.x6z9MR1vqI@avalon> <1500137353.2353.1.camel@ndufresne.ca> From: Jacob Chen Date: Sun, 16 Jul 2017 12:19:41 +0800 Message-ID: Subject: Re: [PATCH v2 2/6] [media] rockchip/rga: v4l2 m2m support To: Personnel Cc: Laurent Pinchart , "open list:ARM/Rockchip SoC..." , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, Heiko Stuebner , robh+dt@kernel.org, Mauro Carvalho Chehab , Linux Media Mailing List , laurent.pinchart+renesas@ideasonboard.com, Hans Verkuil , s.nawrocki@samsung.com, Tomasz Figa Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v6G4JmIZ020344 Content-Length: 4249 Lines: 101 Hi all, 2017-07-16 0:49 GMT+08:00 Personnel : > Le samedi 15 juillet 2017 à 12:42 +0300, Laurent Pinchart a écrit : >> Hi Jacob, >> >> Thank you for the patch. >> >> On Saturday 15 Jul 2017 14:58:36 Jacob Chen wrote: >> > Rockchip RGA is a separate 2D raster graphic acceleration unit. It >> > accelerates 2D graphics operations, such as point/line drawing, image >> > scaling, rotation, BitBLT, alpha blending and image blur/sharpness. >> > >> > The drvier is mostly based on s5p-g2d v4l2 m2m driver. >> > And supports various operations from the rendering pipeline. >> > - copy >> > - fast solid color fill >> > - rotation >> > - flip >> > - alpha blending >> >> I notice that you don't support the drawing operations. How do you plan to >> support them later through the V4L2 M2M API ? I hate stating the obvious, but >> wouldn't the DRM API be better fit for a graphic accelerator ? > > It could fit, maybe, but it really lacks some framework. Also, DRM is > not really meant for M2M operation, and it's also not great for multi- > process. Until recently, there was competing drivers for Exynos, both > implemented in V4L2 and DRM, for similar rational, all DRM ones are > being deprecated/removed. > > I think 2D blitters in V4L2 are fine, but they terribly lack something > to differentiate them from converters/scalers when looking up the HW > list. Could be as simple as a capability flag, if I can suggest. For > the reference, the 2D blitter on IMX6 has been used to implement a live > video mixer in GStreamer. > > https://bugzilla.gnome.org/show_bug.cgi?id=772766 > We have write a drm RGA driver. https://patchwork.kernel.org/patch/8630841/ Here are the reasons that why i rewrite it to V4l2 M2M. 1. V4l2 have a better buffer framework. If it use DRM-GEM to handle buffers, there will be much redundant cache flush, and we have to add much hack code to workaround. 2. This driver will be used in rockchip linux project. We mostly use it to scale/colorconvert/rotate/mix video/camera stream. A V4L2 M2M drvier can be directly used in gstreamer. The disadvantages of V4l2 M2M API is that it's not stateless. It's inconvenient if user change size frequently, but it's OK, we have not yet need this and I think it's possible to extend. ; ) >> >> Additionally, V4L2 M2M has one source and one destination. How do you >> implement alpha blending in that case, which by definition requires at least >> two sources ? > > This type of HW only do in-place blits. When using such a node, the > buffer queued on the V4L2_CAPTURE contains the destination image, and > the buffer queued on the V4L2_OUTPUT is the source image. > Yep. >> >> > The code in rga-hw.c is used to configure regs accroding to operations. >> > >> > The code in rga-buf.c is used to create private mmu table for RGA. >> > The tables is stored in a list, and be removed when buffer is cleanup. >> >> Looking at the implementation it seems to be a scatter-gather list, not an >> MMU. Is that right ? Does the hardware documentation refer to it as an MMU ? >> It's a 1-level MMU... We use it like a scatter-gather list, It's also the reason why we don't use RGA with DRM API. >> > Signed-off-by: Jacob Chen >> > --- >> > drivers/media/platform/Kconfig | 11 + >> > drivers/media/platform/Makefile | 2 + >> > drivers/media/platform/rockchip-rga/Makefile | 3 + >> > drivers/media/platform/rockchip-rga/rga-buf.c | 122 ++++ >> > drivers/media/platform/rockchip-rga/rga-hw.c | 652 ++++++++++++++++++ >> > drivers/media/platform/rockchip-rga/rga-hw.h | 437 ++++++++++++ >> > drivers/media/platform/rockchip-rga/rga.c | 958 +++++++++++++++++++++++ >> > drivers/media/platform/rockchip-rga/rga.h | 111 +++ >> > 8 files changed, 2296 insertions(+) >> > create mode 100644 drivers/media/platform/rockchip-rga/Makefile >> > create mode 100644 drivers/media/platform/rockchip-rga/rga-buf.c >> > create mode 100644 drivers/media/platform/rockchip-rga/rga-hw.c >> > create mode 100644 drivers/media/platform/rockchip-rga/rga-hw.h >> > create mode 100644 drivers/media/platform/rockchip-rga/rga.c >> > create mode 100644 drivers/media/platform/rockchip-rga/rga.h >> >>