From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH 0/9] drm/i915: Plane assert/readout cleanups etc. Date: Thu, 12 Oct 2017 13:35:20 +0200 Message-ID: <20171012113520.GA18419@ulmo> References: <20171011160455.1874-1-ville.syrjala@linux.intel.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1199803802==" Return-path: Received: from mail-wm0-x244.google.com (mail-wm0-x244.google.com [IPv6:2a00:1450:400c:c09::244]) by gabe.freedesktop.org (Postfix) with ESMTPS id 838AB6E7F1 for ; Thu, 12 Oct 2017 11:35:25 +0000 (UTC) Received: by mail-wm0-x244.google.com with SMTP id u138so12553793wmu.4 for ; Thu, 12 Oct 2017 04:35:25 -0700 (PDT) In-Reply-To: <20171011160455.1874-1-ville.syrjala@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ville Syrjala Cc: intel-gfx@lists.freedesktop.org, Alex =?utf-8?B?VmlsbGFjw61z?= Lasso List-Id: intel-gfx@lists.freedesktop.org --===============1199803802== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="RnlQjJ0d97Da+TV1" Content-Disposition: inline --RnlQjJ0d97Da+TV1 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Oct 11, 2017 at 07:04:46PM +0300, Ville Syrjala wrote: > From: Ville Syrj=C3=A4l=C3=A4 >=20 > This series aims to clean up some of the plane state readout and > sanitation, and clean up the enum plane mess a bit by renaming it > to enum old_plane_id. >=20 > The one actual bugfix here is the plane<->crtc sanitation > change. Previously we tried to shut down the entire pipe when > the plane mapping wasn't what we want, now we just shut down the > plane, which is easier. >=20 > Most of the other stuff is just polish, but I also decided to > throw the gen2/3 and chv primary plane windowing support on on top > just because it's been bugging me for years, and I was already > in the neighbourhood. >=20 > Series available here: > git://github.com/vsyrjala/linux.git plane_sanitation_2 >=20 > Cc: Thierry Reding > Cc: Alex Villac=C3=ADs Lasso >=20 > Ville Syrj=C3=A4l=C3=A4 (9): > drm/i915: Add .get_hw_state() method for planes > drm/i915: Redo plane sanitation during readout > drm/i915: s/enum plane/enum old_plane_id/ > drm/i915: Use enum old_plane_id for the .get_fifo_size() hooks > drm/i915: Cleanup enum pipe/enum plane_id/enum old_plane_id in initial > fb readout > drm/i915: Nuke ironlake_get_initial_plane_config() > drm/i915: Switch fbc over to for_each_new_intel_plane_in_state() > drm/i915: Nuke crtc->plane > drm/i915: Add windowing for primary planes on gen2/3 and chv >=20 > drivers/gpu/drm/i915/i915_drv.h | 16 +- > drivers/gpu/drm/i915/intel_display.c | 500 +++++++++++++++--------------= ------ > drivers/gpu/drm/i915/intel_drv.h | 8 +- > drivers/gpu/drm/i915/intel_fbc.c | 27 +- > drivers/gpu/drm/i915/intel_pm.c | 36 +-- > drivers/gpu/drm/i915/intel_sprite.c | 43 +++ > 6 files changed, 299 insertions(+), 331 deletions(-) I take it that this is the same as the plane_sanitation branch that I tested on yesterday, and it fixes both issues I had seen, so for the whole series: Tested-by: Thierry Reding --RnlQjJ0d97Da+TV1 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAlnfU3QACgkQ3SOs138+ s6G92RAAqz2+kWFmIp3OQqPZd38EyBu/8qOgfc9AEZroKInwRGvUxL4V8hXWx6BB oJ/YiPz5J1hGNpDGHlaA2lLi9nV7f2wLZRfiq+TZvjkUjLA7JsG1b3WzphDTtH86 oZSSZ8Vaj36H7IsJ6mWfnGNEdDy2E6rosWXninbyZkuBzur7+25lJDHCNBUW4L1f 1UZwlGmpt9qz/VKEgwAZQL8N+fiMhuY19IJQVxrO/thBZCgHRz9hLPN1NcBnwYHg e3qTjszIJ2GLYu/fkHcuoQQIwXMx/yM0KuTN82ioFa/CfahanR/A4LwfqyQp6S3i LF45nEIlbY6qq8YI2+smzzKXl247IdUwfuNCB/RxKf+56DUw8NLf4clV0DVvwgQC QndJzDtQ3p75b+eEAh4KRwQ9QDNc9oQpQeowAN15Y+Lrqq5GenoHAfWLGG4ekHwB zmER5bv54r5PzaXOn6v7XoC2SEdPLtfXSBTJdV4bYBgNOkigXThLZw2AFEDQKy9D /SG9oiU/Yzy2J3dokPKD9QD1uzZ1/cRYKhHaIPQEs8P2tuq/628qgepXoN/t7OBQ tWMs99tdHWyYIWSA9C57frZ8R0SvX94Gjhrx6IWm3xqGANQOpBhPWjDf0RWV4dxD 5s3J7weDF7N1CY6LYQ7paOD90MpFlVMLQjEuTawd15dDKo54fZM= =jy7z -----END PGP SIGNATURE----- --RnlQjJ0d97Da+TV1-- --===============1199803802== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg== --===============1199803802==--