On Mon, Jul 08, 2013 at 09:44:18PM +0200, Daniel Vetter wrote: > On Mon, Jul 8, 2013 at 4:01 PM, Paulo Zanoni <przanoni at gmail.com> wrote: > > 2013/7/5 Daniel Vetter <daniel at ffwll.ch>: > >> On Fri, Jul 05, 2013 at 12:29:14PM -0700, Ben Widawsky wrote: > >>> On Thu, Jul 04, 2013 at 08:49:44PM +0200, Daniel Vetter wrote: > >>> > At least for the common cases where we only need special file > >>> > operations. The forcewake file is still rather more special. > >>> > > >>> > v2: Fix up the debugfs unregister code. > >>> > > >>> > v3: Actually squash in the right fixup. > >>> > > >>> > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > >>> Acked-by: Ben Widawsky <ben at bwidawsk.net> > >> > >> And merged. > > > > And another patch which you decided to abandon got desperate because > > it didn't want to die, so it tried to hide inside this patch instead > > of being sent to /dev/null: > > > > http://cgit.freedesktop.org/~danvet/drm-intel/commit/drivers/gpu/drm/i915/i915_irq.c?h=drm-intel-next-queued&id=c032d5491a4d8d56d8364f4919364815b55d3437 > > Oops, this seems to be indeed the most mis-laid diff hunk I've ever > written ;-) Thanks for catching it again, I've dropped it from the > patch in dinq. Actually I've merged an old version of the patch, which was broken in other ways, too (which Ben spotted). Hopefully the right version is now in dinq. Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
