Re: [PATCH] drm/i915: convert debugfs creation/destruction to table

[Daniel Vetter <daniel@ffwll.ch>]

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@gmail.com> wrote:
> > 2013/7/5 Daniel Vetter <daniel@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@ffwll.ch>
> >>> Acked-by: Ben Widawsky <ben@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