From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Alex Villacis Lasso <alexvillacislasso@hotmail.com>
Cc: "intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
Thierry Reding <thierry.reding@gmail.com>
Subject: Re: [PATCH 0/9] drm/i915: Plane assert/readout cleanups etc.
Date: Wed, 11 Oct 2017 19:38:43 +0300 [thread overview]
Message-ID: <20171011163843.GY10981@intel.com> (raw)
In-Reply-To: <DM5PR04MB026950720AE5FF737E7248C4BA4A0@DM5PR04MB0269.namprd04.prod.outlook.com>
On Wed, Oct 11, 2017 at 04:21:58PM +0000, Alex Villacis Lasso wrote:
> El 11/10/17 a las 11:04, Ville Syrjala escribió:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > 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.
> >
> > 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.
> >
> > 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.
> >
> > Series available here:
> > git://github.com/vsyrjala/linux.git plane_sanitation_2
> >
> > Cc: Thierry Reding <thierry.reding@gmail.com>
> > Cc: Alex Villacís Lasso <alexvillacislasso@hotmail.com>
> >
> > Ville Syrjälä (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
> >
> > 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(-)
> >
> Sorry if this sounds like a newbie question, but what kernel version should these two patches be applied against? Can they be applied on top of 4.13.5?
Not sure they apply cleanly to something so old. In general we develop
everything on top of 'git://anongit.freedesktop.org/drm-tip drm-tip' so
that's where they would at least apply. But in that case it's actually
easier to just grab my plane_sanitation_2 branch directly since it's
sitting on top of the latest drm-tip.
Hmm. Looks like only trivial conflicts when cherry-picking the first
two patches onto 4.13.5. I pushed the result to here:
git://github.com/vsyrjala/linux.git plane_sanitation_2_v4.13
but note that I only compile tested it so it's still possible it won't
actually work.
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2017-10-11 16:38 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-11 16:04 [PATCH 0/9] drm/i915: Plane assert/readout cleanups etc Ville Syrjala
2017-10-11 16:04 ` [PATCH 1/9] drm/i915: Add .get_hw_state() method for planes Ville Syrjala
2017-10-12 18:59 ` Daniel Vetter
2017-10-13 10:31 ` Ville Syrjälä
2017-10-11 16:04 ` [PATCH 2/9] drm/i915: Redo plane sanitation during readout Ville Syrjala
2017-10-12 19:03 ` Daniel Vetter
2017-10-11 16:04 ` [PATCH 3/9] drm/i915: s/enum plane/enum old_plane_id/ Ville Syrjala
2017-10-12 19:06 ` Daniel Vetter
2017-10-13 10:35 ` Ville Syrjälä
2017-10-16 15:57 ` Daniel Vetter
2017-10-11 16:04 ` [PATCH 4/9] drm/i915: Use enum old_plane_id for the .get_fifo_size() hooks Ville Syrjala
2017-10-12 19:08 ` Daniel Vetter
2017-10-11 16:04 ` [PATCH 5/9] drm/i915: Cleanup enum pipe/enum plane_id/enum old_plane_id in initial fb readout Ville Syrjala
2017-10-12 19:11 ` Daniel Vetter
2017-10-11 16:04 ` [PATCH 6/9] drm/i915: Nuke ironlake_get_initial_plane_config() Ville Syrjala
2017-10-12 19:17 ` Daniel Vetter
2017-10-13 10:36 ` Ville Syrjälä
2017-10-11 16:04 ` [PATCH 7/9] drm/i915: Switch fbc over to for_each_new_intel_plane_in_state() Ville Syrjala
2017-10-12 19:21 ` Daniel Vetter
2017-10-13 10:38 ` Ville Syrjälä
2017-10-11 16:04 ` [PATCH 8/9] drm/i915: Nuke crtc->plane Ville Syrjala
2017-10-12 19:38 ` Daniel Vetter
2017-10-13 10:41 ` Ville Syrjälä
2017-10-11 16:04 ` [PATCH 9/9] drm/i915: Add windowing for primary planes on gen2/3 and chv Ville Syrjala
2017-10-12 19:42 ` Daniel Vetter
2017-10-11 16:21 ` [PATCH 0/9] drm/i915: Plane assert/readout cleanups etc Alex Villacis Lasso
2017-10-11 16:38 ` Ville Syrjälä [this message]
2017-10-13 16:28 ` Alex Villacis Lasso
[not found] ` <c5c1b3e5-4640-9df7-45a7-4228802142f9@hotmail.com>
2017-10-14 6:45 ` Alex Villacis Lasso
2017-10-16 14:55 ` Jani Nikula
2017-10-16 15:13 ` Alex Villacís Lasso
2017-10-11 17:20 ` ✓ Fi.CI.BAT: success for " Patchwork
2017-10-11 23:43 ` ✓ Fi.CI.IGT: " Patchwork
2017-10-12 11:35 ` [PATCH 0/9] " Thierry Reding
2017-10-12 12:19 ` Ville Syrjälä
2017-10-12 13:29 ` Thierry Reding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171011163843.GY10981@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=alexvillacislasso@hotmail.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).