2015-01-08 Inki Dae <inki.dae@xxxxxxxxxxx>: > On 2015년 01월 08일 03:06, Gustavo Padovan wrote: > > 2014-12-26 Inki Dae <inki.dae@xxxxxxxxxxx>: > > > >> On 2014년 12월 22일 22:04, Gustavo Padovan wrote: > >>> Hi Dave, > >>> > >>> Here goes a bunch of clean up for the exynos driver. I've posted this work in > >>> the mailing list twice but never got a review on it, first time was about a > >> > >> Never no. I already had a review and they - your first time patch set - > >> had been merged to exynos-drm-next-todo. I was moving them to > >> exynos-drm-next locally but one of your patch set was not reasonable to > >> me so I gave you one comment. After that, you posted next patch set > >> which include new changes and patches just 9 days ago. So they should > >> also be reviewed enough at least for two weeks. > > > > So please notify on the mailing list what you are doing. To me it looked that > > all patches but one didn't get any review. I have a lot of pending work to do > > on top of this pull request and review on them is quite important for me to > > keep working on new changes. > > > >> > >> Please, do not hurry. Such big changes should really be reviewed enough. > >> I will wait for other reviews and them merge them if reviewed enough. If > >> nobody have reviews then I will merge them. So please, don't worry about > >> that. > > > > It is been a way more than two weeks now, can you please merge all my > > patches but the 3 ones you had commented on? I'll send updated patches for > > those. > > Actually, I am waiting for your reply because your second patch series > includes the changes and updates related to atomic page flip and mode > setting, which doesn't have consistency with your first patch series. > Anyway, I'll merge the only cleanup parts to exynos-drm-next. Please, > post atomic parts after updating later. In addition, I'll not merge > '[PATCH 01/29] drm/exynos/fimd: only finish pageflip if START == > START_S' because not clear to me as I already mentioned before. Thank you. Yes, If you merge all the patches on this pull request I'll be fine with it. It has only the clean ups. Gustavo _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
