Hi Chris, During the development of runtime power management we faced some random issue around RC6 during the suspend. After some analyzing, we found by following the flow of doing Forcewake Get before the RC6 Disable and forcewake Put after we enable the RC6 solved most of the issues. This patch adds the Forcewake get before disabling the RC6 My commit message was to generic I guess :) I will change the commit message. The flow is between the driver and punit. :) Thanks Deepak -----Original Message----- From: Chris Wilson [mailto:chris@xxxxxxxxxxxxxxxxxx] Sent: Thursday, January 9, 2014 7:32 PM To: S, Deepak Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx Subject: Re: [PATCH v2] drm/i915: Bring UP Power Wells before disabling RC6. On Thu, Jan 09, 2014 at 07:28:27PM +0530, deepak.s@xxxxxxxxx wrote: > From: Deepak S <deepak.s@xxxxxxxxx> > > We need do forcewake before Disabling RC6, This is what the BIOS > expects while going into suspend. What suspend are we talking about? When does the BIOS run, how is that serialised with the get/put of forcewake? -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
