Hi all, I've gone through some of the contentions point in this patch review. With my community guy hat on I really want to make drm atomic a success, exynos atomic is important for that. On Wed, Feb 04, 2015 at 04:37:04PM +0900, Joonyoung Shim wrote: > Hi, > > On 02/04/2015 04:14 AM, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > > > > Hi, > > > > This series clean ups a few more paths from exynos-drm with the most important > > being the removal of the global page flip queue and the removal in driver > > internal data (struct *_win_data) that was replicating plane data. > > > > Following these patches comes the first step torwards atomic modesetting > > support on exynos. > > > > It's better to split cleanup and atomic support, not one patchset. Imo the cleanups make perfect sense as prep work for atomic. They're definitely needed afaict from what I've seen reading exynos code and these patches here before we can implement atomic. Personally I'd have delayered even more aggressively before going into the atomic stuff. But in the end I guess Padovan wants to be able to ship exynos atomic preferrably sooner than later. Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |