Re: [PATCH v3 00/14] Sunxi-Cedrus driver for the Allwinner Video Engine, using media requests

From: Paul Kocialkowski
Date: Mon May 07 2018 - 10:51:37 EST


[...]

On Mon, 2018-05-07 at 14:44 +0200, Paul Kocialkowski wrote:
> Remaining tasks:
> * cleaning up registers description and documenting the fields used;
> * removing the assigned-clocks property and setting the clock rate
> in the driver directly;
> * checking the series with checkpatch and fixing warnings;
> * documenting the MB32 NV12 format and adding it to v4l_fill_fmtdesc;
> * reworking and documenting the MPEG2 header, then adding it to
> v4l_fill_fmtdesc;
> * checking and fixing the error paths;
> * testing on more platforms.

Another item for the tasks list that is not yet in this revision:
* changing the id for V4L2_CTRL_TYPE_MPEG2_FRAME_HDR;

Paul

> Cheers!
>
> [0]: https://patchwork.kernel.org/patch/9299073/
> [1]: https://git.linuxtv.org/hverkuil/media_tree.git/log/?h=reqv13
>
> Florent Revest (1):
> media: v4l: Add definitions for MPEG2 frame format and header metadata
>
> Maxime Ripard (4):
> drivers: soc: sunxi: Add support for the C1 SRAM region
> ARM: sun5i: Add support for the C1 SRAM region with the SRAM
> controller
> ARM: sun7i-a20: Add support for the C1 SRAM region with the SRAM
> controller
> ARM: sun8i-a33: Add SRAM controller node and C1 SRAM region
>
> Paul Kocialkowski (9):
> drivers: soc: sunxi: Add dedicated compatibles for the A13, A20 and
> A33
> ARM: dts: sun5i: Use dedicated SRAM controller compatible
> ARM: dts: sun7i-a20: Use dedicated SRAM controller compatible
> media: v4l: Add definition for Allwinner's MB32-tiled NV12 format
> dt-bindings: media: Document bindings for the Sunxi-Cedrus VPU driver
> media: platform: Add Sunxi-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
>
> .../devicetree/bindings/media/sunxi-cedrus.txt | 58 +++
> MAINTAINERS | 7 +
> arch/arm/boot/dts/sun5i.dtsi | 47 +-
> arch/arm/boot/dts/sun7i-a20.dtsi | 47 +-
> arch/arm/boot/dts/sun8i-a33.dtsi | 54 +++
> drivers/media/platform/Kconfig | 15 +
> drivers/media/platform/Makefile | 1 +
> drivers/media/platform/sunxi/cedrus/Makefile | 4 +
> drivers/media/platform/sunxi/cedrus/sunxi_cedrus.c | 333 ++++++++++++++
> .../platform/sunxi/cedrus/sunxi_cedrus_common.h | 128 ++++++
> .../media/platform/sunxi/cedrus/sunxi_cedrus_dec.c | 188 ++++++++
> .../media/platform/sunxi/cedrus/sunxi_cedrus_dec.h | 35 ++
> .../media/platform/sunxi/cedrus/sunxi_cedrus_hw.c | 240 ++++++++++
> .../media/platform/sunxi/cedrus/sunxi_cedrus_hw.h | 37 ++
> .../platform/sunxi/cedrus/sunxi_cedrus_mpeg2.c | 160 +++++++
> .../platform/sunxi/cedrus/sunxi_cedrus_mpeg2.h | 33 ++
> .../platform/sunxi/cedrus/sunxi_cedrus_regs.h | 175 +++++++
> .../platform/sunxi/cedrus/sunxi_cedrus_video.c | 505 +++++++++++++++++++++
> .../platform/sunxi/cedrus/sunxi_cedrus_video.h | 31 ++
> drivers/media/v4l2-core/v4l2-ctrls.c | 10 +
> drivers/media/v4l2-core/v4l2-ioctl.c | 1 +
> drivers/soc/sunxi/sunxi_sram.c | 13 +
> include/uapi/linux/v4l2-controls.h | 26 ++
> include/uapi/linux/videodev2.h | 4 +
> 24 files changed, 2150 insertions(+), 2 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/media/sunxi-cedrus.txt
> create mode 100644 drivers/media/platform/sunxi/cedrus/Makefile
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus.c
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_common.h
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_dec.c
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_dec.h
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_hw.c
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_hw.h
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.c
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.h
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_regs.h
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_video.c
> create mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_video.h
>
--
Paul Kocialkowski, Bootlin (formerly Free Electrons)
Embedded Linux and kernel engineering
https://bootlin.com

Attachment: signature.asc
Description: This is a digitally signed message part