Re: [PATCH v3 00/32] drm/exynos: Refactor parts of the exynos driver

From: Inki Dae <inki.dae@samsung.com>
To: Sean Paul <seanpaul@chromium.org>
Cc: "Stéphane Marchesin" <marcheu@chromium.org>,
	"DRI mailing list" <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH v3 00/32] drm/exynos: Refactor parts of the exynos driver
Date: Thu, 7 Nov 2013 14:48:28 +0900	[thread overview]
Message-ID: <CAAQKjZOGpr3UPfdqL2d1qrMh9raCqpaW-oJs6UjA2wLM2MYeiQ@mail.gmail.com> (raw)
In-Reply-To: <1383063198-10526-1-git-send-email-seanpaul@chromium.org>

Hi Sean,

When are you going to post your next version?  Need more time? Other
DRM drivers have been merged to drm-next except Exynos. So plz hurry
up if you want to merge this re-factoring patch series during merge
window.

Thanks,
Inki Dae

2013/10/30 Sean Paul <seanpaul@chromium.org>:
> This patchset refactors parts of the exynos driver to move it closer to a proper
> drm driver (rather than just implementing a drm layer on top of the hardware
> drivers). The hope is to get to a point where the dp/hdmi drivers can implement
> drm_connector/drm_encoder directly, and fimd/mixer can directly implement
> drm_crtc.
>
> The notable changes in this set:
>         - drm_encoder funcs no longer route through the crtc
>         - DP driver has been moved from video into the drm driver
>         - fimd no longer implements encoder callbacks
>         - exynos_drm_hdmi is removed in favor of generic manager/display
>           handling
>
> At a glance, differences between v1 and v2:
>         - Passing manager/display in callbacks instead of ctx
>         - Tacked on some dpms patches on the end to handle suspend/resume
>           through the dpms path
>
> At a glance, differences between v2 and v3:
>         - Fixed vidi issues
>         - Moved exynos_drm_hdmi.c removal to the right place
>         - Added the exynos_drm_connector removal patches on the end
>
> Sean
>
>
> Daniel Kurtz (1):
>   drm/exynos: hdmi: remove the i2c drivers and use devtree
>
> Sean Paul (30):
>   drm/exynos: Merge overlay_ops into manager_ops
>   drm/exynos: Add an initialize function to manager and display
>   drm/exynos: Use manager_op initialize in fimd
>   drm/exynos: hdmi: Implement initialize op for hdmi
>   drm/exynos: Pass exynos_drm_manager in manager ops instead of dev
>   drm/exynos: Remove apply manager callback
>   drm/exynos: Remove dpms link between encoder/connector
>   drm/exynos: Rename display_op power_on to dpms
>   drm/exynos: Don't keep dpms state in encoder
>   drm/exynos: Use unsigned long for possible_crtcs
>   drm/exynos: Split manager/display/subdrv
>   drm/exynos: Remove exynos_drm_hdmi shim
>   drm/exynos: Use drm_mode_copy to copy modes
>   drm/exynos: Disable unused crtc planes from crtc
>   drm/exynos: Add mode_set manager operation
>   drm/exynos: Implement mode_fixup manager operation
>   drm/exynos: Use mode_set to configure fimd
>   drm/exynos: Remove unused/useless fimd_context members
>   drm/exynos: Move dp driver from video/ to drm/
>   drm/exynos: Move display implementation into dp
>   ARM: dts: Move display-timings node from fimd to dp
>   drm/exynos: Implement dpms display callback in DP
>   drm/exynos: Clean up FIMD power on/off routines
>   drm/exynos: Consolidate suspend/resume in drm_drv
>   drm/exynos: Add create_connector callback
>   drm/exynos: Implement drm_connector in hdmi directly
>   drm/exynos: Implement drm_connector directly in dp driver
>   drm/exynos: Implement drm_connector directly in vidi driver
>   drm/exynos: Move lvds bridge discovery into DP driver
>   drm/exynos: Remove the exynos_drm_connector shim
>
> Stéphane Marchesin (1):
>   drm/exynos: Remove useless slab.h include
>
>  .../devicetree/bindings/video/exynos_dp.txt        |   17 +
>  .../devicetree/bindings/video/samsung-fimd.txt     |    2 +
>  arch/arm/boot/dts/exynos5250-arndale.dts           |    7 +-
>  arch/arm/boot/dts/exynos5250-smdk5250.dts          |    7 +-
>  arch/arm/boot/dts/exynos5250-snow.dts              |    7 +-
>  arch/arm/boot/dts/exynos5420-smdk5420.dts          |    7 +-
>  drivers/gpu/drm/exynos/Kconfig                     |    7 +
>  drivers/gpu/drm/exynos/Makefile                    |    7 +-
>  drivers/gpu/drm/exynos/exynos_ddc.c                |   63 -
>  drivers/gpu/drm/exynos/exynos_dp_core.c            | 1399 ++++++++++++++++++++
>  drivers/gpu/drm/exynos/exynos_dp_core.h            |  220 +++
>  drivers/gpu/drm/exynos/exynos_dp_reg.c             | 1245 +++++++++++++++++
>  drivers/gpu/drm/exynos/exynos_dp_reg.h             |  366 +++++
>  drivers/gpu/drm/exynos/exynos_drm_connector.c      |  304 -----
>  drivers/gpu/drm/exynos/exynos_drm_connector.h      |   24 -
>  drivers/gpu/drm/exynos/exynos_drm_core.c           |  233 ++--
>  drivers/gpu/drm/exynos/exynos_drm_crtc.c           |  139 +-
>  drivers/gpu/drm/exynos/exynos_drm_crtc.h           |   20 +-
>  drivers/gpu/drm/exynos/exynos_drm_drv.c            |  155 ++-
>  drivers/gpu/drm/exynos/exynos_drm_drv.h            |  154 ++-
>  drivers/gpu/drm/exynos/exynos_drm_encoder.c        |  357 +----
>  drivers/gpu/drm/exynos/exynos_drm_encoder.h        |   18 +-
>  drivers/gpu/drm/exynos/exynos_drm_fb.c             |    4 +-
>  drivers/gpu/drm/exynos/exynos_drm_fimd.c           |  875 ++++++------
>  drivers/gpu/drm/exynos/exynos_drm_hdmi.c           |  439 ------
>  drivers/gpu/drm/exynos/exynos_drm_hdmi.h           |   67 -
>  drivers/gpu/drm/exynos/exynos_drm_plane.c          |   17 +-
>  drivers/gpu/drm/exynos/exynos_drm_plane.h          |    2 +-
>  drivers/gpu/drm/exynos/exynos_drm_vidi.c           |  434 +++---
>  drivers/gpu/drm/exynos/exynos_hdmi.c               |  360 ++---
>  drivers/gpu/drm/exynos/exynos_hdmi.h               |   23 -
>  drivers/gpu/drm/exynos/exynos_hdmiphy.c            |   65 -
>  drivers/gpu/drm/exynos/exynos_mixer.c              |  280 ++--
>  drivers/gpu/drm/exynos/exynos_mixer.h              |   20 +
>  drivers/video/exynos/Kconfig                       |    7 -
>  drivers/video/exynos/Makefile                      |    1 -
>  drivers/video/exynos/exynos_dp_core.c              | 1214 -----------------
>  drivers/video/exynos/exynos_dp_core.h              |  210 ---
>  drivers/video/exynos/exynos_dp_reg.c               | 1245 -----------------
>  drivers/video/exynos/exynos_dp_reg.h               |  366 -----
>  40 files changed, 4735 insertions(+), 5652 deletions(-)
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_ddc.c
>  create mode 100644 drivers/gpu/drm/exynos/exynos_dp_core.c
>  create mode 100644 drivers/gpu/drm/exynos/exynos_dp_core.h
>  create mode 100644 drivers/gpu/drm/exynos/exynos_dp_reg.c
>  create mode 100644 drivers/gpu/drm/exynos/exynos_dp_reg.h
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_connector.c
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_connector.h
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_hdmi.c
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_hdmi.h
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_hdmi.h
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_hdmiphy.c
>  create mode 100644 drivers/gpu/drm/exynos/exynos_mixer.h
>  delete mode 100644 drivers/video/exynos/exynos_dp_core.c
>  delete mode 100644 drivers/video/exynos/exynos_dp_core.h
>  delete mode 100644 drivers/video/exynos/exynos_dp_reg.c
>  delete mode 100644 drivers/video/exynos/exynos_dp_reg.h
>
> --
> 1.8.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel