Re: [PATCH v3 0/5] DRM OF graph clean-up

From: Sean Paul <seanpaul@chromium.org>
To: Rob Herring <robh@kernel.org>
Cc: David Airlie <airlied@linux.ie>,
	Daniel Vetter <daniel.vetter@intel.com>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Boris Brezillon <boris.brezillon@free-electrons.com>,
	Archit Taneja <architt@codeaurora.org>,
	Jingoo Han <jingoohan1@gmail.com>,
	Inki Dae <inki.dae@samsung.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Kukjin Kim <kgene@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Javier Martinez Canillas <javier@osg.samsung.com>,
	Stefan Agner <stefan@agner.ch>,
	Alison Wang <alison.wang@freescale.com>,
	Xinliang Liu <z.liuxinliang@hisilicon.com>,
	Rongrong Zou <zourongrong@gmail.com>,
	Xinwei Kong <kong.kongxinwei@hisilicon.com>,
	Chen Feng <puck.chen@hisilicon.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	CK Hu <ck.hu@mediatek.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Marek Vasut <marex@denx.de>, Mark Yao <mark.yao@rock-chips.com>,
	Heiko Stuebner <heiko@sntech.de>,
	Maxime Ripard <maxime.ripard@free-electrons.com>,
	Chen-Yu Tsai <wens@csie.org>, Liviu Dudau <liviu.dudau@arm.com>,
	Mali DP Maintainers <malidp@foss.arm.com>,
	Neil Armstrong <narmstrong@baylibre.com>,
	Carlo Caione <carlo@caione.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Rob Clark <robdclark@gmail.com>, Jyri Sarha <jsarha@ti.com>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Eric Anholt <eric@anholt.net>,
	Russell King <rmk+kernel@armlinux.org.uk>
Subject: Re: [PATCH v3 0/5] DRM OF graph clean-up
Date: Thu, 6 Apr 2017 17:03:06 -0400	[thread overview]
Message-ID: <CAOw6vbJio6zL52YZ+7k+OP_Dkry4hw1HXA4O09US2DKRd0PyBA@mail.gmail.com> (raw)
In-Reply-To: <20170322132608.17353-1-robh@kernel.org>

On Wed, Mar 22, 2017 at 9:26 AM, Rob Herring <robh@kernel.org> wrote:
> 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 adapts DRM drivers to use a new OF
> graph helper and added DRM helper.
>
> The DT dependency went into 4.11, so this series can be applied to the DRM
> tree without any cross tree dependencies.
>
> I've done some build testing only, so testing appreciated. A git branch is
> here[1].
>
> Rob
>
> [1] git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git of-graph-helpers
>
>
> Rob Herring (5):
>   drm: make of_drm_find_panel also depend on CONFIG_DRM_PANEL
>   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
>

Hi Rob,
Thank you for the series, and sorry for the delay. I've fixed the 2
conflicts and applied to -misc-next.

Sean

>  drivers/gpu/drm/arm/hdlcd_drv.c                  |  22 +----
>  drivers/gpu/drm/arm/malidp_drv.c                 |  28 +------
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c |  73 +++++-----------
>  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                         |  52 ++++++++++++
>  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          |  25 +-----
>  drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c        |  39 ++-------
>  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                    |  27 ++----
>  drivers/gpu/drm/imx/parallel-display.c           |  36 +-------
>  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_venc_cvbs.c          |  19 +----
>  drivers/gpu/drm/msm/dsi/dsi_host.c               |   2 +-
>  drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c          |  28 +------
>  drivers/gpu/drm/mxsfb/mxsfb_out.c                |  40 ++-------
>  drivers/gpu/drm/omapdrm/dss/dpi.c                |   2 +-
>  drivers/gpu/drm/omapdrm/dss/dsi.c                |   3 +-
>  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              |   3 +-
>  drivers/gpu/drm/omapdrm/dss/omapdss.h            |  11 ---
>  drivers/gpu/drm/omapdrm/dss/sdi.c                |   2 +-
>  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                |  11 +--
>  drivers/gpu/drm/sun4i/sun4i_tcon.c               |  90 ++------------------
>  drivers/gpu/drm/tilcdc/tilcdc_crtc.c             |  12 +--
>  drivers/gpu/drm/tilcdc/tilcdc_external.c         |  68 ++-------------
>  drivers/gpu/drm/vc4/vc4_dpi.c                    |  15 +---
>  include/drm/drm_of.h                             |  13 +++
>  include/drm/drm_panel.h                          |   2 +-
>  44 files changed, 238 insertions(+), 881 deletions(-)
>
> --
> 2.10.1
>