From: Daniel Vetter <daniel-/w4YWyX8dFk@public.gmane.org>
To: Hyun Kwon <hyunk-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
Cc: Daniel Vetter <daniel-/w4YWyX8dFk@public.gmane.org>,
"dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Michal Simek
<michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH 00/10] Xilinx ZynqMP DisplayPort subsystem DRM KMS driver
Date: Thu, 11 Jan 2018 09:07:38 +0100 [thread overview]
Message-ID: <20180111080738.GC13066@phenom.ffwll.local> (raw)
In-Reply-To: <BY1PR0201MB10002CAFCC860052538BA14DD6160-QYJsKn8jqXK8fGmG9BO4UxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
On Thu, Jan 11, 2018 at 02:07:08AM +0000, Hyun Kwon wrote:
> Hi Daniel,
>
> > -----Original Message-----
> > From: Daniel Vetter [mailto:daniel.vetter-/w4YWyX8dFk@public.gmane.org] On Behalf Of Daniel
> > Vetter
> > Sent: Tuesday, January 09, 2018 1:57 AM
> > To: Hyun Kwon <hyunk-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
> > Cc: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Michal
> > Simek <michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
> > Subject: Re: [PATCH 00/10] Xilinx ZynqMP DisplayPort subsystem DRM
> > KMS driver
> >
> > 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-drivers
> >
> > for details. Ideally we'd need 2 xilinx maintainers to be able to push
> > patches & cross-review stuff.
>
> I don't have any preference on how to maintain, so I'll follow your
> suggestion. One thing that may be worth a note is that there is sizable
> amount of development within Xilinx, and those will come in near future
> (considering what can be done with FPGA :-)). I'll look for the 2nd
> reviewer, and specify that in the next patch if found.
If the xilinx activity gets too much we can always split things up again.
But if it's just the occasional burst (around a new product for example),
then drm-misc has ample of bandwidth to absorb that.
And yes the idea is very much that all regular contributors would have
commit rights too. All to reduce friction and make it easier to
contribute.
-Daniel
>
> Thanks,
> -hyun
>
> > -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-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2018-01-11 8:07 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-05 2:05 [PATCH 00/10] Xilinx ZynqMP DisplayPort subsystem DRM KMS driver Hyun Kwon
[not found] ` <1515117959-18068-1-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-05 2:05 ` [PATCH 01/10] dt-bindings: display: xlnx: Add Xilinx kms bindings Hyun Kwon
[not found] ` <1515117959-18068-2-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-09 4:00 ` Rob Herring
2018-01-11 2:04 ` Hyun Kwon
[not found] ` <BY1PR0201MB1000969ECDC38A62F68B7238D6160-QYJsKn8jqXK8fGmG9BO4UxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2018-01-11 14:43 ` Rob Herring
[not found] ` <CAL_JsqJ_84qg=oJb=HzwgdP9T8osczNT-Eo+u5wjJfT3B8gAQQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-01-11 19:22 ` Hyun Kwon
2018-01-05 2:05 ` [PATCH 02/10] drm: xlnx: Add xlnx crtc of Xilinx DRM KMS Hyun Kwon
2018-01-09 9:37 ` Daniel Vetter
[not found] ` <20180109093733.GG26573-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2018-01-11 2:04 ` Hyun Kwon
2018-01-11 7:48 ` Daniel Vetter
2018-01-05 2:05 ` [PATCH 03/10] drm: xlnx: Add xlnx fb " Hyun Kwon
2018-01-09 9:35 ` Daniel Vetter
2018-01-11 2:04 ` Hyun Kwon
2018-01-05 2:05 ` [PATCH 04/10] drm: xlnx: Add xlnx gem " Hyun Kwon
2018-01-05 2:05 ` [PATCH 05/10] drm: xlnx: Xilinx DRM KMS driver Hyun Kwon
[not found] ` <1515117959-18068-6-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-09 9:51 ` Daniel Vetter
2018-01-11 2:05 ` Hyun Kwon
2018-01-05 2:05 ` [PATCH 06/10] dt-bindings: display: xlnx: Add ZynqMP DP subsystem bindings Hyun Kwon
2018-01-09 4:07 ` Rob Herring
2018-01-11 2:06 ` Hyun Kwon
2018-01-05 2:05 ` [PATCH 07/10] drm: xlnx: DRM KMS driver for Xilinx ZynqMP DP subsystem display Hyun Kwon
[not found] ` <1515117959-18068-8-git-send-email-hyun.kwon-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2018-01-09 9:46 ` Daniel Vetter
[not found] ` <20180109094652.GH26573-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2018-01-11 2:04 ` Hyun Kwon
2018-01-05 2:05 ` [PATCH 08/10] drm: xlnx: DRM KMS driver for Xilinx ZynqMP DisplayPort Hyun Kwon
2018-01-05 2:05 ` [PATCH 09/10] drm: xlnx: ZynqMP DP subsystem DRM KMS driver Hyun Kwon
2018-01-05 2:05 ` [PATCH 10/10] drm: xlnx: zynqmp: Add debugfs Hyun Kwon
2018-01-09 9:54 ` Daniel Vetter
2018-01-11 2:05 ` Hyun Kwon
[not found] ` <BY1PR0201MB10001A1C38398BFBAEC56D39D6160-QYJsKn8jqXK8fGmG9BO4UxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2018-01-11 8:06 ` Daniel Vetter
[not found] ` <20180111080605.GB13066-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2018-01-11 16:57 ` Hyun Kwon
2018-01-09 9:56 ` [PATCH 00/10] Xilinx ZynqMP DisplayPort subsystem DRM KMS driver Daniel Vetter
[not found] ` <20180109095649.GK26573-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2018-01-11 2:07 ` Hyun Kwon
[not found] ` <BY1PR0201MB10002CAFCC860052538BA14DD6160-QYJsKn8jqXK8fGmG9BO4UxrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2018-01-11 8:07 ` Daniel Vetter [this message]
[not found] ` <20180111080738.GC13066-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2018-01-11 8:16 ` Michal Simek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180111080738.GC13066@phenom.ffwll.local \
--to=daniel-/w4ywyx8dfk@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=hyunk-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org \
--cc=michal.simek-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.