From mboxrd@z Thu Jan  1 00:00:00 1970
From: Imre Deak <imre.deak@intel.com>
Subject: Re: [PATCH 10/11] drm/i915: don't get/put runtime PM at
 the debugfs forcewake file
Date: Fri, 28 Feb 2014 16:34:20 +0200
Message-ID: <1393598060.3479.61.camel@intelbox>
References: <1393001548-2883-1-git-send-email-przanoni@gmail.com>
	<1393001548-2883-11-git-send-email-przanoni@gmail.com>
Reply-To: imre.deak@intel.com
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0444647269=="
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mga02.intel.com (mga02.intel.com [134.134.136.20])
	by gabe.freedesktop.org (Postfix) with ESMTP id A9634FB8AC
	for <intel-gfx@lists.freedesktop.org>;
	Fri, 28 Feb 2014 06:34:22 -0800 (PST)
In-Reply-To: <1393001548-2883-11-git-send-email-przanoni@gmail.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@lists.freedesktop.org
Errors-To: intel-gfx-bounces@lists.freedesktop.org
To: Paulo Zanoni <przanoni@gmail.com>
Cc: intel-gfx@lists.freedesktop.org, Paulo Zanoni <paulo.r.zanoni@intel.com>
List-Id: intel-gfx@lists.freedesktop.org


--===============0444647269==
Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature";
	boundary="=-ckTKbkFzUnOi8/177Stb"


--=-ckTKbkFzUnOi8/177Stb
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Fri, 2014-02-21 at 13:52 -0300, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni@intel.com>
>=20
> Because gen6_gt_force_wake_{get,put} should already be responsible for
> getting/putting runtime PM.
>=20
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

Reviewed-by: Imre Deak <imre.deak@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 2 --
>  1 file changed, 2 deletions(-)
>=20
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i=
915_debugfs.c
> index 62d0c0915..2ec7b05 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -3597,7 +3597,6 @@ static int i915_forcewake_open(struct inode *inode,=
 struct file *file)
>  	if (INTEL_INFO(dev)->gen < 6)
>  		return 0;
> =20
> -	intel_runtime_pm_get(dev_priv);
>  	gen6_gt_force_wake_get(dev_priv, FORCEWAKE_ALL);
> =20
>  	return 0;
> @@ -3612,7 +3611,6 @@ static int i915_forcewake_release(struct inode *ino=
de, struct file *file)
>  		return 0;
> =20
>  	gen6_gt_force_wake_put(dev_priv, FORCEWAKE_ALL);
> -	intel_runtime_pm_put(dev_priv);
> =20
>  	return 0;
>  }


--=-ckTKbkFzUnOi8/177Stb
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)

iQEcBAABAgAGBQJTEJ5sAAoJEORIIAnNuWDFOpoH/jNQeAlFkKaTNlsrXW0FJLTG
qjjilFi1T0ns02tmr0vv83EehXLGjzj9rBRpr4JrdTZcz2UIzZ65qD0CPRMumUud
vL2G67IG+sl9prvAwscd5BVToS8AZ5hzCNYb2OwZtAboEtr6LQ8xVLqmwBxpYj6G
rRvyb+7OwKNQ9JMYTpg56fGG5WED58bxHXzU3zE3wNICwEgBfrcLbjw+bGarTjjt
famc8O2KKCmsI/qu6nTl8SQWwj5VEX3VWxJY8nk+Mfh2t2+4ck75KMk59XA8SyEf
6t6DRFBm1nW6/fCvqXmjnfnSJMq97yL6Aawo06fiTGJCoUG3tBqE3JmQ2lOsmT8=
=AOrH
-----END PGP SIGNATURE-----

--=-ckTKbkFzUnOi8/177Stb--


--===============0444647269==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

--===============0444647269==--