mbox series

[v3,0/5] DRM OF graph clean-up

Message ID 20170322132608.17353-1-robh@kernel.org
Headers show
Series DRM OF graph clean-up | expand

Message

Rob Herring March 22, 2017, 1:26 p.m. UTC
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

 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

Comments

Sean Paul April 6, 2017, 9:03 p.m. UTC | #1
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

>