Re: [PATCH 01/53] drm/i915: Remove ironlake rc6 support

Daniel Vetter
2015-03-06T16:30:37Z

On Thu, Mar 05, 2015 at 03:36:03PM +0000, John Harrison wrote:
> On 05/03/2015 15:22, Daniel Vetter wrote:
> >On Thu, Mar 05, 2015 at 02:03:03PM +0000, John.C.Harrison@Intel.com wrote:
> >>From: John Harrison <John.C.Harrison@Intel.com>
> >>
> >>Apparently, this has never worked reliably and is currently disabled. Also, the
> >>gains are not particularly impressive. Thus rather than try to keep unused code
> >>from decaying and having to update it for other driver changes, it was decided
> >>to simply remove it.
> >>
> >>For: VIZ-5115
> >>Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> >Queued for -next, thanks for the patch.
> >-Daniel
>
> Can you hold off merging any of the other patches in the series for the
> moment. Tomas Elf is currently posting review comments, some with R-B some
> with change requests. Some of the re-work either for his comments or for
> yours causes changes which ripple through a number of the patches in the
> series. So I would like to repost the whole set once all the comments have
> been addressed. Otherwise, it all just gets far too messy as to which
> patches have or haven't been updated, what needs re-merging, which version
> is which, etc.

Fully agreed since that was my plan. I specifically pulled this one here
in since it should be otherwise free-standing and would likely cause
needless conflicts. For all the other patches it's better to first
converge on the review a bit more before proceeding with merging.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch