[Intel-gfx] [PATCH] drm/i915: move dev_priv->suspend around

Daniel Vetter daniel at ffwll.ch
Wed Mar 12 16:12:06 CET 2014


On Wed, Mar 12, 2014 at 11:01:35AM +0200, Jani Nikula wrote:
> On Mon, 10 Mar 2014, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> > When adding new gunk, _always_ think of a good place. Start/end
> > usually just means that this didn't happen, and on top of that results
> > in needless conflicts with other patches doing the same.
> 
> I'm a fan of unnamed structs within structs to group stuff. The .wm one
> in dev priv is a great example.

Me too, but I haven't found a suitable companion yet for this stuff ...
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch



More information about the Intel-gfx mailing list