From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 06/14] drm/i915: Reorganize plane register writes to make them more atomic
Date: Wed, 7 Nov 2018 23:38:37 +0200 [thread overview]
Message-ID: <20181107213837.GQ9144@intel.com> (raw)
In-Reply-To: <20181107212618.GI2237@intel.com>
On Wed, Nov 07, 2018 at 01:26:18PM -0800, Rodrigo Vivi wrote:
> On Thu, Nov 01, 2018 at 05:05:57PM +0200, Ville Syrjala wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > Some observations about the plane registers:
> > - the control register will self-arm if the plane is not already
> > enabled, thus we want to write it as close to (or ideally after)
> > the surface register
> > - tileoff/linoff/offset/aux_offset are self-arming as well so we want
> > them close to the surface register as well
> > - color keying registers we maybe self arming before SKL. Not 100%
> > sure but we can try to keep them near to the surface register
> > as well
> > - chv pipe b csc register are double buffered but self arming so
> > moving them down a bit
> > - the rest should be mostly armed by the surface register so we can
> > safely write them first, and to just for some consistency let's try
> > to follow keep them in order based on the register offset
> >
> > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 40 +++++-----
> > drivers/gpu/drm/i915/intel_sprite.c | 114 +++++++++++++++------------
> > 2 files changed, 86 insertions(+), 68 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index c5ce3892d583..9521cff5fb44 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -3328,7 +3328,6 @@ static void i9xx_update_plane(struct intel_plane *plane,
> > enum i9xx_plane_id i9xx_plane = plane->i9xx_plane;
> > u32 linear_offset;
> > u32 dspcntr = plane_state->ctl;
> > - i915_reg_t reg = DSPCNTR(i9xx_plane);
> > int x = plane_state->color_plane[0].x;
> > int y = plane_state->color_plane[0].y;
> > unsigned long irqflags;
> > @@ -3343,41 +3342,45 @@ static void i9xx_update_plane(struct intel_plane *plane,
> >
> > spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> >
> > + I915_WRITE_FW(DSPSTRIDE(i9xx_plane), plane_state->color_plane[0].stride);
> > +
> > if (INTEL_GEN(dev_priv) < 4) {
> > /* pipesrc and dspsize control the size that is scaled from,
> > * which should always be the user's requested size.
> > */
> > + I915_WRITE_FW(DSPPOS(i9xx_plane), 0);
> > I915_WRITE_FW(DSPSIZE(i9xx_plane),
> > ((crtc_state->pipe_src_h - 1) << 16) |
> > (crtc_state->pipe_src_w - 1));
> > - I915_WRITE_FW(DSPPOS(i9xx_plane), 0);
> > } else if (IS_CHERRYVIEW(dev_priv) && i9xx_plane == PLANE_B) {
> > + I915_WRITE_FW(PRIMPOS(i9xx_plane), 0);
> > I915_WRITE_FW(PRIMSIZE(i9xx_plane),
> > ((crtc_state->pipe_src_h - 1) << 16) |
> > (crtc_state->pipe_src_w - 1));
> > - I915_WRITE_FW(PRIMPOS(i9xx_plane), 0);
> > I915_WRITE_FW(PRIMCNSTALPHA(i9xx_plane), 0);
> > }
> >
> > - I915_WRITE_FW(reg, dspcntr);
> > -
> > - I915_WRITE_FW(DSPSTRIDE(i9xx_plane), plane_state->color_plane[0].stride);
> > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) {
> > - I915_WRITE_FW(DSPSURF(i9xx_plane),
> > - intel_plane_ggtt_offset(plane_state) +
> > - dspaddr_offset);
> > I915_WRITE_FW(DSPOFFSET(i9xx_plane), (y << 16) | x);
> > } else if (INTEL_GEN(dev_priv) >= 4) {
> > + I915_WRITE_FW(DSPTILEOFF(i9xx_plane), (y << 16) | x);
> > + I915_WRITE_FW(DSPLINOFF(i9xx_plane), linear_offset);
> > + }
> > +
> > + /*
> > + * The control register self-arms if the plane was previously
> > + * disabled. Try to make the plane enable atomic by writing
> > + * the control register just before the surface register.
> > + */
> > + I915_WRITE_FW(DSPCNTR(i9xx_plane), dspcntr);
> > + if (INTEL_GEN(dev_priv) >= 4)
> > I915_WRITE_FW(DSPSURF(i9xx_plane),
> > intel_plane_ggtt_offset(plane_state) +
> > dspaddr_offset);
> > - I915_WRITE_FW(DSPTILEOFF(i9xx_plane), (y << 16) | x);
> > - I915_WRITE_FW(DSPLINOFF(i9xx_plane), linear_offset);
> > - } else {
> > + else
> > I915_WRITE_FW(DSPADDR(i9xx_plane),
> > intel_plane_ggtt_offset(plane_state) +
> > dspaddr_offset);
> > - }
> >
> > spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
> > }
> > @@ -10045,8 +10048,8 @@ static void i9xx_update_cursor(struct intel_plane *plane,
> > * On some platforms writing CURCNTR first will also
> > * cause CURPOS to be armed by the CURBASE write.
> > * Without the CURCNTR write the CURPOS write would
> > - * arm itself. Thus we always start the full update
> > - * with a CURCNTR write.
> > + * arm itself. Thus we always update CURCNTR before
> > + * CURPOS.
> > *
> > * On other platforms CURPOS always requires the
> > * CURBASE write to arm the update. Additonally
> > @@ -10056,15 +10059,16 @@ static void i9xx_update_cursor(struct intel_plane *plane,
> > * cursor that doesn't appear to move, or even change
> > * shape. Thus we always write CURBASE.
> > *
> > - * CURCNTR and CUR_FBC_CTL are always
> > - * armed by the CURBASE write only.
> > + * The other registers are armed by by the CURBASE write
> > + * except when the plane is getting enabled at which time
> > + * the CURCNTR write arms the update.
> > */
> > if (plane->cursor.base != base ||
> > plane->cursor.size != fbc_ctl ||
> > plane->cursor.cntl != cntl) {
> > - I915_WRITE_FW(CURCNTR(pipe), cntl);
> > if (HAS_CUR_FBC(dev_priv))
> > I915_WRITE_FW(CUR_FBC_CTL(pipe), fbc_ctl);
> > + I915_WRITE_FW(CURCNTR(pipe), cntl);
> > I915_WRITE_FW(CURPOS(pipe), pos);
> > I915_WRITE_FW(CURBASE(pipe), base);
> >
> > diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c
> > index 8a40879abe30..455b2d0cbaa6 100644
> > --- a/drivers/gpu/drm/i915/intel_sprite.c
> > +++ b/drivers/gpu/drm/i915/intel_sprite.c
> > @@ -404,24 +404,12 @@ skl_program_plane(struct intel_plane *plane,
> >
> > spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> >
> > - if (INTEL_GEN(dev_priv) >= 10 || IS_GEMINILAKE(dev_priv))
> > - I915_WRITE_FW(PLANE_COLOR_CTL(pipe, plane_id),
> > - plane_state->color_ctl);
> > -
> > - I915_WRITE_FW(PLANE_KEYVAL(pipe, plane_id), key->min_value);
> > - I915_WRITE_FW(PLANE_KEYMAX(pipe, plane_id), keymax);
> > - I915_WRITE_FW(PLANE_KEYMSK(pipe, plane_id), keymsk);
> > -
> > - I915_WRITE_FW(PLANE_OFFSET(pipe, plane_id), (y << 16) | x);
> > I915_WRITE_FW(PLANE_STRIDE(pipe, plane_id), stride);
> > + I915_WRITE_FW(PLANE_POS(pipe, plane_id), (crtc_y << 16) | crtc_x);
> > I915_WRITE_FW(PLANE_SIZE(pipe, plane_id), (src_h << 16) | src_w);
> > I915_WRITE_FW(PLANE_AUX_DIST(pipe, plane_id),
> > - (plane_state->color_plane[1].offset - surf_addr) | aux_stride);
> > -
> > - if (INTEL_GEN(dev_priv) < 11)
> > - I915_WRITE_FW(PLANE_AUX_OFFSET(pipe, plane_id),
> > - (plane_state->color_plane[1].y << 16) |
> > - plane_state->color_plane[1].x);
> > + (plane_state->color_plane[1].offset - surf_addr) |
> > + aux_stride);
> >
> > if (icl_is_hdr_plane(plane)) {
> > u32 cus_ctl = 0;
> > @@ -444,15 +432,33 @@ skl_program_plane(struct intel_plane *plane,
> > I915_WRITE_FW(PLANE_CUS_CTL(pipe, plane_id), cus_ctl);
> > }
> >
> > - if (!slave && plane_state->scaler_id >= 0)
> > - skl_program_scaler(plane, crtc_state, plane_state);
> > + if (INTEL_GEN(dev_priv) >= 10 || IS_GEMINILAKE(dev_priv))
> > + I915_WRITE_FW(PLANE_COLOR_CTL(pipe, plane_id),
> > + plane_state->color_ctl);
> >
> > - I915_WRITE_FW(PLANE_POS(pipe, plane_id), (crtc_y << 16) | crtc_x);
> > + I915_WRITE_FW(PLANE_KEYVAL(pipe, plane_id), key->min_value);
> > + I915_WRITE_FW(PLANE_KEYMAX(pipe, plane_id), keymax);
> > + I915_WRITE_FW(PLANE_KEYMSK(pipe, plane_id), keymsk);
> > +
> > + I915_WRITE_FW(PLANE_OFFSET(pipe, plane_id), (y << 16) | x);
> >
> > + if (INTEL_GEN(dev_priv) < 11)
> > + I915_WRITE_FW(PLANE_AUX_OFFSET(pipe, plane_id),
> > + (plane_state->color_plane[1].y << 16) |
> > + plane_state->color_plane[1].x);
> > +
> > + /*
> > + * The control register self-arms if the plane was previously
> > + * disabled. Try to make the plane enable atomic by writing
> > + * the control register just before the surface register.
> > + */
> > I915_WRITE_FW(PLANE_CTL(pipe, plane_id), plane_ctl);
> > I915_WRITE_FW(PLANE_SURF(pipe, plane_id),
> > intel_plane_ggtt_offset(plane_state) + surf_addr);
> >
> > + if (!slave && plane_state->scaler_id >= 0)
> > + skl_program_scaler(plane, crtc_state, plane_state);
> > +
> > spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
> > }
> >
> > @@ -690,7 +696,6 @@ vlv_update_plane(struct intel_plane *plane,
> > const struct intel_plane_state *plane_state)
> > {
> > struct drm_i915_private *dev_priv = to_i915(plane->base.dev);
> > - const struct drm_framebuffer *fb = plane_state->base.fb;
> > enum pipe pipe = plane->pipe;
> > enum plane_id plane_id = plane->id;
> > u32 sprctl = plane_state->ctl;
> > @@ -715,6 +720,12 @@ vlv_update_plane(struct intel_plane *plane,
> >
> > vlv_update_clrc(plane_state);
> >
> > + I915_WRITE_FW(SPSTRIDE(pipe, plane_id),
> > + plane_state->color_plane[0].stride);
> > + I915_WRITE_FW(SPPOS(pipe, plane_id), (crtc_y << 16) | crtc_x);
> > + I915_WRITE_FW(SPSIZE(pipe, plane_id), (crtc_h << 16) | crtc_w);
> > + I915_WRITE_FW(SPCONSTALPHA(pipe, plane_id), 0);
> > +
> > if (IS_CHERRYVIEW(dev_priv) && pipe == PIPE_B)
> > chv_update_csc(plane_state);
> >
> > @@ -723,18 +734,15 @@ vlv_update_plane(struct intel_plane *plane,
> > I915_WRITE_FW(SPKEYMAXVAL(pipe, plane_id), key->max_value);
> > I915_WRITE_FW(SPKEYMSK(pipe, plane_id), key->channel_mask);
> > }
> > - I915_WRITE_FW(SPSTRIDE(pipe, plane_id),
> > - plane_state->color_plane[0].stride);
> > - I915_WRITE_FW(SPPOS(pipe, plane_id), (crtc_y << 16) | crtc_x);
> > -
> > - if (fb->modifier == I915_FORMAT_MOD_X_TILED)
> > - I915_WRITE_FW(SPTILEOFF(pipe, plane_id), (y << 16) | x);
> > - else
> > - I915_WRITE_FW(SPLINOFF(pipe, plane_id), linear_offset);
> >
> > - I915_WRITE_FW(SPCONSTALPHA(pipe, plane_id), 0);
> > + I915_WRITE_FW(SPTILEOFF(pipe, plane_id), (y << 16) | x);
> > + I915_WRITE_FW(SPLINOFF(pipe, plane_id), linear_offset);
> >
> > - I915_WRITE_FW(SPSIZE(pipe, plane_id), (crtc_h << 16) | crtc_w);
> > + /*
> > + * The control register self-arms if the plane was previously
> > + * disabled. Try to make the plane enable atomic by writing
> > + * the control register just before the surface register.
> > + */
> > I915_WRITE_FW(SPCNTR(pipe, plane_id), sprctl);
> > I915_WRITE_FW(SPSURF(pipe, plane_id),
> > intel_plane_ggtt_offset(plane_state) + sprsurf_offset);
> > @@ -848,7 +856,6 @@ ivb_update_plane(struct intel_plane *plane,
> > const struct intel_plane_state *plane_state)
> > {
> > struct drm_i915_private *dev_priv = to_i915(plane->base.dev);
> > - const struct drm_framebuffer *fb = plane_state->base.fb;
> > enum pipe pipe = plane->pipe;
> > u32 sprctl = plane_state->ctl, sprscale = 0;
> > u32 sprsurf_offset = plane_state->color_plane[0].offset;
> > @@ -877,27 +884,32 @@ ivb_update_plane(struct intel_plane *plane,
> >
> > spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> >
> > + I915_WRITE_FW(SPRSTRIDE(pipe), plane_state->color_plane[0].stride);
> > + I915_WRITE_FW(SPRPOS(pipe), (crtc_y << 16) | crtc_x);
> > + I915_WRITE_FW(SPRSIZE(pipe), (crtc_h << 16) | crtc_w);
> > + if (IS_IVYBRIDGE(dev_priv))
> > + I915_WRITE_FW(SPRSCALE(pipe), sprscale);
> > +
> > if (key->flags) {
> > I915_WRITE_FW(SPRKEYVAL(pipe), key->min_value);
> > I915_WRITE_FW(SPRKEYMAX(pipe), key->max_value);
> > I915_WRITE_FW(SPRKEYMSK(pipe), key->channel_mask);
> > }
> >
> > - I915_WRITE_FW(SPRSTRIDE(pipe), plane_state->color_plane[0].stride);
> > - I915_WRITE_FW(SPRPOS(pipe), (crtc_y << 16) | crtc_x);
> > -
> > /* HSW consolidates SPRTILEOFF and SPRLINOFF into a single SPROFFSET
> > * register */
> > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
> > + if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) {
> > I915_WRITE_FW(SPROFFSET(pipe), (y << 16) | x);
> > - else if (fb->modifier == I915_FORMAT_MOD_X_TILED)
> > + } else {
> > I915_WRITE_FW(SPRTILEOFF(pipe), (y << 16) | x);
> > - else
> > I915_WRITE_FW(SPRLINOFF(pipe), linear_offset);
> > + }
> >
> > - I915_WRITE_FW(SPRSIZE(pipe), (crtc_h << 16) | crtc_w);
> > - if (IS_IVYBRIDGE(dev_priv))
> > - I915_WRITE_FW(SPRSCALE(pipe), sprscale);
> > + /*
> > + * The control register self-arms if the plane was previously
> > + * disabled. Try to make the plane enable atomic by writing
> > + * the control register just before the surface register.
> > + */
> > I915_WRITE_FW(SPRCTL(pipe), sprctl);
> > I915_WRITE_FW(SPRSURF(pipe),
> > intel_plane_ggtt_offset(plane_state) + sprsurf_offset);
> > @@ -915,7 +927,7 @@ ivb_disable_plane(struct intel_plane *plane, struct intel_crtc *crtc)
> > spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> >
> > I915_WRITE_FW(SPRCTL(pipe), 0);
> > - /* Can't leave the scaler enabled... */
> > + /* Disable the scaler */
> > if (IS_IVYBRIDGE(dev_priv))
> > I915_WRITE_FW(SPRSCALE(pipe), 0);
> > I915_WRITE_FW(SPRSURF(pipe), 0);
> > @@ -1017,7 +1029,6 @@ g4x_update_plane(struct intel_plane *plane,
> > const struct intel_plane_state *plane_state)
> > {
> > struct drm_i915_private *dev_priv = to_i915(plane->base.dev);
> > - const struct drm_framebuffer *fb = plane_state->base.fb;
> > enum pipe pipe = plane->pipe;
> > u32 dvscntr = plane_state->ctl, dvsscale = 0;
> > u32 dvssurf_offset = plane_state->color_plane[0].offset;
> > @@ -1046,22 +1057,25 @@ g4x_update_plane(struct intel_plane *plane,
> >
> > spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> >
> > + I915_WRITE_FW(DVSSTRIDE(pipe), plane_state->color_plane[0].stride);
> > + I915_WRITE_FW(DVSPOS(pipe), (crtc_y << 16) | crtc_x);
> > + I915_WRITE_FW(DVSSIZE(pipe), (crtc_h << 16) | crtc_w);
> > + I915_WRITE_FW(DVSSCALE(pipe), dvsscale);
> > +
> > if (key->flags) {
> > I915_WRITE_FW(DVSKEYVAL(pipe), key->min_value);
> > I915_WRITE_FW(DVSKEYMAX(pipe), key->max_value);
> > I915_WRITE_FW(DVSKEYMSK(pipe), key->channel_mask);
> > }
> >
> > - I915_WRITE_FW(DVSSTRIDE(pipe), plane_state->color_plane[0].stride);
> > - I915_WRITE_FW(DVSPOS(pipe), (crtc_y << 16) | crtc_x);
> > -
> > - if (fb->modifier == I915_FORMAT_MOD_X_TILED)
>
> I believe this removed if deserves a separated patch...
Forgot I even did that :) But yeah, I'll split that out into a separate
patch.
>
> the rest was hard to review, but after you pointed out how to check
> on bspec the armed-by field I think it all makes sense.
Cool. And sorry for the messy patch. Couldn't think of a nice way to
split it except maybe "one register at a time" which would have been
a bit much perhaps.
>
> > - I915_WRITE_FW(DVSTILEOFF(pipe), (y << 16) | x);
> > - else
> > - I915_WRITE_FW(DVSLINOFF(pipe), linear_offset);
> > + I915_WRITE_FW(DVSTILEOFF(pipe), (y << 16) | x);
> > + I915_WRITE_FW(DVSLINOFF(pipe), linear_offset);
> >
> > - I915_WRITE_FW(DVSSIZE(pipe), (crtc_h << 16) | crtc_w);
> > - I915_WRITE_FW(DVSSCALE(pipe), dvsscale);
> > + /*
> > + * The control register self-arms if the plane was previously
> > + * disabled. Try to make the plane enable atomic by writing
> > + * the control register just before the surface register.
> > + */
> > I915_WRITE_FW(DVSCNTR(pipe), dvscntr);
> > I915_WRITE_FW(DVSSURF(pipe),
> > intel_plane_ggtt_offset(plane_state) + dvssurf_offset);
> > --
> > 2.18.1
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2018-11-07 21:38 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-01 15:05 [PATCH 00/14] drm/i915: Program SKL+ watermarks/ddb more carefully Ville Syrjala
2018-11-01 15:05 ` [PATCH 01/14] drm/i915: Nuke posting reads from plane update/disable funcs Ville Syrjala
2018-11-01 18:08 ` Rodrigo Vivi
2018-11-01 15:05 ` [PATCH 02/14] drm/i915: Clean up skl_program_scaler() Ville Syrjala
2018-11-01 15:17 ` [PATCH v2 " Ville Syrjala
2018-11-01 18:13 ` Rodrigo Vivi
2018-11-07 18:29 ` Ville Syrjälä
2018-11-09 17:24 ` Ville Syrjälä
2018-11-01 15:05 ` [PATCH 03/14] drm/i915: Remove the PS_PWR_GATE write from skl_program_scaler() Ville Syrjala
2018-11-07 18:53 ` Rodrigo Vivi
2018-11-01 15:05 ` [PATCH 04/14] drm/i915: Polish the skl+ plane keyval/msk/max register setup Ville Syrjala
2018-11-07 18:41 ` [PATCH v2 " Ville Syrjala
2018-11-07 19:55 ` [PATCH " Rodrigo Vivi
2018-11-07 20:56 ` Ville Syrjälä
2018-11-01 15:05 ` [PATCH 05/14] drm/i915: Clean up skl+ PLANE_POS vs. scaler handling Ville Syrjala
2018-11-07 19:56 ` Rodrigo Vivi
2018-11-01 15:05 ` [PATCH 06/14] drm/i915: Reorganize plane register writes to make them more atomic Ville Syrjala
2018-11-07 18:42 ` [PATCH v2 " Ville Syrjala
2018-11-08 19:30 ` Matt Roper
2018-11-08 19:48 ` Ville Syrjälä
2018-11-07 21:26 ` [PATCH " Rodrigo Vivi
2018-11-07 21:38 ` Ville Syrjälä [this message]
2018-11-01 15:05 ` [PATCH 07/14] drm/i915: Move single buffered plane register writes to the end Ville Syrjala
2018-11-07 21:26 ` Rodrigo Vivi
2018-11-08 22:06 ` Matt Roper
2018-11-09 13:55 ` Ville Syrjälä
2018-11-01 15:05 ` [PATCH 08/14] drm/i915: Generalize skl_ddb_allocation_overlaps() Ville Syrjala
2018-11-07 21:28 ` Rodrigo Vivi
2018-11-01 15:06 ` [PATCH 09/14] drm/i915: Introduce crtc_state->update_planes bitmask Ville Syrjala
2018-11-07 21:49 ` Rodrigo Vivi
2018-11-08 11:32 ` Ville Syrjälä
2018-11-08 23:22 ` Matt Roper
2018-11-09 13:57 ` Ville Syrjälä
2018-11-01 15:06 ` [PATCH 10/14] drm/i915: Pass the new crtc_state to ->disable_plane() Ville Syrjala
2018-11-07 22:08 ` Rodrigo Vivi
2018-11-08 11:43 ` Ville Syrjälä
2018-11-08 23:52 ` Matt Roper
2018-11-09 14:32 ` Ville Syrjälä
2018-11-01 15:06 ` [PATCH 11/14] drm/i915: Fix latency==0 handling for level 0 watermark on skl+ Ville Syrjala
2018-11-07 22:09 ` Rodrigo Vivi
2018-11-09 0:01 ` Matt Roper
2018-11-09 14:34 ` Ville Syrjälä
2018-11-01 15:06 ` [PATCH 12/14] drm/i915: Remove some useless zeroing on skl+ wm calculations Ville Syrjala
2018-11-07 18:43 ` [PATCH v2 " Ville Syrjala
2018-11-07 22:11 ` [PATCH " Rodrigo Vivi
2018-11-08 11:46 ` Ville Syrjälä
2018-11-01 15:06 ` [PATCH 13/14] drm/i915: Move ddb/wm programming into plane update/disable hooks on skl+ Ville Syrjala
2018-11-07 18:44 ` [PATCH v2 " Ville Syrjala
2018-11-09 1:38 ` Matt Roper
2018-11-09 14:53 ` Ville Syrjälä
2018-11-01 15:06 ` [PATCH 14/14] drm/i915: Commit skl+ planes in an order that avoids ddb overlaps Ville Syrjala
2018-11-01 15:08 ` ✗ Fi.CI.BAT: failure for drm/i915: Program SKL+ watermarks/ddb more carefully Patchwork
2018-11-01 16:04 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Program SKL+ watermarks/ddb more carefully (rev2) Patchwork
2018-11-01 16:09 ` ✗ Fi.CI.SPARSE: " Patchwork
2018-11-01 16:26 ` ✓ Fi.CI.BAT: success " Patchwork
2018-11-01 18:13 ` ✓ Fi.CI.IGT: " Patchwork
2018-11-07 19:01 ` ✗ Fi.CI.BAT: failure for drm/i915: Program SKL+ watermarks/ddb more carefully (rev6) Patchwork
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=20181107213837.GQ9144@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=rodrigo.vivi@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.