Re: [PATCH v3 1/7] drm/exynos: rename zpos to index

2015-12-24 Thread Inki Dae
Hi Marek, Seems this patch could be more cleaned up. Each ctx object of each crtc driver has its own plane config object which includes already zpos value. So I think we wouldn't need to keep zpos of the plane config and index of the plane object. How about removing index from exynos plane stru

Re: [PATCH v3 1/7] drm/exynos: rename zpos to index

2015-12-24 Thread Inki Dae
Below just trivial issue, 2015년 12월 16일 21:21에 Marek Szyprowski 이(가) 쓴 글: > This patch renames zpos entry to index, because in most places it is > used as index for selecting hardware layer/window instead of > configurable layer position. This will later enable to make the zpos > property configur

Re: [PATCH v3 1/7] drm/exynos: rename zpos to index

2015-12-28 Thread Marek Szyprowski
Hello, On 2015-12-24 09:15, Inki Dae wrote: Seems this patch could be more cleaned up. Each ctx object of each crtc driver has its own plane config object which includes already zpos value. So I think we wouldn't need to keep zpos of the plane config and index of the plane object. How about r

Re: [PATCH v3 1/7] drm/exynos: rename zpos to index

2016-01-04 Thread Inki Dae
Hi Marek, 2015년 12월 28일 21:34에 Marek Szyprowski 이(가) 쓴 글: > Hello, > > On 2015-12-24 09:15, Inki Dae wrote: >> Seems this patch could be more cleaned up. >> >> Each ctx object of each crtc driver has its own plane config object which >> includes already zpos value. >> So I think we wouldn't need