Re: [Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-29 Thread Ville Syrjälä
On Tue, Jan 29, 2013 at 08:52:36AM -0800, Ben Widawsky wrote: > On Tue, 29 Jan 2013 09:54:18 +0100 > Daniel Vetter wrote: > > > On Tue, Jan 29, 2013 at 08:41:12AM +, Damien Lespiau wrote: > > > On Mon, Jan 28, 2013 at 03:32:15PM -0800, Ben Widawsky wrote: > > > > /sys/kernel/debug has more or

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-29 Thread Ben Widawsky
On Tue, 29 Jan 2013 09:54:18 +0100 Daniel Vetter wrote: > On Tue, Jan 29, 2013 at 08:41:12AM +, Damien Lespiau wrote: > > On Mon, Jan 28, 2013 at 03:32:15PM -0800, Ben Widawsky wrote: > > > /sys/kernel/debug has more or less been the standard location of > > > debugfs for several years now. O

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-29 Thread Jani Nikula
On Tue, 29 Jan 2013, Daniel Vetter wrote: > Queued for -next, thanks for the patch. Oh and fixed up a bikeshed from > checkpatch.pl while applying: > > Applying: drm/i915: Error state should print /sys/kernel/debug > WARNING: line over 80 characters > #22: FILE: drivers/gpu/drm/i915/i915_irq.c:130

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-29 Thread Daniel Vetter
On Tue, Jan 29, 2013 at 08:41:12AM +, Damien Lespiau wrote: > On Mon, Jan 28, 2013 at 03:32:15PM -0800, Ben Widawsky wrote: > > /sys/kernel/debug has more or less been the standard location of debugfs > > for several years now. Other parts of DRM already use this location, so > > we should as w

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-29 Thread Damien Lespiau
On Mon, Jan 28, 2013 at 03:32:15PM -0800, Ben Widawsky wrote: > /sys/kernel/debug has more or less been the standard location of debugfs > for several years now. Other parts of DRM already use this location, so > we should as well. > > Signed-off-by: Ben Widawsky Reviewed-by: Damien Lespiau --

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-28 Thread Carl Worth
Ben Widawsky writes: > /sys/kernel/debug has more or less been the standard location of debugfs > for several years now. Other parts of DRM already use this location, so > we should as well. > > Signed-off-by: Ben Widawsky Reviewed-by: Carl Worth -Carl pgpygWO4962Aw.pgp Description: PGP sign

[Intel-gfx] [PATCH 1/2] drm/i915: Error state should print /sys/kernel/debug

2013-01-28 Thread Ben Widawsky
/sys/kernel/debug has more or less been the standard location of debugfs for several years now. Other parts of DRM already use this location, so we should as well. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_irq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/