On Thu, Jan 04, 2018 at 06:05:49PM -0800, Hyun Kwon wrote:
Hi,
This patchset adds the DRM KMS driver for Xilinx ZynqMP DisplayPort subsystem. The Xilinx ZynqMP SoC has a hardened full display pipeline which supports blending of up to 2 planes, and the encoder is DisplayPort v1.2 compatible.
This series mainly includes 2 sets: Xilinx DRM KMS (patch 1/10 - 5/10) and ZynqMP DP subsystem drivers (patch 6/10 - 10/10).
The Xilinx DRM KMS is intended as a common layer shared across other (upcoming) Xilinx sub-drivers. It helps sub-drivers for both hardened as well as soft IPs interoperate together.
ZynqMP DP subsystem driver is a sub-driver that implements corresponding drm objects (crtc, plane, encoder, connector,,,) for ZynqMP SoC display pipeline. The entire pipeline is mainly partitioned into 2 blocks: generic display logic (zynqmp_disp.c) such as blending, csc,,, and the DP transmitter logic (zynqmp_dp.c).
I read through it all (well mostly the drm relevant bits, not your backend code) and looks fairly resonable. Few minor clenaups and code removals tbh.
Wrt merging/maintianing, do you want to maintain it as part of the drm-misc small drivers group? Highly recommended imo. See
https://01.org/linuxgraphics/gfx-docs/maintainer-tools/drm-misc.html#small-d...
for details. Ideally we'd need 2 xilinx maintainers to be able to push patches & cross-review stuff. -Daniel
Thanks, -hyun
Hyun Kwon (10): dt-bindings: display: xlnx: Add Xilinx kms bindings drm: xlnx: Add xlnx crtc of Xilinx DRM KMS drm: xlnx: Add xlnx fb of Xilinx DRM KMS drm: xlnx: Add xlnx gem of Xilinx DRM KMS drm: xlnx: Xilinx DRM KMS driver dt-bindings: display: xlnx: Add ZynqMP DP subsystem bindings drm: xlnx: DRM KMS driver for Xilinx ZynqMP DP subsystem display drm: xlnx: DRM KMS driver for Xilinx ZynqMP DisplayPort drm: xlnx: ZynqMP DP subsystem DRM KMS driver drm: xlnx: zynqmp: Add debugfs
.../devicetree/bindings/display/xlnx/xlnx,kms.txt | 20 + .../bindings/display/xlnx/xlnx,zynqmp-dpsub.txt | 94 + MAINTAINERS | 8 + drivers/gpu/drm/Kconfig | 2 + drivers/gpu/drm/Makefile | 1 + drivers/gpu/drm/xlnx/Kconfig | 44 + drivers/gpu/drm/xlnx/Makefile | 5 + drivers/gpu/drm/xlnx/xlnx_crtc.c | 195 ++ drivers/gpu/drm/xlnx/xlnx_crtc.h | 70 + drivers/gpu/drm/xlnx/xlnx_drv.c | 436 +++ drivers/gpu/drm/xlnx/xlnx_drv.h | 22 + drivers/gpu/drm/xlnx/xlnx_fb.c | 468 +++ drivers/gpu/drm/xlnx/xlnx_fb.h | 30 + drivers/gpu/drm/xlnx/xlnx_gem.c | 39 + drivers/gpu/drm/xlnx/xlnx_gem.h | 18 + drivers/gpu/drm/xlnx/zynqmp_disp.c | 3261 ++++++++++++++++++++ drivers/gpu/drm/xlnx/zynqmp_disp.h | 28 + drivers/gpu/drm/xlnx/zynqmp_dp.c | 2168 +++++++++++++ drivers/gpu/drm/xlnx/zynqmp_dp.h | 29 + drivers/gpu/drm/xlnx/zynqmp_dpsub.c | 141 + drivers/gpu/drm/xlnx/zynqmp_dpsub.h | 19 + 21 files changed, 7098 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/xlnx/xlnx,kms.txt create mode 100644 Documentation/devicetree/bindings/display/xlnx/xlnx,zynqmp-dpsub.txt create mode 100644 drivers/gpu/drm/xlnx/Kconfig create mode 100644 drivers/gpu/drm/xlnx/Makefile create mode 100644 drivers/gpu/drm/xlnx/xlnx_crtc.c create mode 100644 drivers/gpu/drm/xlnx/xlnx_crtc.h create mode 100644 drivers/gpu/drm/xlnx/xlnx_drv.c create mode 100644 drivers/gpu/drm/xlnx/xlnx_drv.h create mode 100644 drivers/gpu/drm/xlnx/xlnx_fb.c create mode 100644 drivers/gpu/drm/xlnx/xlnx_fb.h create mode 100644 drivers/gpu/drm/xlnx/xlnx_gem.c create mode 100644 drivers/gpu/drm/xlnx/xlnx_gem.h create mode 100644 drivers/gpu/drm/xlnx/zynqmp_disp.c create mode 100644 drivers/gpu/drm/xlnx/zynqmp_disp.h create mode 100644 drivers/gpu/drm/xlnx/zynqmp_dp.c create mode 100644 drivers/gpu/drm/xlnx/zynqmp_dp.h create mode 100644 drivers/gpu/drm/xlnx/zynqmp_dpsub.c create mode 100644 drivers/gpu/drm/xlnx/zynqmp_dpsub.h
-- 2.7.4
dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel