]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: remove extraneous VGA power domain put calls
authorImre Deak <imre.deak@intel.com>
Fri, 25 Apr 2014 14:28:00 +0000 (17:28 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 5 May 2014 07:09:17 +0000 (09:09 +0200)
commit1bb066ec65495ddb1b7394ef42bd2c3000802b36
treef50d12f28fb5b6533815f1a30379f0e3533d399c
parentc6da4eef96ba6b4b62747b19eac9547e52eb3040
drm/i915: remove extraneous VGA power domain put calls

In recent dmesg logs reported for unrelated issues I noticed some power
domain WARNs caused by the following.

The workaround

commit c7386f02aebc518f1b5618cc68ecdce94996e4ae
Author: Ville Syrjälä <ville.syrjala@linux.intel.com>
Date:   Fri Sep 20 10:14:23 2013 +0300

    drm/i915: Fix unclaimed register access due to delayed VGA memory disable

and following fixup of it

commit 338f50bb8c77e3ddda1d24ff1cde534d39eb2850
Author: Ville Syrjälä <ville.syrjala@linux.intel.com>
Date:   Mon Sep 16 17:38:34 2013 +0300

    drm/i915: Move power well init earlier during driver load

was partially reverted by

commit f8d74323e63a3bd47702eb0074ac0285a65b7b8f
Merge: c516d86 c82a390
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Mon Nov 4 16:28:47 2013 +0100

    Merge tag 'v3.12' into drm-intel-next

but kept the power domain put calls on the error path.

I think for now we can keep things as-is (not reintroduce the w/a) and just fix
the error path, since
- nobody complained seeing this issue
- according to Ville someone is reworking the VGA arbitration scheme at the
  moment and when that's ready we have to rethink this part anyway

So fix this by just removing the put calls from the error path as well.

Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_dma.c