[Intel-gfx] [PATCH 0/5] refactor power management into intel_pm

Eugeni Dodonov eugeni at dodonov.net
Tue Apr 17 11:56:27 CEST 2012


On Tue, Apr 17, 2012 at 05:57, Daniel Vetter <daniel at ffwll.ch> wrote:

> On Mon, Apr 16, 2012 at 10:20:33PM -0300, Eugeni Dodonov wrote:
> > As Chris Wilson noticed, my previous patch that did the refactoring as
> one big
> > patch which moved everything at once was extremely difficult to review
> and
> > maintain. So I split the same refactoring into a series of smaller
> patches,
> > which move one subsystem at a time.
>
> Yeah, diffs are not actually readable, so looks like we have to do this
> piece-wise.


Please, tell me that you meant 'diffs are *now* actually readable' with a
'not' being a typo instead of 'now' there by a chance...

Because otherwise I couldn't understand what you meant with piece-wise.
Should I move each function at a time? :)

I'll do the intel_pm_init (or intel_init_pm) and send it on top of this
series. It makes lots of sense and will avoid exposing all those
platform-specific stuff where we won't need it except for initialization.

-- 
Eugeni Dodonov
<http://eugeni.dodonov.net/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120417/cd4050dc/attachment.html>


More information about the Intel-gfx mailing list