Re: [Intel-gfx] [PATCH] drm/i915: Add breadcrumbs for why the backlight is being set

2013-10-14 Thread Jani Nikula
On Sun, 13 Oct 2013, Chris Wilson ch...@chris-wilson.co.uk wrote: At the moment we have 3 paths that lead to actually_set_backlight(), from modesetting, ACPI/OpRegion requests and our very own intel_backlight interface, and we have no way of distinguishing them in the debug log. So add a debug

Re: [Intel-gfx] [PATCH] drm/i915: Add breadcrumbs for why the backlight is being set

2013-10-14 Thread Daniel Vetter
On Sun, Oct 13, 2013 at 12:56:31PM +0100, Chris Wilson wrote: At the moment we have 3 paths that lead to actually_set_backlight(), from modesetting, ACPI/OpRegion requests and our very own intel_backlight interface, and we have no way of distinguishing them in the debug log. So add a debug

Re: [Intel-gfx] [PATCH] drm/i915: Add breadcrumbs for why the backlight is being set

2013-10-14 Thread Jani Nikula
On Mon, 14 Oct 2013, Chris Wilson ch...@chris-wilson.co.uk wrote: On Mon, Oct 14, 2013 at 10:42:37AM +0300, Jani Nikula wrote: On Sun, 13 Oct 2013, Chris Wilson ch...@chris-wilson.co.uk wrote: diff --git a/drivers/gpu/drm/i915/intel_opregion.c b/drivers/gpu/drm/i915/intel_opregion.c index

[Intel-gfx] [PATCH] drm/i915: Add breadcrumbs for why the backlight is being set

2013-10-13 Thread Chris Wilson
At the moment we have 3 paths that lead to actually_set_backlight(), from modesetting, ACPI/OpRegion requests and our very own intel_backlight interface, and we have no way of distinguishing them in the debug log. So add a debug breadcrumb to explain the source of the backlight changes.