2015-01-08 Inki Dae <inki.dae at samsung.com>:
> On 2015ë
01ì 08ì¼ 03:06, Gustavo Padovan wrote:
> > 2014-12-26 Inki Dae <inki.dae at samsung.com>:
> >
> >> 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
