Hello,
The DPSUB is the DisplayPort subsystem, a set of hard IP cores found in
the ZynqMP family of SoCs. It combines a DisplayPort encoder, a video
blender with two input channels, and a DMA engine. The zynqmp_dpsub
driver exposes this as a DRM device with one CRTC and two planes.
In addition to those features, the DPSUB can interface with the
programmable logic (PL) found in the ZynqMP SoC. Each input to the video
blender can come from the PL instead of the DMA engine, and the blender
output can also be routed to the PL. This creates a very configurable
device that can accommodate lots of use cases, but it also makes it
difficult to model it as a DRM/KMS device.
This patch series implements initial support for live video inputs, by
restricting the supported use cases to a single live video input. In
that mode, the video blender is configured in pass-through mode, with
the whole DPSUB essentially operating as a DisplayPort encoder only. The
CRTC and plane functions are then implemented by IP cores in the PL.
To support this, the series start with patch 01/37 to model the
connections to the PL in DT using OF graph bindings. This fixes a
historical mistake that forgot to model the connection to the DP
connector in DT.
With that in place, patches 02/37 to 10/37 refactor the driver to turn
the DisplayPort encoder implementation, modelled as a DRM encoder, into
a DRM bridge. Please see individual patches for details. The rework is
internal only, simplifies the code by making use of the DRM bridge
connector helper, but doesn't bring any functional change.
Patches 11/37 to 31/37 continue refactoring of the driver, to cleanly
separate the DRM planes, CRTC, encoder and connector from the DRM bridge
implementation. The goal is to make the latter available to a DRM driver
for the PL display pipeline without registering any DRM device in the
DPSUB driver itself. Patches 32/37 to 35/37 implement this, reading
information about the connection to the PL from the device tree to
decide in which mode to operate.
Finally, patch 36/37 and 37/37 update the ZynqMP core and ZCU106A board
device tree files to create ports and connect the DPSUB to the
DisplayPort connector. I have tested the whole series without these two
patches to ensure that backward compatibility with older DT isn't
broken.
With this series applied, the DPSUB can be used as a DisplayPort encoder
by a PL display pipeline. A careful reviewer may ask me where drivers
for such a display pipelines are, and that would be a very good
question.
PL display pipelines are currently supported in the Xilinx downstream
kernel only, which is something I want to address next. That road will
be full of challenges, as in theory anything can be implemented in the
PL, including pipelines that connect cameras and displays together. If
anynoe is interested in discussing this topic, please let me know.
Laurent Pinchart (37):
dt-bindings: display: xlnx: zynqmp-dpsub: Add OF graph ports
drm: xlnx: zynqmp_dpsub: Switch to atomic encoder enable/disable
drm: xlnx: zynqmp_dpsub: Constify mode argument to function
drm: xlnx: zynqmp_dpsub: Create DRM bridge to model DP encoder
drm: xlnx: zynqmp_dpsub: Don't access connector in
zynqmp_dp_set_format()
drm: xlnx: zynqmp_dpsub: Move connector registration to bridge attach
drm: xlnx: zynqmp_dpsub: Move encoder to DPSUB core
drm: xlnx: zynqmp_dpsub: Attach to the next bridge
drm: xlnx: zynqmp_dpsub: Use DRM connector bridge helper
drm: xlnx: zynqmp_dpsub: Report HPD through the bridge
drm: xlnx: zynqmp_dpsub: Drop unused zynqmp_disp.event field
drm: xlnx: zynqmp_dpsub: Drop unused zynqmp_disp_format.bus_fmt field
drm: xlnx: zynqmp_dpsub: Don't pass CRTC to zynqmp_disp_setup_clock()
drm: xlnx: zynqmp_dpsub: Configure blender in zynqmp_disp_enable()
drm: xlnx: zynqmp_dpsub: Use local variable in
zynqmp_disp_layer_update()
drm: xlnx: zynqmp_dpsub: Pass format info to
zynqmp_disp_layer_set_format()
drm: xlnx: zynqmp_dpsub: Remplace hardcoded values with ARRAY_SIZE()
drm: xlnx: zynqmp_dpsub: Don't use drmm_kcalloc() for temporary data
drm: xlnx: zynqmp_dpsub: Move pclk from zynqmp_disp to zynqmp_dpsub
drm: xlnx: zynqmp_dpsub: Move audio clk from zynqmp_disp to
zynqmp_dpsub
drm: xlnx: zynqmp_dpsub: Move CRTC to zynqmp_dpsub structure
drm: xlnx: zynqmp_dpsub: Move planes to zynqmp_dpsub structure
drm: xlnx: zynqmp_dpsub: Move DRM/KMS initialization to separate file
drm: xlnx: zynqmp_dpsub: Move CRTC handling to zynqmp_kms.c
drm: xlnx: zynqmp_dpsub: Move planes handling to zynqmp_kms.c
drm: xlnx: zynqmp_dpsub: Register AUX bus at bridge attach time
drm: xlnx: zynqmp_dpsub: Move DP bridge init to zynqmp_dp_probe()
drm: xlnx: zynqmp_dpsub: Manage DP and DISP allocations manually
drm: xlnx: zynqmp_dpsub: Move all DRM init and cleanup to zynqmp_kms.c
drm: xlnx: zynqmp_dpsub: Decouple DRM device from zynqmp_dpsub
drm: xlnx: zynqmp_dpsub: Rename zynqmp_dpsub_handle_vblank with DRM
prefix
drm: xlnx: zynqmp_dpsub: Parse DT to find connected ports
drm: xlnx: zynqmp_dpsub: Allow configuration of layer mode
drm: xlnx: zynqmp_dpsub: Support operation without DMA engine
drm: xlnx: zynqmp_dpsub: Add support for live video input
arm64: dts: zynqmp: Add ports for the DisplayPort subsystem
arm64: dts: zynqmp: zcu106a: Describe DisplayPort connector
.../display/xlnx/xlnx,zynqmp-dpsub.yaml | 67 ++
.../boot/dts/xilinx/zynqmp-zcu106-revA.dts | 20 +
arch/arm64/boot/dts/xilinx/zynqmp.dtsi | 24 +
drivers/gpu/drm/xlnx/Makefile | 2 +-
drivers/gpu/drm/xlnx/zynqmp_disp.c | 647 ++++--------------
drivers/gpu/drm/xlnx/zynqmp_disp.h | 48 +-
drivers/gpu/drm/xlnx/zynqmp_dp.c | 482 +++++++------
drivers/gpu/drm/xlnx/zynqmp_dp.h | 4 +-
drivers/gpu/drm/xlnx/zynqmp_dpsub.c | 316 +++++----
drivers/gpu/drm/xlnx/zynqmp_dpsub.h | 46 +-
drivers/gpu/drm/xlnx/zynqmp_kms.c | 534 +++++++++++++++
drivers/gpu/drm/xlnx/zynqmp_kms.h | 46 ++
12 files changed, 1336 insertions(+), 900 deletions(-)
create mode 100644 drivers/gpu/drm/xlnx/zynqmp_kms.c
create mode 100644 drivers/gpu/drm/xlnx/zynqmp_kms.h
base-commit: 1c23f9e627a7b412978b4e852793c5e3c3efc555
--
Regards,
Laurent Pinchart
