From patchwork Tue Aug 28 07:34:16 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Kocialkowski X-Patchwork-Id: 10577977 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 19EE913B8 for ; Tue, 28 Aug 2018 07:36:04 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 061DC287ED for ; Tue, 28 Aug 2018 07:36:04 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id EE1A7294DE; Tue, 28 Aug 2018 07:36:03 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 3CBD1287ED for ; Tue, 28 Aug 2018 07:36:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727201AbeH1LZW (ORCPT ); Tue, 28 Aug 2018 07:25:22 -0400 Received: from mail.bootlin.com ([62.4.15.54]:33340 "EHLO mail.bootlin.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726439AbeH1LZV (ORCPT ); Tue, 28 Aug 2018 07:25:21 -0400 Received: by mail.bootlin.com (Postfix, from userid 110) id 43B66207EB; Tue, 28 Aug 2018 09:35:00 +0200 (CEST) Received: from localhost.localdomain (AAubervilliers-681-1-53-19.w90-88.abo.wanadoo.fr [90.88.170.19]) by mail.bootlin.com (Postfix) with ESMTPSA id B579D20787; Tue, 28 Aug 2018 09:34:59 +0200 (CEST) From: Paul Kocialkowski To: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devel@driverdev.osuosl.org Cc: Mauro Carvalho Chehab , Rob Herring , Mark Rutland , Maxime Ripard , Chen-Yu Tsai , Greg Kroah-Hartman , Paul Kocialkowski , Thomas Petazzoni , Randy Li , Hans Verkuil , Ezequiel Garcia , Tomasz Figa , Alexandre Courbot , Philipp Zabel , Laurent Pinchart , Sakari Ailus , linux-sunxi@googlegroups.com Subject: [PATCH v8 0/8] Cedrus driver for the Allwinner Video Engine, using media requests Date: Tue, 28 Aug 2018 09:34:16 +0200 Message-Id: <20180828073424.30247-1-paul.kocialkowski@bootlin.com> X-Mailer: git-send-email 2.18.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This is the eighth iteration of the updated Cedrus driver, that supports the Video Engine found on most Allwinner SoCs, starting with the A10. It was tested on the A13, A20, A33 and H3. The initial version of this driver[0] was originally written and submitted by Florent Revest using a previous version of the request API that is necessary to provide coherency between controls and the buffers they apply to. The driver was adapted to use the latest version of the media request API[1], as submitted by Hans Verkuil. Media request API support is a hard requirement for the Cedrus driver. The driver itself currently only supports MPEG2 and more codecs will be added eventually. The default output frame format provided by the Video Engine is a multi-planar tiled YUV format (based on NV12). A specific format is introduced in the V4L2 API to describe it. Starting with the A33, the Video Engine can also output untiled YUV formats. This implementation is based on the significant work that was conducted by various members of the linux-sunxi community for understanding and documenting the Video Engine's innards. In addition to the media requests API, the following series are required for Cedrus: * vicodec: the Virtual Codec driver * allwinner: a64: add SRAM controller / system control * SRAM patches from the Cedrus VPU driver series version 5 Changes since v7: * Rebased on the latest Request API; * Brought back comment about DRAM width; * Fixed building as a module; * Fixed copyright headers. Changes since v6: * Reworked MPEG2 controls to stick closer to the bitstream; * Updated controls documentation accordingly and added requested fixes; * Renamed tiled format to V4L2_PIX_FMT_SUNXI_TILED_NV12; * Added various minor driver fixes based on Hans' feedback; * Fixed dst frame alignment based on Jernej's feedback and tests; * Removed set bits for the disabled secondary output. Changes since v5: * Added MPEG2 quantization matrices definitions and support; * Cleaned up registers definitions; * Moved the driver to staging as requested; * Removed label and newline in device-tree sources; * Made it possible to build the driver for COMPILE_TEST; * Fixed various strict checkpatch warnings; * Used v4l2_m2m_register_media_controller and MEDIA_ENT_F_PROC_VIDEO_DECODER; * Moved capabilities to compatible-specific variants; * Removed overkill buffer checks in device_run; * Renamed from Sunxi-Cedrus to Cedrus. Changes since v4: * updated to version 16 of the media requests API; * added support for VPU-based untiling (starting with the A33); * added support for the H3, with SRAM support; * reworked SRAM support and associated compatibles; * improved failure paths; * added some MPEG2 input data validation; * reworked video/format functions to handle multiple formats; * removed in-driver buffer queues; * used a threaded irq instead of a workqueue; * merged various improvements and cleanups from Maxime; * renamed MPEG2_SLICE_HEADER to MPEG2_SLICE_PARAMS; * added prefixes to MPEG2 picture coding types; * used single-buffer allocations to ensure contiguous planes Changes since v3: * updated to version 15 of the media request API; * got rid of untested MPEG1 support; * added definitons for picture coding types; * added documentation about MPEG2 slice header fields; * added documentation about MPEG2 slice format; * added documentation about the MB32 NV12 format; * added MPEG2 slice header validation; * removed the assigned-clocks property; * reworked and fixed error paths; * harmonized debug prints, with v4l2 helpers when applicable; * checked the series through checkpatch; * switched to SPDX license headers; * renamed MPEG2 frame header to slice header for consistency and clarity; * removed A20 SRAM compatible from the driver's list. Changes since v2: * updated to version 13 of the media request API; * integrated various changes from Maxime Ripard; * reworked memory reservation to use CMA, dynamic allocation and allow DMABUF; * removed reserved memory binding since the CMA pool is the default one (and allow ENODEV in the driver, for that use case); * added SRAM controller support for the SRAM region used by the VE; * updated the device-tree bindings the for SRAM region; * added per-platform bindings; * added A13 support; * renamed VE node name and label; * fixed Florent's authorship for the MPEG2 headers; * added a MAINTAINERS entry. Changes since v1: * use the latest version of the request API for Hans Verkuil; * added media controller support and dependency * renamed v4l2 format to the more explicit V4L2_PIX_FMT_MB32_NV12; * reworked bindings documentation; * moved driver to drivers/media/platforms/sunxi/cedrus to pair with incoming CSI support ; * added a workqueue and lists to schedule buffer completion, since it cannot be done in interrupt context; * split mpeg2 support into a setup and a trigger function to avoid race condition; * split video-related ops to a dedicated sunxi_cedrus_video file; * cleaned up the included headers for each file; * used device PFN offset instead of subtracting PHYS_BASE; * used reset_control_reset instead of assert+deassert; * put the device in reset when removing driver; * changed dt bindings to use the last 96 Mib of the first 256 MiB of DRAM; * made it clear in the mpeg frame header structure that forward and backward indexes are used as reference frames for motion vectors; * lots of small cosmetic and consistency changes, including naming harmonization and headers text rework. Cheers! [0]: https://patchwork.kernel.org/patch/9299073/ [1]: https://git.linuxtv.org/hverkuil/media_tree.git/log/?h=reqv18-1 Paul Kocialkowski (8): media: v4l: Add definitions for MPEG2 slice format and metadata media: v4l: Add definition for the Sunxi tiled NV12 format dt-bindings: media: Document bindings for the Cedrus VPU driver media: platform: Add Cedrus VPU decoder driver ARM: dts: sun5i: Add Video Engine and reserved memory nodes ARM: dts: sun7i-a20: Add Video Engine and reserved memory nodes ARM: dts: sun8i-a33: Add Video Engine and reserved memory nodes ARM: dts: sun8i-h3: Add Video Engine and reserved memory nodes .../devicetree/bindings/media/cedrus.txt | 54 ++ .../media/uapi/v4l/extended-controls.rst | 177 ++++++ .../media/uapi/v4l/pixfmt-compressed.rst | 14 + .../media/uapi/v4l/pixfmt-reserved.rst | 15 +- .../media/uapi/v4l/vidioc-queryctrl.rst | 14 +- .../media/videodev2.h.rst.exceptions | 2 + MAINTAINERS | 7 + arch/arm/boot/dts/sun5i.dtsi | 26 + arch/arm/boot/dts/sun7i-a20.dtsi | 26 + arch/arm/boot/dts/sun8i-a33.dtsi | 26 + arch/arm/boot/dts/sun8i-h3.dtsi | 25 + drivers/media/v4l2-core/v4l2-ctrls.c | 63 ++ drivers/media/v4l2-core/v4l2-ioctl.c | 2 + drivers/staging/media/Kconfig | 2 + drivers/staging/media/Makefile | 1 + drivers/staging/media/sunxi/Kconfig | 15 + drivers/staging/media/sunxi/Makefile | 1 + drivers/staging/media/sunxi/cedrus/Kconfig | 14 + drivers/staging/media/sunxi/cedrus/Makefile | 3 + drivers/staging/media/sunxi/cedrus/cedrus.c | 420 +++++++++++++ drivers/staging/media/sunxi/cedrus/cedrus.h | 167 +++++ .../staging/media/sunxi/cedrus/cedrus_dec.c | 116 ++++ .../staging/media/sunxi/cedrus/cedrus_dec.h | 28 + .../staging/media/sunxi/cedrus/cedrus_hw.c | 322 ++++++++++ .../staging/media/sunxi/cedrus/cedrus_hw.h | 30 + .../staging/media/sunxi/cedrus/cedrus_mpeg2.c | 235 +++++++ .../staging/media/sunxi/cedrus/cedrus_regs.h | 233 +++++++ .../staging/media/sunxi/cedrus/cedrus_video.c | 574 ++++++++++++++++++ .../staging/media/sunxi/cedrus/cedrus_video.h | 32 + include/media/v4l2-ctrls.h | 18 +- include/uapi/linux/v4l2-controls.h | 65 ++ include/uapi/linux/videodev2.h | 6 + 32 files changed, 2723 insertions(+), 10 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/cedrus.txt create mode 100644 drivers/staging/media/sunxi/Kconfig create mode 100644 drivers/staging/media/sunxi/Makefile create mode 100644 drivers/staging/media/sunxi/cedrus/Kconfig create mode 100644 drivers/staging/media/sunxi/cedrus/Makefile create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus.c create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus.h create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_dec.c create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_dec.h create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_hw.c create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_hw.h create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_mpeg2.c create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_regs.h create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_video.c create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_video.h