Hello,
This patch series adds support for CRC calculation to the rcar-du-drm driver.
CRC calculation is supported starting at the Renesas R-Car Gen3 SoCs, as
earlier versions don't have the necessary hardware. On Gen3 SoCs, the CRC is
computed by the DISCOM module part of the VSP-D and VSP-DL.
The DISCOM is interfaced to the VSP through the UIF glue and appears as a VSP
entity with a sink pad and a source pad.
The series starts with two fixes. Patch 1/9 fixes a display stall when
requesting more than two planes from the VGA output. It is a temporary fix to
avoid complete stalls during unit testing, and a better fix will follow but
will take longer to develop. Patch 2/9 then fixes debug messages that confused
me when I was debugging the rest of this patch series.
The series then continues with cleanup and refactoring. Patches 3/9 and 4/9
prepare for DISCOM and UIF support by extending generic code to make it usable
for the UIF. Patch 5/9 documents a structure that will receive new fields.
Patch 6/9 then extends the API exposed by the VSP driver to the DU driver to
support CRC computation configuration and reporting. The patch unfortunately
needs to touch both the VSP and DU drivers. The whole series will need to be
merged through a single tree, I will check when it will be time to submit a
pull request which option would generate the least conflicts.
Patch 7/9 adds support for the DISCOM and UIF in the VSP driver, patch 8/9
integrates it in the DRM pipeline, and patch 9/9 finally implements the CRC
API in the DU driver to expose CRC computation to userspace.
The hardware supports computing the CRC at any arbitrary point in the
pipeline on a configurable window of the frame. This patch series supports CRC
computation on input planes or pipeline output, but on the full frame only.
Support for CRC window configuration can be added later if needed but will
require extending the userspace API, as the DRM/KMS CRC API doesn't support
this feature.
Note that exposing the DISCOM and UIF though the V4L2 API isn't supported as
the module is only found in VSP-D and VSP-DL instances that are not exposed
through V4L2. It is possible to expose those instances through V4L2 with a
small modification to the driver for testing purpose. If the need arises to
test DISCOM and UIF with such an out-of-tree patch, support for CRC reporting
through a V4L2 control can be added later without affecting how CRC is exposed
through the DRM/KMS API.
The patches are based on top of a merge of the Linux media master branch and
the pending DU patches. For convenience they are available at
git://linuxtv.org/pinchartl/media.git vsp1-discom-v1-20171203
The code has been tested through the kms-test-crc.py script part of the DU
test suite available at
git://git.ideasonboard.com/renesas/kms-tests.git discom
Laurent Pinchart (9):
v4l: vsp1: Fix display stalls when requesting too many inputs
v4l: vsp1: Print the correct blending unit name in debug messages
v4l: vsp1: Share the CLU, LIF and LUT set_fmt pad operation code
v4l: vsp1: Reset the crop and compose rectangles in the set_fmt helper
v4l: vsp1: Document the vsp1_du_atomic_config structure
v4l: vsp1: Extend the DU API to support CRC computation
v4l: vsp1: Add support for the DISCOM entity
v4l: vsp1: Integrate DISCOM in display pipeline
drm: rcar-du: Add support for CRC computation
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 148 +++++++++++++++-
drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 19 +++
drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 13 +-
drivers/media/platform/vsp1/Makefile | 2 +-
drivers/media/platform/vsp1/vsp1.h | 4 +
drivers/media/platform/vsp1/vsp1_clu.c | 65 ++-----
drivers/media/platform/vsp1/vsp1_drm.c | 138 +++++++++++++--
drivers/media/platform/vsp1/vsp1_drm.h | 14 +-
drivers/media/platform/vsp1/vsp1_drv.c | 20 +++
drivers/media/platform/vsp1/vsp1_entity.c | 97 +++++++++++
drivers/media/platform/vsp1/vsp1_entity.h | 7 +
drivers/media/platform/vsp1/vsp1_histo.c | 59 +------
drivers/media/platform/vsp1/vsp1_lif.c | 65 ++-----
drivers/media/platform/vsp1/vsp1_lut.c | 65 ++-----
drivers/media/platform/vsp1/vsp1_regs.h | 41 +++++
drivers/media/platform/vsp1/vsp1_uif.c | 275 ++++++++++++++++++++++++++++++
drivers/media/platform/vsp1/vsp1_uif.h | 36 ++++
include/media/vsp1.h | 39 ++++-
18 files changed, 870 insertions(+), 237 deletions(-)
create mode 100644 drivers/media/platform/vsp1/vsp1_uif.c
create mode 100644 drivers/media/platform/vsp1/vsp1_uif.h
--
Regards,
Laurent Pinchart
