From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Subject: Re: [PATCH] drm/i915: Clear FORCEWAKE when taking over
	from BIOS
Date: Fri, 19 Oct 2012 12:10:15 +0300
Message-ID: <876266zv2g.fsf@gaia.fi.intel.com>
References: <507E4D8F.4080901@fisher-privat.net>
	<1350557170-6268-1-git-send-email-chris@chris-wilson.co.uk>
	<87bofzylh2.fsf@gaia.fi.intel.com>
	<b94cdc$70ukpd@fmsmga001.fm.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from mga09.intel.com (mga09.intel.com [134.134.136.24])
	by gabe.freedesktop.org (Postfix) with ESMTP id E20A29EBA4
	for <intel-gfx@lists.freedesktop.org>;
	Fri, 19 Oct 2012 02:10:18 -0700 (PDT)
In-Reply-To: <b94cdc$70ukpd@fmsmga001.fm.intel.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org
To: Chris Wilson <chris@chris-wilson.co.uk>, intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Fri, 19 Oct 2012 09:02:10 +0100, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> On Fri, 19 Oct 2012 10:22:49 +0300, Mika Kuoppala <mika.kuoppala@linux.intel.com> wrote:
> > Would it be possible to call in here:
> > 
> >       if (HAS_FORCE_WAKE(dev))
> >          dev_priv->gt.force_wake_put(dev_priv);
> > 
> > and get rid of the per gen forcewake resets?
> 
> No. Perhaps I would have better gone with intel_gt_sanitize(), as the
> purpose is not to undo our own settings of the forcewake but undo that
> of nefarious foreign interests such as the BIOS. As such we need to
> explicitly clear the register rather than clear a bit.
> -Chris

Thanks for explanation.

Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>

> -- 
> Chris Wilson, Intel Open Source Technology Centre
---------------------------------------------------------------------
Intel Finland Oy
Registered Address: PL 281, 00181 Helsinki 
Business Identity Code: 0357606 - 4 
Domiciled in Helsinki 

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.