From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: David Airlie <airlied-cv59FeDIM0c@public.gmane.org>,
Daniel Vetter
<daniel.vetter-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
Sean Paul <seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Cc: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Frank Rowand
<frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Boris Brezillon
<boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Archit Taneja <architt-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
Jingoo Han <jingoohan1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Inki Dae <inki.dae-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Joonyoung Shim
<jy0922.shim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Seung-Woo Kim
<sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Kyungmin Park
<kyungmin.park-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Kukjin Kim <kgene-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Krzysztof Kozlowski
<krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Javier Martinez Canillas
<javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>,
Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>,
Alison Wang <alison.wang-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
Xinliang Liu
<z.liuxinliang-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>,
Rongrong Zou
<zourongrong-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Xinwei Kong
<kong.kongxinwei-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>,
Chen Feng <puc>
Subject: [PATCH 0/5] DRM OF graph clean-up
Date: Fri, 3 Feb 2017 21:36:30 -0600 [thread overview]
Message-ID: <20170204033635.10250-1-robh@kernel.org> (raw)
I've been unhappy with the OF graph API for some time and decided to
do something about it. The problem is drivers have to do too much of the
graph parsing and walking themselves. This has led to the same pattern
duplicated over and over. This series adds 2 new helpers and adapts DRM
drivers to use them. It only adds one new graph helper, but reduces the
use of the others which I hope to remove at some point. But we're not
there yet.
There's a few other things I'd like to clean-up.
The Armada and Rockchip drivers remain oddballs with their own graph
parsing. I can't see how the armada driver even can work. There's
nothing to instantiate the armada-drm device either in DT or the kernel.
The omapdrm driver still has some custom helpers similar to the common
ones, but still slightly different. It got to be more changes than I
wanted to make to address.
Build tested only (and probably not every last driver).
Rob
Rob Herring (5):
of: introduce of_graph_get_remote_node
drm: of: introduce drm_of_find_panel_or_bridge
drm: convert drivers to use of_graph_get_remote_node
drm: convert drivers to use drm_of_find_panel_or_bridge
drm: omap: use common OF graph helpers
drivers/gpu/drm/arm/hdlcd_drv.c | 22 +----
drivers/gpu/drm/arm/malidp_drv.c | 29 +------
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 64 +++-----------
drivers/gpu/drm/bridge/adv7511/adv7533.c | 12 +--
drivers/gpu/drm/bridge/dumb-vga-dac.c | 15 +---
drivers/gpu/drm/bridge/nxp-ptn3460.c | 16 +---
drivers/gpu/drm/bridge/parade-ps8622.c | 16 +---
drivers/gpu/drm/bridge/tc358767.c | 27 +-----
drivers/gpu/drm/bridge/ti-tfp410.c | 15 ++--
drivers/gpu/drm/drm_of.c | 50 +++++++++++
drivers/gpu/drm/exynos/exynos_dp.c | 35 +++-----
drivers/gpu/drm/exynos/exynos_drm_dpi.c | 16 +---
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 13 +--
drivers/gpu/drm/exynos/exynos_drm_mic.c | 27 +-----
drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 49 ++++-------
drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.c | 27 +-----
drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 30 +------
drivers/gpu/drm/imx/imx-ldb.c | 28 ++-----
drivers/gpu/drm/imx/parallel-display.c | 35 +-------
drivers/gpu/drm/mediatek/mtk_dpi.c | 12 +--
drivers/gpu/drm/mediatek/mtk_dsi.c | 23 ++---
drivers/gpu/drm/mediatek/mtk_hdmi.c | 26 +-----
drivers/gpu/drm/meson/meson_drv.c | 12 +--
drivers/gpu/drm/meson/meson_venc_cvbs.c | 19 +----
drivers/gpu/drm/msm/dsi/dsi_host.c | 3 +-
drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c | 28 +------
drivers/gpu/drm/mxsfb/mxsfb_out.c | 36 ++------
drivers/gpu/drm/omapdrm/dss/dpi.c | 2 +-
drivers/gpu/drm/omapdrm/dss/dsi.c | 2 +-
drivers/gpu/drm/omapdrm/dss/dss-of.c | 102 +----------------------
drivers/gpu/drm/omapdrm/dss/dss.c | 61 +++-----------
drivers/gpu/drm/omapdrm/dss/hdmi4.c | 3 +-
drivers/gpu/drm/omapdrm/dss/hdmi5.c | 2 +-
drivers/gpu/drm/omapdrm/dss/omapdss.h | 11 ---
drivers/gpu/drm/omapdrm/dss/venc.c | 3 +-
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 26 +-----
drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 18 ++--
drivers/gpu/drm/sun4i/sun4i_rgb.c | 17 ++--
drivers/gpu/drm/sun4i/sun4i_tcon.c | 90 ++------------------
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 11 +--
drivers/gpu/drm/tilcdc/tilcdc_external.c | 66 ++-------------
drivers/gpu/drm/vc4/vc4_dpi.c | 15 +---
drivers/of/base.c | 28 +++++++
include/drm/drm_of.h | 13 +++
include/linux/of_graph.h | 8 ++
45 files changed, 272 insertions(+), 891 deletions(-)
--
2.10.1
--
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 reply other threads:[~2017-02-04 3:36 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-04 3:36 Rob Herring [this message]
2017-02-04 3:36 ` [PATCH 1/5] of: introduce of_graph_get_remote_node Rob Herring
2017-02-06 8:50 ` Daniel Vetter
2017-02-06 13:41 ` Rob Herring
[not found] ` <20170204033635.10250-2-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-02-04 16:10 ` Vladimir Zapolskiy
2017-02-06 10:32 ` Philipp Zabel
2017-02-06 13:54 ` Rob Herring
[not found] ` <CAL_JsqJ2HO6WkSGccqGaxHPExnJGyRwrrL2wDOORO+auA=Jbrg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-02-06 14:03 ` Philipp Zabel
2017-02-04 3:36 ` [PATCH 2/5] drm: of: introduce drm_of_find_panel_or_bridge Rob Herring
[not found] ` <20170204033635.10250-3-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-02-06 10:18 ` Liviu Dudau
[not found] ` <20170206101851.GU3140-A/Nd4k6kWRHZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>
2017-02-06 16:20 ` Rob Herring
2017-02-06 10:42 ` Philipp Zabel
[not found] ` <1486377768.3005.34.camel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2017-02-06 16:53 ` Rob Herring
2017-02-06 17:45 ` Philipp Zabel
2017-02-10 19:42 ` Frank Rowand
2017-02-04 3:36 ` [PATCH 3/5] drm: convert drivers to use of_graph_get_remote_node Rob Herring
[not found] ` <20170204033635.10250-4-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-02-06 8:31 ` Jyri Sarha
2017-02-06 10:17 ` Neil Armstrong
2017-02-06 10:29 ` Liviu Dudau
[not found] ` <20170206102933.GV3140-A/Nd4k6kWRHZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>
2017-02-06 17:09 ` Rob Herring
2017-02-06 17:23 ` Liviu Dudau
[not found] ` <20170206172306.GY3140-A/Nd4k6kWRHZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>
2017-02-06 17:34 ` Russell King - ARM Linux
[not found] ` <20170206173407.GK27312-l+eeeJia6m9URfEZ8mYm6t73F7V6hmMc@public.gmane.org>
2017-02-06 17:55 ` Liviu Dudau
[not found] ` <20170206175533.GZ3140-A/Nd4k6kWRHZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>
2017-02-06 18:09 ` Russell King - ARM Linux
2017-02-06 17:42 ` Rob Herring
2017-02-06 10:52 ` Philipp Zabel
2017-02-06 13:40 ` Rob Herring
2017-02-08 11:57 ` Liviu Dudau
2017-02-08 20:44 ` Eric Anholt
2017-02-04 3:36 ` [PATCH 4/5] drm: convert drivers to use drm_of_find_panel_or_bridge Rob Herring
[not found] ` <20170204033635.10250-5-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-02-04 20:26 ` Fabio Estevam
2017-02-05 22:25 ` Rob Herring
[not found] ` <CAL_JsqKnPryTDhXKQtRZ05KZ1KDgNi40JP5W8HFgcEZJUTSBFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-02-06 0:01 ` Fabio Estevam
[not found] ` <CAOMZO5AmrpfDaC3rmH+DTON+xgYuFeXKpVhYHMxYqF9eh3Af-A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-02-06 1:22 ` Fabio Estevam
2017-02-06 10:03 ` Maxime Ripard
2017-02-06 17:32 ` Rob Herring
2017-02-08 7:46 ` Maxime Ripard
2017-02-06 11:07 ` Philipp Zabel
2017-02-04 3:36 ` [PATCH 5/5] drm: omap: use common OF graph helpers Rob Herring
2017-02-04 10:47 ` [PATCH 0/5] DRM OF graph clean-up Russell King - ARM Linux
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=20170204033635.10250-1-robh@kernel.org \
--to=robh-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=airlied-cv59FeDIM0c@public.gmane.org \
--cc=alison.wang-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
--cc=architt-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=daniel.vetter-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=inki.dae-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org \
--cc=jingoohan1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=jy0922.shim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=kgene-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=kong.kongxinwei-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
--cc=krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=kyungmin.park-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=stefan-XLVq0VzYD2Y@public.gmane.org \
--cc=sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=z.liuxinliang-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
--cc=zourongrong-Re5JQEeQqe8AvxtiuMwx3w@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).