[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-06 Thread Joonyoung Shim
Hi, On 02/05/2015 10:06 PM, Daniel Vetter wrote: > On Thu, Feb 05, 2015 at 12:48:07PM +, Daniel Stone wrote: >> Hi, >> >> On 5 February 2015 at 12:26, Rob Clark wrote: >>> On Thu, Feb 5, 2015 at 4:15 AM, Daniel Vetter wrote: Yeah I noticed the zpos fun when hacking around too. Exynos sh

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-05 Thread Daniel Vetter
On Thu, Feb 05, 2015 at 12:48:07PM +, Daniel Stone wrote: > Hi, > > On 5 February 2015 at 12:26, Rob Clark wrote: > > On Thu, Feb 5, 2015 at 4:15 AM, Daniel Vetter wrote: > >> Yeah I noticed the zpos fun when hacking around too. Exynos should > >> probably switch defaults so that overlays ar

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-05 Thread Daniel Stone
Hi, On 5 February 2015 at 12:26, Rob Clark wrote: > On Thu, Feb 5, 2015 at 4:15 AM, Daniel Vetter wrote: >> Yeah I noticed the zpos fun when hacking around too. Exynos should >> probably switch defaults so that overlays are visible by default. And we >> need to standardize the zpos property so t

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-05 Thread Joonyoung Shim
Hi Daniel, On 02/04/2015 11:28 PM, Daniel Vetter wrote: > On Wed, Feb 04, 2015 at 04:44:12PM +0900, Joonyoung Shim wrote: >> Hi, >> >> On 02/04/2015 04:14 AM, Gustavo Padovan wrote: >>> From: Gustavo Padovan >>> >>> struct {fimd,mixer,vidi}_win_data was just keeping the same data >>> as struct ex

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-05 Thread Daniel Vetter
On Thu, Feb 05, 2015 at 11:37:13AM +0900, Joonyoung Shim wrote: > Hi Daniel, > > On 02/04/2015 11:28 PM, Daniel Vetter wrote: > > On Wed, Feb 04, 2015 at 04:44:12PM +0900, Joonyoung Shim wrote: > >> Hi, > >> > >> On 02/04/2015 04:14 AM, Gustavo Padovan wrote: > >>> From: Gustavo Padovan > >>> > >

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-05 Thread Rob Clark
On Thu, Feb 5, 2015 at 4:15 AM, Daniel Vetter wrote: > On Thu, Feb 05, 2015 at 11:37:13AM +0900, Joonyoung Shim wrote: >> Hi Daniel, >> >> On 02/04/2015 11:28 PM, Daniel Vetter wrote: >> > On Wed, Feb 04, 2015 at 04:44:12PM +0900, Joonyoung Shim wrote: >> >> Hi, >> >> >> >> On 02/04/2015 04:14 AM,

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-04 Thread Joonyoung Shim
Hi, On 02/04/2015 04:14 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > struct {fimd,mixer,vidi}_win_data was just keeping the same data > as struct exynos_drm_plane thus get ride of it and use exynos_drm_plane > directly. > > It changes how planes are created and remove .win_mode_set()

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-04 Thread Daniel Vetter
On Wed, Feb 04, 2015 at 04:44:12PM +0900, Joonyoung Shim wrote: > Hi, > > On 02/04/2015 04:14 AM, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > struct {fimd,mixer,vidi}_win_data was just keeping the same data > > as struct exynos_drm_plane thus get ride of it and use exynos_drm_plane

[PATCH 04/14] drm/exynos: remove struct *_win_data abstraction on planes

2015-02-03 Thread Gustavo Padovan
From: Gustavo Padovan struct {fimd,mixer,vidi}_win_data was just keeping the same data as struct exynos_drm_plane thus get ride of it and use exynos_drm_plane directly. It changes how planes are created and remove .win_mode_set() callback that was only filling all *_win_data structs. Signed-off