Re: [Intel-gfx] [PATCH] drm/i915: enable the power well before module unload

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 15, 2013 at 12:07:44PM +0100, Damien Lespiau wrote:
> On Wed, Aug 14, 2013 at 02:40:37PM -0300, Paulo Zanoni wrote:
> > From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> > 
> > Our driver initialization doesn't seem to be ready to load when the
> > power well is disabled: we hit a few "Unclaimed register" messages. So
> > do just like we already do for the suspend/resume path: enable the
> > power well before unloading.
> > 
> > At some point we'll want to be able to survive suspend/resume and
> > load/unload with the power well disabled, but for now let's just fix
> > the regression.
> > 
> > Regression introduced by the following commit:
> > 
> > commit bf51d5e2cda5d36d98e4b46ac7fca9461e512c41
> > Author: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> > Date:   Wed Jul 3 17:12:13 2013 -0300
> >     drm/i915: switch disable_power_well default value to 1
> > 
> > Bug can be reproduced by running the "module_reload" script from
> > intel-gpu-tools.
> > 
> > Cc: stable@xxxxxxxxxxxxxxx
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67813
> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> 
> Reviewed-by: Damien Lespiau <damien.lespiau@xxxxxxxxx>

Queued for -next (with the cc: stable dropped since module unload is a
developer feature), thanks for the patch.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]