Re: [PATCH -v2 0/8] drm/exynos: clean up patches (preparing for atomic)

[Joonyoung Shim <jy0922.shim@samsung.com>]

Hi,

On 03/26/2015 11:10 PM, Gustavo Padovan wrote:
> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> 
> Hi,
> 
> Here goes some clean ups to the exynos drivers. The main clean ups is
> the presetting and zpos making the property immutable and the removal
> of *_win_data structures.
> 
> v2 contains a extra patch to fix alpha setting for planes in fimd, so
> now fimd works fine even after the removal of struct fimd_win_data.
> 

Looks good to me except trivial comment.

Thanks.

> 	Gustavo
> 	
> ---
> Gustavo Padovan (7):
>   drm/exynos: fimd: fix alpha setting for XR24 pixel format
>   drm/exynos: remove unused exynos_crtc->win_enable() callback
>   drm/exynos: remove struct *_win_data abstraction on planes
>   drm/exynos: preset zpos value for overlay planes
>   drm/exynos: make zpos property immutable
>   drm/exynos: remove exynos_plane_destroy()
>   drm/exynos: remove leftover functions declarations
> 
> Mandeep Singh Baines (1):
>   drm/exynos: track vblank events on a per crtc basis
> 
>  drivers/gpu/drm/exynos/exynos7_drm_decon.c | 176 ++++++++--------------
>  drivers/gpu/drm/exynos/exynos_drm_crtc.c   | 101 ++++++-------
>  drivers/gpu/drm/exynos/exynos_drm_crtc.h   |   7 +-
>  drivers/gpu/drm/exynos/exynos_drm_drv.c    |  27 ----
>  drivers/gpu/drm/exynos/exynos_drm_drv.h    |  20 +--
>  drivers/gpu/drm/exynos/exynos_drm_fimd.c   | 226 ++++++++++++-----------------
>  drivers/gpu/drm/exynos/exynos_drm_plane.c  |  66 ++-------
>  drivers/gpu/drm/exynos/exynos_drm_plane.h  |   7 +-
>  drivers/gpu/drm/exynos/exynos_drm_vidi.c   | 134 ++++-------------
>  drivers/gpu/drm/exynos/exynos_mixer.c      | 217 ++++++++++-----------------
>  include/video/samsung_fimd.h               |   5 +
>  11 files changed, 337 insertions(+), 649 deletions(-)
>