Re: [Intel-gfx] [PATCH 1/3] drm/i915/icl: Release TC ports when unloading or suspending driver

2018-11-29 Thread Imre Deak
On Wed, Nov 28, 2018 at 11:54:21PM +0200, Souza, Jose wrote:
> On Wed, 2018-11-28 at 13:34 +0200, Imre Deak wrote:
> > On Wed, Nov 07, 2018 at 04:05:52PM -0800, José Roberto de Souza
> > wrote:
> > > When suspending or unloading the driver, it needs to release the
> > > TC ports so HW can change it state without wait for driver
> > > handshake.
> > 
> > According to 
> > https://bugs.freedesktop.org/show_bug.cgi?id=108070#c26
> > 
> > this patch should fix the bug reported at
> > https://bugs.freedesktop.org/show_bug.cgi?id=108070#c17
> > 
> > but, I can't see how the change here would fix the corresponding
> > problem
> > described in
> > https://bugs.freedesktop.org/show_bug.cgi?id=108070#c18
> > 
> > Would you explain?
> > 
> > I think there are more fundamental problems in TypeC HPD handling as
> > we
> > discussed earlier here on the list, which should be fixed first:
> > 
> > - Switching to/from type C mode from the interrupt handler without
> >   considering if we have an active mode or an ongoing AUX transfer.
> 
> Yes that is still missing, I plan to work on that after I deliver a few
> tasks. Also I read the emails in discussion that you started about
> that.

The point is that imo those fundamental problems should be fixed first.

> 
> > 
> > - Not having any way for handling the case where we'd do a modeset
> > after
> >   an HPD disconnect interrupt, like in the case of
> >   common-hpd-after-suspend in the bug report.
> 
> For what I understood about this test it will suspend, disconnect
> chamelium board from source, wakeup and check if connector state match,
> as we did not marked port as safe before suspending we are left in a
> invalid state where PD firmware do not trigger a interruption after
> wakeup.

No, it's not about the lack of HPD interrupts. What happens is that the
sink disappears during system suspend/resume, but we still have an
active mode enabled on the corresponding port. We try to restore this
mode during resume but that modeset will fail.

> 
> > 
> > More comments below:
> > 
> > > Cc: Imre Deak 
> > > Cc: Ville Syrjälä 
> > > Signed-off-by: José Roberto de Souza 
> > > ---
> > >  drivers/gpu/drm/i915/i915_drv.c  |  1 +
> > >  drivers/gpu/drm/i915/i915_drv.h  |  1 +
> > >  drivers/gpu/drm/i915/i915_irq.c  |  1 +
> > >  drivers/gpu/drm/i915/intel_dp.c  | 19 +++
> > >  drivers/gpu/drm/i915/intel_drv.h |  1 +
> > >  drivers/gpu/drm/i915/intel_hotplug.c |  9 +
> > >  6 files changed, 32 insertions(+)
> > > 
> > > diff --git a/drivers/gpu/drm/i915/i915_drv.c
> > > b/drivers/gpu/drm/i915/i915_drv.c
> > > index acb516308262..14331c396278 100644
> > > --- a/drivers/gpu/drm/i915/i915_drv.c
> > > +++ b/drivers/gpu/drm/i915/i915_drv.c
> > > @@ -1911,6 +1911,7 @@ static int i915_drm_suspend(struct drm_device
> > > *dev)
> > >  
> > >   intel_runtime_pm_disable_interrupts(dev_priv);
> > >   intel_hpd_cancel_work(dev_priv);
> > > + intel_hpd_suspend(dev_priv);
> > 
> > What about runtime suspend? We won't receive HPD interrupts after
> > that
> > either, so we'd need to disconnect in that case too according to the
> > spec.
> 
> Oh my bad I was thinking that the regular suspend would handle this too
> but I was wrong.
> 
> So I will add a intel_hpd_suspend() call to intel_runtime_suspend()
> leaving the i915_drv.c changes as this.
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.c
> b/drivers/gpu/drm/i915/i915_drv.c
> index b1d23c73c147..948914a79c67 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -1905,6 +1905,7 @@ static int i915_drm_suspend(struct drm_device
> *dev)
> 
> intel_runtime_pm_disable_interrupts(dev_priv);
> intel_hpd_cancel_work(dev_priv);
> +   intel_hpd_suspend(dev_priv);
> 
> intel_suspend_encoders(dev_priv);
> 
> @@ -2909,6 +2910,7 @@ static int intel_runtime_suspend(struct device
> *kdev)
> intel_uc_suspend(dev_priv);
> 
> intel_runtime_pm_disable_interrupts(dev_priv);
> +   intel_hpd_suspend(dev_priv);
> 
> intel_uncore_suspend(dev_priv);
> 
> Also I will drop the next patch as we don't call
> intel_hpd_cancel_work() in intel_runtime_suspend() path.
> 
> 
> > 
> > And how are we handling resume then where we'll be in a disconnected
> > state due to the change in this patch and would continue to do a
> > modeset (to restore the mode we had before suspend).
> 
> The drm_kms_helper_poll_enable() calls will trigger the call to the
> detection function of the connectors that will call
> intel_digital_port_connected() thal will call ICL TC functions that
> will take care of update everything.

No detection will be triggered, since we're not polling this port (and
polling can be disabled as a whole). In any case that would be too late,
since we're trying to restore the mode before we call
drm_kms_helper_poll_enable().  Also doing a detection is not enough
here, since if detection fails (as it would in our case) we still have a

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/backlight: Restore backlight on resume (rev2)

2018-11-29 Thread Patchwork
== Series Details ==

Series: series starting with [1/2] drm/i915/backlight: Restore backlight on 
resume (rev2)
URL   : https://patchwork.freedesktop.org/series/53239/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5222 -> Patchwork_10953


Summary
---

  **SUCCESS**

  No regressions found.

  External URL: 
https://patchwork.freedesktop.org/api/1.0/series/53239/revisions/2/mbox/

Known issues


  Here are the changes found in Patchwork_10953 that come from known issues:

### IGT changes ###

 Issues hit 

  * {igt@runner@aborted}:
- {fi-icl-y}: NOTRUN -> FAIL [fdo#108070]

  
 Possible fixes 

  * igt@gem_mmap_gtt@basic:
- fi-glk-dsi: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
  the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#108070]: https://bugs.freedesktop.org/show_bug.cgi?id=108070
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (49 -> 44)
--

  Additional (1): fi-icl-y 
  Missing(6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks 
fi-bsw-cyan fi-ctg-p8600 


Build changes
-

* Linux: CI_DRM_5222 -> Patchwork_10953

  CI_DRM_5222: a6f85043a0ca86eb0072e69bf92b77f7d9d6d5d5 @ 
git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4735: b05c028ccdb6ac8e8d8499a041bb14dfe358ee26 @ 
git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10953: bbcaa34deb2437826b94f61b7fa80d81df2137c0 @ 
git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

bbcaa34deb24 drm/i915/backlight: Fix backlight takeover on LPT, v2.
65214d12d27f drm/i915/backlight: Restore backlight on resume

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10953/
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] ✓ Fi.CI.BAT: success for Add support for Gen 11 pipe color features (rev3)

2018-11-29 Thread Patchwork
== Series Details ==

Series: Add support for Gen 11 pipe color features (rev3)
URL   : https://patchwork.freedesktop.org/series/51408/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5222 -> Patchwork_10957


Summary
---

  **SUCCESS**

  No regressions found.

  External URL: 
https://patchwork.freedesktop.org/api/1.0/series/51408/revisions/3/mbox/

Known issues


  Here are the changes found in Patchwork_10957 that come from known issues:

### IGT changes ###

 Issues hit 

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
- fi-byt-clapper: PASS -> FAIL [fdo#103191] / [fdo#107362]

  * {igt@runner@aborted}:
- {fi-icl-y}: NOTRUN -> FAIL [fdo#108070]

  
 Possible fixes 

  * igt@gem_mmap_gtt@basic:
- fi-glk-dsi: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
  the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#108070]: https://bugs.freedesktop.org/show_bug.cgi?id=108070
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (49 -> 44)
--

  Additional (1): fi-icl-y 
  Missing(6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks 
fi-bsw-cyan fi-ctg-p8600 


Build changes
-

* Linux: CI_DRM_5222 -> Patchwork_10957

  CI_DRM_5222: a6f85043a0ca86eb0072e69bf92b77f7d9d6d5d5 @ 
git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4735: b05c028ccdb6ac8e8d8499a041bb14dfe358ee26 @ 
git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10957: 653e346a63d4c9f5d2d2e559ed0b73ea3cce7029 @ 
git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

653e346a63d4 drm/i915/icl: Add degamma and gamma lut size to gen11 caps
5a32a67a4649 drm/i915/icl: Enable ICL Pipe CSC block
1427600ab4e5 drm/i915/icl: Add icl pipe degamma and gamma support

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10957/
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


Re: [Intel-gfx] [PATCH] drm: Fix up drm_atomic_state_helper.[hc] extraction

2018-11-29 Thread Sean Paul
On Wed, Nov 28, 2018 at 5:07 AM Daniel Vetter  wrote:
>
> I've misplaced two functions by accident:
> - drm_atomic_helper_duplicate_state is really part of the
>   resume/suspend/shutdown device-wide helpers.
> - drm_atomic_helper_legacy_gamma_set is part of the legacy ioctl
>   compat helpers.
>
> Move them both back.
>
> Fixes: 9ef8a9dc4b21 ("drm: Extract drm_atomic_state_helper.[hc]")
> Cc: Ville Syrjälä 
> Signed-off-by: Daniel Vetter 

Reviewed-by: Sean Paul 

> ---
>  drivers/gpu/drm/drm_atomic_helper.c   | 157 ++
>  drivers/gpu/drm/drm_atomic_state_helper.c | 157 --
>  include/drm/drm_atomic_helper.h   |   7 +
>  include/drm/drm_atomic_state_helper.h |   7 -
>  4 files changed, 164 insertions(+), 164 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c 
> b/drivers/gpu/drm/drm_atomic_helper.c
> index 9b22774a9867..de7d872f9f1a 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -3135,6 +3135,93 @@ void drm_atomic_helper_shutdown(struct drm_device *dev)
>  }
>  EXPORT_SYMBOL(drm_atomic_helper_shutdown);
>
> +/**
> + * drm_atomic_helper_duplicate_state - duplicate an atomic state object
> + * @dev: DRM device
> + * @ctx: lock acquisition context
> + *
> + * Makes a copy of the current atomic state by looping over all objects and
> + * duplicating their respective states. This is used for example by suspend/
> + * resume support code to save the state prior to suspend such that it can
> + * be restored upon resume.
> + *
> + * Note that this treats atomic state as persistent between save and restore.
> + * Drivers must make sure that this is possible and won't result in confusion
> + * or erroneous behaviour.
> + *
> + * Note that if callers haven't already acquired all modeset locks this might
> + * return -EDEADLK, which must be handled by calling drm_modeset_backoff().
> + *
> + * Returns:
> + * A pointer to the copy of the atomic state object on success or an
> + * ERR_PTR()-encoded error code on failure.
> + *
> + * See also:
> + * drm_atomic_helper_suspend(), drm_atomic_helper_resume()
> + */
> +struct drm_atomic_state *
> +drm_atomic_helper_duplicate_state(struct drm_device *dev,
> + struct drm_modeset_acquire_ctx *ctx)
> +{
> +   struct drm_atomic_state *state;
> +   struct drm_connector *conn;
> +   struct drm_connector_list_iter conn_iter;
> +   struct drm_plane *plane;
> +   struct drm_crtc *crtc;
> +   int err = 0;
> +
> +   state = drm_atomic_state_alloc(dev);
> +   if (!state)
> +   return ERR_PTR(-ENOMEM);
> +
> +   state->acquire_ctx = ctx;
> +
> +   drm_for_each_crtc(crtc, dev) {
> +   struct drm_crtc_state *crtc_state;
> +
> +   crtc_state = drm_atomic_get_crtc_state(state, crtc);
> +   if (IS_ERR(crtc_state)) {
> +   err = PTR_ERR(crtc_state);
> +   goto free;
> +   }
> +   }
> +
> +   drm_for_each_plane(plane, dev) {
> +   struct drm_plane_state *plane_state;
> +
> +   plane_state = drm_atomic_get_plane_state(state, plane);
> +   if (IS_ERR(plane_state)) {
> +   err = PTR_ERR(plane_state);
> +   goto free;
> +   }
> +   }
> +
> +   drm_connector_list_iter_begin(dev, _iter);
> +   drm_for_each_connector_iter(conn, _iter) {
> +   struct drm_connector_state *conn_state;
> +
> +   conn_state = drm_atomic_get_connector_state(state, conn);
> +   if (IS_ERR(conn_state)) {
> +   err = PTR_ERR(conn_state);
> +   drm_connector_list_iter_end(_iter);
> +   goto free;
> +   }
> +   }
> +   drm_connector_list_iter_end(_iter);
> +
> +   /* clear the acquire context so that it isn't accidentally reused */
> +   state->acquire_ctx = NULL;
> +
> +free:
> +   if (err < 0) {
> +   drm_atomic_state_put(state);
> +   state = ERR_PTR(err);
> +   }
> +
> +   return state;
> +}
> +EXPORT_SYMBOL(drm_atomic_helper_duplicate_state);
> +
>  /**
>   * drm_atomic_helper_suspend - subsystem-level suspend helper
>   * @dev: DRM device
> @@ -3418,3 +3505,73 @@ int drm_atomic_helper_page_flip_target(struct drm_crtc 
> *crtc,
> return ret;
>  }
>  EXPORT_SYMBOL(drm_atomic_helper_page_flip_target);
> +
> +/**
> + * drm_atomic_helper_legacy_gamma_set - set the legacy gamma correction table
> + * @crtc: CRTC object
> + * @red: red correction table
> + * @green: green correction table
> + * @blue: green correction table
> + * @size: size of the tables
> + * @ctx: lock acquire context
> + *
> + * Implements support for legacy gamma correction table for drivers
> + * that support color management through the DEGAMMA_LUT/GAMMA_LUT
> + * properties. See 

[Intel-gfx] [PATCH] drm/i915/ddi: Check for unexpectedly disabled transcoders

2018-11-29 Thread Imre Deak
The DDI eDP/pipe A/B/C transcoders must be enabled whenever we observe
the DDI buffer of the port connected to them to be enabled (via
DDI_BUF_CTL). Add a consistency check for this to the HW readout code of
these transcoders.

Signed-off-by: Imre Deak 
---
 drivers/gpu/drm/i915/intel_ddi.c | 4 
 1 file changed, 4 insertions(+)

diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
index ad11540ac436..0476d1d33bfb 100644
--- a/drivers/gpu/drm/i915/intel_ddi.c
+++ b/drivers/gpu/drm/i915/intel_ddi.c
@@ -2029,6 +2029,8 @@ static void intel_ddi_get_encoder_pipes(struct 
intel_encoder *encoder,
if (port == PORT_A) {
tmp = I915_READ(TRANS_DDI_FUNC_CTL(TRANSCODER_EDP));
 
+   WARN_ON(!(tmp & TRANS_DDI_FUNC_ENABLE));
+
switch (tmp & TRANS_DDI_EDP_INPUT_MASK) {
default:
MISSING_CASE(tmp & TRANS_DDI_EDP_INPUT_MASK);
@@ -2057,6 +2059,8 @@ static void intel_ddi_get_encoder_pipes(struct 
intel_encoder *encoder,
if ((tmp & TRANS_DDI_PORT_MASK) != TRANS_DDI_SELECT_PORT(port))
continue;
 
+   WARN_ON(!(tmp & TRANS_DDI_FUNC_ENABLE));
+
if ((tmp & TRANS_DDI_MODE_SELECT_MASK) ==
TRANS_DDI_MODE_SELECT_DP_MST)
mst_pipe_mask |= BIT(p);
-- 
2.13.2

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/cnl: Fix the formulae for register offsets (rev2)

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915/cnl: Fix the formulae for register offsets (rev2)
URL   : https://patchwork.freedesktop.org/series/52960/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5220_full -> Patchwork_10944_full


Summary
---

  **WARNING**

  Minor unknown changes coming with Patchwork_10944_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_10944_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
---

  Here are the unknown changes that may have been introduced in 
Patchwork_10944_full:

### IGT changes ###

 Warnings 

  * igt@tools_test@tools_test:
- shard-hsw:  SKIP -> PASS

  
Known issues


  Here are the changes found in Patchwork_10944_full that come from known 
issues:

### IGT changes ###

 Issues hit 

  * igt@gem_ctx_isolation@vecs0-s3:
- shard-kbl:  PASS -> INCOMPLETE [fdo#103665]

  * igt@gem_eio@in-flight-1us:
- shard-glk:  PASS -> INCOMPLETE [fdo#103359] / [k.org#198133]

  * igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels:
- {shard-iclb}:   PASS -> DMESG-WARN [fdo#107724] +8

  * igt@kms_busy@extended-modeset-hang-newfb-render-b:
- shard-kbl:  PASS -> DMESG-WARN [fdo#107956]
- shard-snb:  NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
- shard-hsw:  PASS -> DMESG-WARN [fdo#107956]

  * igt@kms_cursor_crc@cursor-64x21-sliding:
- shard-apl:  PASS -> FAIL [fdo#103232]

  * igt@kms_cursor_crc@cursor-64x64-suspend:
- shard-apl:  PASS -> FAIL [fdo#103191] / [fdo#103232]

  * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt:
- {shard-iclb}:   PASS -> DMESG-FAIL [fdo#107724] +2

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt:
- {shard-iclb}:   PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite:
- shard-apl:  PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-gtt:
- shard-glk:  PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-render:
- {shard-iclb}:   PASS -> DMESG-WARN [fdo#107724] / [fdo#108336] +2

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
- shard-apl:  PASS -> INCOMPLETE [fdo#103927]

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
- {shard-iclb}:   PASS -> DMESG-FAIL [fdo#103166] / [fdo#107724]

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-y:
- {shard-iclb}:   PASS -> FAIL [fdo#103166] +1

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-y:
- shard-apl:  PASS -> FAIL [fdo#103166]

  * igt@kms_psr@no_drrs:
- {shard-iclb}:   PASS -> FAIL [fdo#108341]

  * igt@kms_rotation_crc@primary-rotation-180:
- shard-snb:  NOTRUN -> FAIL [fdo#103925]

  * igt@kms_setmode@basic:
- shard-kbl:  PASS -> FAIL [fdo#99912]

  * igt@pm_rpm@universal-planes:
- {shard-iclb}:   PASS -> DMESG-WARN [fdo#108654]

  
 Possible fixes 

  * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
- shard-snb:  DMESG-WARN [fdo#107956] -> PASS

  * igt@kms_color@pipe-a-degamma:
- shard-apl:  FAIL [fdo#104782] / [fdo#108145] -> PASS

  * igt@kms_color@pipe-b-degamma:
- shard-apl:  FAIL [fdo#104782] -> PASS +1

  * igt@kms_cursor_crc@cursor-256x256-suspend:
- shard-apl:  FAIL [fdo#103191] / [fdo#103232] -> PASS

  * igt@kms_cursor_crc@cursor-256x85-onscreen:
- shard-apl:  FAIL [fdo#103232] -> PASS +2

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-glk:  FAIL [fdo#102887] / [fdo#105363] -> PASS

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-move:
- {shard-iclb}:   FAIL [fdo#103167] -> PASS

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-yf:
- shard-apl:  FAIL [fdo#103166] -> PASS

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-x:
- {shard-iclb}:   FAIL [fdo#103166] -> PASS +1

  * igt@kms_setmode@basic:
- shard-apl:  FAIL [fdo#99912] -> PASS

  * igt@pm_rpm@basic-pci-d3-state:
- {shard-iclb}:   INCOMPLETE [fdo#108840] -> PASS +1

  
 Warnings 

  * igt@kms_cursor_crc@cursor-64x64-suspend:
- {shard-iclb}:   FAIL [fdo#103232] -> INCOMPLETE [fdo#107713]

  
  {name}: This element is suppressed. This means it is ignored when computing
  the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#102887]: https://bugs.freedesktop.org/show_bug.cgi?id=102887
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add support for Gen 11 pipe color features (rev3)

2018-11-29 Thread Patchwork
== Series Details ==

Series: Add support for Gen 11 pipe color features (rev3)
URL   : https://patchwork.freedesktop.org/series/51408/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
1427600ab4e5 drm/i915/icl: Add icl pipe degamma and gamma support
-:74: CHECK:SPACING: No space is necessary after a cast
#74: FILE: drivers/gpu/drm/i915/intel_color.c:548:
+   (struct drm_color_lut *) state->degamma_lut->data;

total: 0 errors, 0 warnings, 1 checks, 112 lines checked
5a32a67a4649 drm/i915/icl: Enable ICL Pipe CSC block
653e346a63d4 drm/i915/icl: Add degamma and gamma lut size to gen11 caps

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/icl: dsi enabling (rev6)

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915/icl: dsi enabling (rev6)
URL   : https://patchwork.freedesktop.org/series/51011/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5222 -> Patchwork_10956


Summary
---

  **SUCCESS**

  No regressions found.

  External URL: 
https://patchwork.freedesktop.org/api/1.0/series/51011/revisions/6/mbox/

Possible new issues
---

  Here are the unknown changes that may have been introduced in Patchwork_10956:

### IGT changes ###

 Possible regressions 

  * igt@kms_busy@basic-flip-b:
- {fi-icl-u3}:PASS -> DMESG-WARN

  * {igt@runner@aborted}:
- {fi-icl-u3}:NOTRUN -> FAIL

  
Known issues


  Here are the changes found in Patchwork_10956 that come from known issues:

### IGT changes ###

 Issues hit 

  * {igt@runner@aborted}:
- {fi-icl-y}: NOTRUN -> FAIL [fdo#108070]

  
 Possible fixes 

  * igt@gem_mmap_gtt@basic:
- fi-glk-dsi: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
  the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#108070]: https://bugs.freedesktop.org/show_bug.cgi?id=108070
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (49 -> 43)
--

  Additional (1): fi-icl-y 
  Missing(7): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks 
fi-bsw-cyan fi-ctg-p8600 fi-pnv-d510 


Build changes
-

* Linux: CI_DRM_5222 -> Patchwork_10956

  CI_DRM_5222: a6f85043a0ca86eb0072e69bf92b77f7d9d6d5d5 @ 
git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4735: b05c028ccdb6ac8e8d8499a041bb14dfe358ee26 @ 
git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10956: d5ba18915e1b1b609b8c361634dbec396627126d @ 
git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

d5ba18915e1b HACK: drm/i915/bios: ignore VBT not overflowing the mailbox
baf028ec4d0d HACK: drm/i915/icl: Configure backlight functions for DSI
e11d2a74b0e8 HACK: drm/i915/icl: Add changes to program DSI panel GPIOs
9ec5126e3d2a drm/i915/icl: add pll mapping for DSI
9ae624b30832 drm/i915/icl: Ungate DSI clocks
583b019c2b85 drm/i915/icl: Gate clocks for DSI
5a1f35547955 drm/i915/icl: add dummy DSI GPIO element execution function
570c6994a5d9 drm/i915/icl: Define display GPIO pins for DSI
3c816c13f322 drm/i915/icl: Define Panel power ctrl register
f80c2ca9d8c4 drm/i915/icl: Define missing bitfield for shortplug reg
b3273c98d705 drm/i915/icl: Get pipe timings for DSI
1bd7b569b632 drm/i915/icl: Consider DSI for getting transcoder state
7db8401305da drm/i915/icl: Configure DSI Dual link mode
63e5965d02c1 drm/i915/icl: Add DSI encoder compute config hook
aa8fb2448b1f drm/i915/icl: Get HW state for DSI encoder
82bf01085a14 drm/i915/icl: Add get config functionality for DSI
7fad6a3ad9da drm/i915/icl: Allocate DSI hosts and imlement host transfer
4dac6888a145 drm/i915/icl: Fill DSI ports info
1556283cacab drm/i915/icl: Use the same pll functions for dsi
6dc7fe9047df drm/i915/icl: Allocate DSI encoder/connector
0f5edfc63f35 drm/i915/icl: Calculate DPLL params for DSI
37ef11ea7aa7 drm/i915/icl: Sanitize DDI port clock gating for DSI ports
a65617b53ade drm/i915/icl: push pll to port mapping/unmapping to ddi encoder 
hooks

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10956/
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Fix the HDMI hot plug disconnection failure (rev4)

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915: Fix the HDMI hot plug disconnection failure (rev4)
URL   : https://patchwork.freedesktop.org/series/50477/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5220_full -> Patchwork_10943_full


Summary
---

  **WARNING**

  Minor unknown changes coming with Patchwork_10943_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_10943_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
---

  Here are the unknown changes that may have been introduced in 
Patchwork_10943_full:

### IGT changes ###

 Warnings 

  * igt@perf_pmu@rc6:
- shard-kbl:  PASS -> SKIP

  * igt@pm_rc6_residency@rc6-accuracy:
- shard-snb:  PASS -> SKIP

  * igt@tools_test@tools_test:
- shard-hsw:  SKIP -> PASS

  
Known issues


  Here are the changes found in Patchwork_10943_full that come from known 
issues:

### IGT changes ###

 Issues hit 

  * igt@gem_ctx_isolation@vecs0-s3:
- shard-kbl:  PASS -> INCOMPLETE [fdo#103665]

  * igt@gem_exec_schedule@pi-ringfull-vebox:
- shard-skl:  NOTRUN -> FAIL [fdo#103158]

  * igt@kms_busy@extended-modeset-hang-newfb-render-b:
- shard-kbl:  PASS -> DMESG-WARN [fdo#107956]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
- shard-hsw:  PASS -> DMESG-WARN [fdo#107956]

  * igt@kms_cursor_crc@cursor-256x256-dpms:
- shard-glk:  PASS -> FAIL [fdo#103232] +1

  * igt@kms_cursor_crc@cursor-256x256-offscreen:
- shard-skl:  PASS -> FAIL [fdo#103232]

  * igt@kms_fbcon_fbt@psr:
- shard-skl:  NOTRUN -> FAIL [fdo#107882]

  * igt@kms_flip@plain-flip-fb-recreate-interruptible:
- shard-skl:  PASS -> FAIL [fdo#100368]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
- shard-apl:  PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-1p-rte:
- shard-glk:  PASS -> FAIL [fdo#103167] / [fdo#105682]

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-cpu:
- shard-glk:  PASS -> FAIL [fdo#103167] +2

  * igt@kms_frontbuffer_tracking@fbc-stridechange:
- {shard-iclb}:   PASS -> FAIL [fdo#105683] +1

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-render:
- shard-skl:  NOTRUN -> FAIL [fdo#103167]

  * igt@kms_panel_fitting@legacy:
- shard-skl:  NOTRUN -> FAIL [fdo#105456]

  * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
- shard-skl:  NOTRUN -> FAIL [fdo#108145] +1

  * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-mid:
- shard-apl:  PASS -> DMESG-WARN [fdo#103558] / [fdo#105602] +10

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-y:
- {shard-iclb}:   PASS -> FAIL [fdo#103166] +1

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-yf:
- shard-apl:  PASS -> DMESG-FAIL [fdo#103166] / [fdo#103558] / 
[fdo#105602]

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-y:
- shard-apl:  PASS -> FAIL [fdo#103166]

  * igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format:
- {shard-iclb}:   PASS -> DMESG-WARN [fdo#107724]

  
 Possible fixes 

  * igt@drm_import_export@import-close-race-flink:
- shard-skl:  TIMEOUT [fdo#108667] -> PASS

  * igt@kms_color@pipe-b-degamma:
- shard-apl:  FAIL [fdo#104782] -> PASS +1

  * igt@kms_cursor_crc@cursor-256x256-onscreen:
- shard-glk:  FAIL [fdo#103232] -> PASS

  * igt@kms_cursor_crc@cursor-256x256-suspend:
- shard-apl:  FAIL [fdo#103191] / [fdo#103232] -> PASS

  * igt@kms_cursor_crc@cursor-256x85-onscreen:
- shard-apl:  FAIL [fdo#103232] -> PASS +2

  * igt@kms_flip_tiling@flip-to-y-tiled:
- {shard-iclb}:   DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +1

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff:
- shard-glk:  FAIL [fdo#103167] -> PASS

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt:
- {shard-iclb}:   DMESG-FAIL [fdo#107724] -> PASS

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-move:
- {shard-iclb}:   FAIL [fdo#103167] -> PASS +1

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-x:
- {shard-iclb}:   FAIL [fdo#103166] -> PASS +1

  * igt@kms_setmode@basic:
- shard-apl:  FAIL [fdo#99912] -> PASS

  * igt@kms_sysfs_edid_timing:
- shard-skl:  FAIL [fdo#100047] -> PASS
- shard-kbl:  FAIL [fdo#100047] -> PASS

  * igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend:
- {shard-iclb}:   DMESG-WARN [fdo#107724] -> PASS +5

  * igt@pm_rpm@basic-pci-d3-state:
- {shard-iclb}:   INCOMPLETE [fdo#108840] -> 

[Intel-gfx] [PATCH 1/2] drm/i915: s/PUNIT_REG_DSPFREQ/PUNIT_REG_DSPSSPM/

2018-11-29 Thread Ville Syrjala
From: Ville Syrjälä 

Rename the punit display power register to match the spec.

Signed-off-by: Ville Syrjälä 
---
 drivers/gpu/drm/i915/i915_reg.h |  2 +-
 drivers/gpu/drm/i915/intel_cdclk.c  | 14 +++---
 drivers/gpu/drm/i915/intel_pm.c |  6 +++---
 drivers/gpu/drm/i915/intel_runtime_pm.c | 12 ++--
 4 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 47baf2fe8f71..42d25872ecfe 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -1020,7 +1020,7 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg)
 /* See configdb bunit SB addr map */
 #define BUNIT_REG_BISOC0x11
 
-#define PUNIT_REG_DSPFREQ  0x36
+#define PUNIT_REG_DSPSSPM  0x36
 #define   DSPFREQSTAT_SHIFT_CHV24
 #define   DSPFREQSTAT_MASK_CHV (0x1f << DSPFREQSTAT_SHIFT_CHV)
 #define   DSPFREQGUAR_SHIFT_CHV8
diff --git a/drivers/gpu/drm/i915/intel_cdclk.c 
b/drivers/gpu/drm/i915/intel_cdclk.c
index 25e3aba9cded..82f60f8a15e9 100644
--- a/drivers/gpu/drm/i915/intel_cdclk.c
+++ b/drivers/gpu/drm/i915/intel_cdclk.c
@@ -468,7 +468,7 @@ static void vlv_get_cdclk(struct drm_i915_private *dev_priv,
   cdclk_state->vco);
 
mutex_lock(_priv->pcu_lock);
-   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ);
+   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM);
mutex_unlock(_priv->pcu_lock);
 
if (IS_VALLEYVIEW(dev_priv))
@@ -542,11 +542,11 @@ static void vlv_set_cdclk(struct drm_i915_private 
*dev_priv,
intel_display_power_get(dev_priv, POWER_DOMAIN_PIPE_A);
 
mutex_lock(_priv->pcu_lock);
-   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ);
+   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM);
val &= ~DSPFREQGUAR_MASK;
val |= (cmd << DSPFREQGUAR_SHIFT);
-   vlv_punit_write(dev_priv, PUNIT_REG_DSPFREQ, val);
-   if (wait_for((vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ) &
+   vlv_punit_write(dev_priv, PUNIT_REG_DSPSSPM, val);
+   if (wait_for((vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM) &
  DSPFREQSTAT_MASK) == (cmd << DSPFREQSTAT_SHIFT),
 50)) {
DRM_ERROR("timed out waiting for CDclk change\n");
@@ -622,11 +622,11 @@ static void chv_set_cdclk(struct drm_i915_private 
*dev_priv,
intel_display_power_get(dev_priv, POWER_DOMAIN_PIPE_A);
 
mutex_lock(_priv->pcu_lock);
-   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ);
+   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM);
val &= ~DSPFREQGUAR_MASK_CHV;
val |= (cmd << DSPFREQGUAR_SHIFT_CHV);
-   vlv_punit_write(dev_priv, PUNIT_REG_DSPFREQ, val);
-   if (wait_for((vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ) &
+   vlv_punit_write(dev_priv, PUNIT_REG_DSPSSPM, val);
+   if (wait_for((vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM) &
  DSPFREQSTAT_MASK_CHV) == (cmd << DSPFREQSTAT_SHIFT_CHV),
 50)) {
DRM_ERROR("timed out waiting for CDclk change\n");
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index a26b4eddda25..a67ec01dd958 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -335,12 +335,12 @@ static void chv_set_memory_pm5(struct drm_i915_private 
*dev_priv, bool enable)
 
mutex_lock(_priv->pcu_lock);
 
-   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ);
+   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM);
if (enable)
val |= DSP_MAXFIFO_PM5_ENABLE;
else
val &= ~DSP_MAXFIFO_PM5_ENABLE;
-   vlv_punit_write(dev_priv, PUNIT_REG_DSPFREQ, val);
+   vlv_punit_write(dev_priv, PUNIT_REG_DSPSSPM, val);
 
mutex_unlock(_priv->pcu_lock);
 }
@@ -6082,7 +6082,7 @@ void vlv_wm_get_hw_state(struct drm_device *dev)
if (IS_CHERRYVIEW(dev_priv)) {
mutex_lock(_priv->pcu_lock);
 
-   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ);
+   val = vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM);
if (val & DSP_MAXFIFO_PM5_ENABLE)
wm->level = VLV_WM_LEVEL_PM5;
 
diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c 
b/drivers/gpu/drm/i915/intel_runtime_pm.c
index 1c2de9b69a19..af499d4a0c66 100644
--- a/drivers/gpu/drm/i915/intel_runtime_pm.c
+++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
@@ -1494,7 +1494,7 @@ static bool chv_pipe_power_well_enabled(struct 
drm_i915_private *dev_priv,
 
mutex_lock(_priv->pcu_lock);
 
-   state = vlv_punit_read(dev_priv, PUNIT_REG_DSPFREQ) & DP_SSS_MASK(pipe);
+   state = vlv_punit_read(dev_priv, PUNIT_REG_DSPSSPM) & DP_SSS_MASK(pipe);
/*
 * We only ever set the 

[Intel-gfx] [PATCH 2/2] drm/i915: Assert that VED and ISP are power gated

2018-11-29 Thread Ville Syrjala
From: Ville Syrjälä 

As there are no upstream drivers for VED or ISP let's just
assert that they are power gated. Otherwise they would
prevent s0ix entry.

For ISP this is only relevant when it is not exposed as a
PCI device and instead is a subordinate of the gunit. When
exposed as a PCI device it will be handled by the
atomisp2_pm driver.

On my VLV FFRD8 board the firmware power gates both of these
by default. Let's assume that is always the case and just
WARN if we ever encounter something different.

Cc: Hans de Goede 
Cc: Alan Cox 
Cc: Andy Shevchenko 
Signed-off-by: Ville Syrjälä 
---
 drivers/gpu/drm/i915/i915_reg.h | 28 +
 drivers/gpu/drm/i915/intel_runtime_pm.c | 33 +
 2 files changed, 61 insertions(+)

diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 42d25872ecfe..c2ecac4cb51b 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -1020,6 +1020,31 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg)
 /* See configdb bunit SB addr map */
 #define BUNIT_REG_BISOC0x11
 
+/* PUNIT_REG_*SSPM0 */
+#define   _SSPM0_SSC(val)  ((val) << 0)
+#define   SSPM0_SSC_MASK   _SSPM0_SSC(0x3)
+#define   SSPM0_SSC_PWR_ON _SSPM0_SSC(0x0)
+#define   SSPM0_SSC_CLK_GATE   _SSPM0_SSC(0x1)
+#define   SSPM0_SSC_RESET  _SSPM0_SSC(0x2)
+#define   SSPM0_SSC_PWR_GATE   _SSPM0_SSC(0x3)
+#define   _SSPM0_SSS(val)  ((val) << 24)
+#define   SSPM0_SSS_MASK   _SSPM0_SSS(0x3)
+#define   SSPM0_SSS_PWR_ON _SSPM0_SSS(0x0)
+#define   SSPM0_SSS_CLK_GATE   _SSPM0_SSS(0x1)
+#define   SSPM0_SSS_RESET  _SSPM0_SSS(0x2)
+#define   SSPM0_SSS_PWR_GATE   _SSPM0_SSS(0x3)
+
+/* PUNIT_REG_*SSPM1 */
+#define   SSPM1_FREQSTAT_SHIFT 24
+#define   SSPM1_FREQSTAT_MASK  (0x1f << SSPM1_FREQSTAT_SHIFT)
+#define   SSPM1_FREQGUAR_SHIFT 8
+#define   SSPM1_FREQGUAR_MASK  (0x1f << SSPM1_FREQGUAR_SHIFT)
+#define   SSPM1_FREQ_SHIFT 0
+#define   SSPM1_FREQ_MASK  (0x1f << SSPM1_FREQ_SHIFT)
+
+#define PUNIT_REG_VEDSSPM0 0x32
+#define PUNIT_REG_VEDSSPM1 0x33
+
 #define PUNIT_REG_DSPSSPM  0x36
 #define   DSPFREQSTAT_SHIFT_CHV24
 #define   DSPFREQSTAT_MASK_CHV (0x1f << DSPFREQSTAT_SHIFT_CHV)
@@ -1045,6 +1070,9 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg)
 #define   DP_SSS_RESET(pipe)   _DP_SSS(0x2, (pipe))
 #define   DP_SSS_PWR_GATE(pipe)_DP_SSS(0x3, (pipe))
 
+#define PUNIT_REG_ISPSSPM0 0x39
+#define PUNIT_REG_ISPSSPM1 0x3a
+
 /*
  * i915_power_well_id:
  *
diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c 
b/drivers/gpu/drm/i915/intel_runtime_pm.c
index af499d4a0c66..416521f45ec7 100644
--- a/drivers/gpu/drm/i915/intel_runtime_pm.c
+++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
@@ -3706,6 +3706,36 @@ static void vlv_cmnlane_wa(struct drm_i915_private 
*dev_priv)
cmn->desc->ops->disable(dev_priv, cmn);
 }
 
+static bool vlv_punit_is_power_gated(struct drm_i915_private *dev_priv, u32 
reg0)
+{
+   bool ret;
+
+   mutex_lock(_priv->pcu_lock);
+   ret = (vlv_punit_read(dev_priv, reg0) & SSPM0_SSC_MASK) == 
SSPM0_SSC_PWR_GATE;
+   mutex_unlock(_priv->pcu_lock);
+
+   return ret;
+}
+
+static void assert_ved_power_gated(struct drm_i915_private *dev_priv)
+{
+   WARN(!vlv_punit_is_power_gated(dev_priv, PUNIT_REG_VEDSSPM0),
+"VED not power gated\n");
+}
+
+static void assert_isp_power_gated(struct drm_i915_private *dev_priv)
+{
+   static const struct pci_device_id isp_ids[] = {
+   {PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0f38)},
+   {PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x22b8)},
+   {}
+   };
+
+   WARN(!pci_dev_present(isp_ids) &&
+!vlv_punit_is_power_gated(dev_priv, PUNIT_REG_ISPSSPM0),
+"ISP not power gated\n");
+}
+
 static void intel_power_domains_verify_state(struct drm_i915_private 
*dev_priv);
 
 /**
@@ -3742,10 +3772,13 @@ void intel_power_domains_init_hw(struct 
drm_i915_private *dev_priv, bool resume)
mutex_lock(_domains->lock);
chv_phy_control_init(dev_priv);
mutex_unlock(_domains->lock);
+   assert_isp_power_gated(dev_priv);
} else if (IS_VALLEYVIEW(dev_priv)) {
mutex_lock(_domains->lock);
vlv_cmnlane_wa(dev_priv);
mutex_unlock(_domains->lock);
+   assert_ved_power_gated(dev_priv);
+   assert_isp_power_gated(dev_priv);
} else if 

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Signal the fences as they are cancelled due to wedging

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915: Signal the fences as they are cancelled due to wedging
URL   : https://patchwork.freedesktop.org/series/53226/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_5221_full -> Patchwork_10947_full


Summary
---

  **FAILURE**

  Serious unknown changes coming with Patchwork_10947_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_10947_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
---

  Here are the unknown changes that may have been introduced in 
Patchwork_10947_full:

### IGT changes ###

 Possible regressions 

  * igt@gem_eio@in-flight-internal-10ms:
- shard-snb:  PASS -> DMESG-WARN +2
- shard-kbl:  PASS -> DMESG-WARN

  * igt@gem_eio@in-flight-internal-1us:
- shard-skl:  PASS -> DMESG-WARN +2

  * igt@gem_eio@in-flight-internal-immediate:
- shard-glk:  PASS -> DMESG-WARN +1
- {shard-iclb}:   PASS -> DMESG-WARN +2
- shard-apl:  PASS -> DMESG-WARN +1

  * {igt@runner@aborted}:
- shard-glk:  NOTRUN -> FAIL

  
Known issues


  Here are the changes found in Patchwork_10947_full that come from known 
issues:

### IGT changes ###

 Issues hit 

  * igt@gem_ctx_isolation@bcs0-s3:
- shard-skl:  PASS -> INCOMPLETE [fdo#104108] / [fdo#107773]

  * igt@gem_eio@in-flight-internal-immediate:
- shard-kbl:  PASS -> INCOMPLETE [fdo#103665] +1

  * igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy:
- shard-snb:  PASS -> INCOMPLETE [fdo#105411]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
- shard-skl:  NOTRUN -> DMESG-WARN [fdo#107956] +3

  * igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
- shard-glk:  PASS -> FAIL [fdo#108145]

  * igt@kms_color@pipe-a-gamma:
- shard-skl:  PASS -> FAIL [fdo#104782]

  * igt@kms_color@pipe-a-legacy-gamma-reset:
- shard-apl:  PASS -> DMESG-WARN [fdo#103558] / [fdo#105602] +3

  * igt@kms_color@pipe-b-degamma:
- shard-apl:  PASS -> FAIL [fdo#104782]

  * igt@kms_cursor_crc@cursor-128x128-sliding:
- shard-apl:  PASS -> FAIL [fdo#103232]

  * igt@kms_cursor_crc@cursor-128x128-suspend:
- shard-apl:  PASS -> FAIL [fdo#103191] / [fdo#103232] +1

  * igt@kms_cursor_crc@cursor-256x256-onscreen:
- shard-glk:  PASS -> FAIL [fdo#103232] +2

  * igt@kms_cursor_crc@cursor-64x21-sliding:
- shard-glk:  NOTRUN -> FAIL [fdo#103232]

  * igt@kms_fbcon_fbt@psr-suspend:
- shard-skl:  NOTRUN -> FAIL [fdo#107882]

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-skl:  NOTRUN -> FAIL [fdo#105363]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff:
- shard-glk:  PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt:
- {shard-iclb}:   PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite:
- shard-apl:  PASS -> FAIL [fdo#103167] +1

  * igt@kms_frontbuffer_tracking@fbc-1p-rte:
- shard-apl:  PASS -> FAIL [fdo#103167] / [fdo#105682]

  * igt@kms_frontbuffer_tracking@fbc-shrfb-scaledprimary:
- shard-skl:  PASS -> FAIL [fdo#105682] +1

  * igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary:
- shard-skl:  PASS -> FAIL [fdo#103167] +2

  * igt@kms_plane@plane-position-covered-pipe-b-planes:
- shard-glk:  PASS -> FAIL [fdo#103166] +1

  * igt@kms_plane@plane-position-covered-pipe-c-planes:
- {shard-iclb}:   PASS -> FAIL [fdo#103166]

  * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
- shard-skl:  NOTRUN -> FAIL [fdo#108145] +2

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-y:
- shard-apl:  PASS -> FAIL [fdo#103166]

  * igt@kms_plane_scaling@pipe-a-scaler-with-pixel-format:
- {shard-iclb}:   PASS -> DMESG-WARN [fdo#107724]

  * igt@kms_properties@connector-properties-atomic:
- shard-skl:  NOTRUN -> FAIL [fdo#108642]

  * igt@kms_psr@no_drrs:
- {shard-iclb}:   PASS -> FAIL [fdo#108341]

  * igt@kms_setmode@basic:
- shard-apl:  PASS -> FAIL [fdo#99912]
- shard-kbl:  PASS -> FAIL [fdo#99912]

  * igt@perf@oa-exponents:
- shard-glk:  PASS -> FAIL [fdo#105483]

  * igt@pm_rpm@dpms-mode-unset-non-lpsp:
- shard-skl:  SKIP -> INCOMPLETE [fdo#107807]

  * igt@pm_rpm@system-suspend:
- shard-skl:  NOTRUN -> INCOMPLETE [fdo#104108] / [fdo#107807]

  * igt@pm_rpm@system-suspend-execbuf:
- shard-skl:  PASS -> INCOMPLETE [fdo#104108] / [fdo#107807]

  
 Possible fixes 

  * 

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Implement half float formats and pixel normalize property

2018-11-29 Thread Ville Syrjälä
On Wed, Nov 28, 2018 at 10:38:13PM -0800, Kevin Strasser wrote:
> 64 bpp half float formats are supported on hdr planes only and are subject
> to the following restrictions:
>   * 90/270 rotation not supported
>   * Yf Tiling not supported
>   * Frame Buffer Compression not supported
>   * Color Keying not supported

Is scaling supported? It wasn't on earlier platforms.

> 
> The behavior of pixel normalize with non-float formats is currently
> undefined. As such, the pixel normalize register is enabled iff the
> framebuffer contains floating point pixel data.
> 
> Signed-off-by: Kevin Strasser 
> Cc: Uma Shankar 
> Cc: Shashank Sharma 
> Cc: Ville Syrjälä 
> Cc: David Airlie 
> Cc: Daniel Vetter 
> Cc: dri-de...@lists.freedesktop.org
> ---
>  drivers/gpu/drm/i915/i915_reg.h  | 15 ++-
>  drivers/gpu/drm/i915/intel_display.c | 48 +
>  drivers/gpu/drm/i915/intel_drv.h |  5 +++
>  drivers/gpu/drm/i915/intel_sprite.c  | 82 
> +---
>  4 files changed, 143 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
> index 47baf2fe..871d293 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -6563,6 +6563,10 @@ enum {
>  #define _PLANE_KEYMAX_1_A0x701a0
>  #define _PLANE_KEYMAX_2_A0x702a0
>  #define  PLANE_KEYMAX_ALPHA(a)   ((a) << 24)
> +#define _PLANE_PIXEL_NORMALIZE_1_A   0x701a8
> +#define _PLANE_PIXEL_NORMALIZE_2_A   0x702a8
> +#define   PLANE_PIXEL_NORMALIZE_ENABLE   (1 << 31)
> +#define   PLANE_PIXEL_NORMALIZE_FACTOR_MASK  0x
>  #define _PLANE_AUX_DIST_1_A  0x701c0
>  #define _PLANE_AUX_DIST_2_A  0x702c0
>  #define _PLANE_AUX_OFFSET_1_A0x701c4
> @@ -6786,7 +6790,16 @@ enum {
>  #define PLANE_COLOR_CTL(pipe, plane) \
>   _MMIO_PLANE(plane, _PLANE_COLOR_CTL_1(pipe), _PLANE_COLOR_CTL_2(pipe))
>  
> -#/* SKL new cursor registers */
> +#define _PLANE_PIXEL_NORMALIZE_1_B   0x711a8
> +#define _PLANE_PIXEL_NORMALIZE_2_B   0x712a8
> +#define _PLANE_PIXEL_NORMALIZE_1(pipe) \
> + _PIPE(pipe, _PLANE_PIXEL_NORMALIZE_1_A, _PLANE_PIXEL_NORMALIZE_1_B)
> +#define _PLANE_PIXEL_NORMALIZE_2(pipe) \
> + _PIPE(pipe, _PLANE_PIXEL_NORMALIZE_2_A, _PLANE_PIXEL_NORMALIZE_2_B)
> +#define PLANE_PIXEL_NORMALIZE(pipe, plane) \
> + _MMIO_PLANE(plane, _PLANE_PIXEL_NORMALIZE_1(pipe), 
> _PLANE_PIXEL_NORMALIZE_2(pipe))
> +
> +/* SKL new cursor registers */
>  #define _CUR_BUF_CFG_A   0x7017c
>  #define _CUR_BUF_CFG_B   0x7117c
>  #define CUR_BUF_CFG(pipe)_MMIO_PIPE(pipe, _CUR_BUF_CFG_A, _CUR_BUF_CFG_B)
> diff --git a/drivers/gpu/drm/i915/intel_display.c 
> b/drivers/gpu/drm/i915/intel_display.c
> index e9f4e22..cbacb4b 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -2618,6 +2618,18 @@ int skl_format_to_fourcc(int format, bool rgb_order, 
> bool alpha)
>   return DRM_FORMAT_RGB565;
>   case PLANE_CTL_FORMAT_NV12:
>   return DRM_FORMAT_NV12;
> + case PLANE_CTL_FORMAT_XRGB_16161616F:
> + if (rgb_order) {
> + if (alpha)
> + return DRM_FORMAT_ABGR16161616H;
> + else
> + return DRM_FORMAT_XBGR16161616H;
> + } else {
> + if (alpha)
> + return DRM_FORMAT_ARGB16161616H;
> + else
> + return DRM_FORMAT_XRGB16161616H;
> + }
>   default:
>   case PLANE_CTL_FORMAT_XRGB_:
>   if (rgb_order) {
> @@ -3505,6 +3517,12 @@ static u32 skl_plane_ctl_format(uint32_t pixel_format)
>   return PLANE_CTL_FORMAT_YUV422 | PLANE_CTL_YUV422_VYUY;
>   case DRM_FORMAT_NV12:
>   return PLANE_CTL_FORMAT_NV12;
> + case DRM_FORMAT_XBGR16161616H:
> + case DRM_FORMAT_ABGR16161616H:
> + return PLANE_CTL_FORMAT_XRGB_16161616F | PLANE_CTL_ORDER_RGBX;
> + case DRM_FORMAT_XRGB16161616H:
> + case DRM_FORMAT_ARGB16161616H:
> + return PLANE_CTL_FORMAT_XRGB_16161616F;
>   default:
>   MISSING_CASE(pixel_format);
>   }
> @@ -3680,6 +3698,32 @@ u32 glk_plane_color_ctl(const struct intel_crtc_state 
> *crtc_state,
>   return plane_color_ctl;
>  }
>  
> +u32 icl_plane_pixel_normalize(uint32_t pixel_format,
> +   enum drm_pixel_normalize_range range)
> +{
> + /* 1.0 in half float */
> + u16 half_float_1 = 0x3c00;
> + /* 3.92E-3 in half float */
> + u16 half_float_255 = 0x1c04;
> +
> + switch (pixel_format) {
> + case DRM_FORMAT_XRGB16161616H:
> + case DRM_FORMAT_XBGR16161616H:
> + case DRM_FORMAT_ARGB16161616H:
> + case 

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT (rev3)

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT (rev3)
URL   : https://patchwork.freedesktop.org/series/49669/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5224 -> Patchwork_10962


Summary
---

  **SUCCESS**

  No regressions found.

  External URL: 
https://patchwork.freedesktop.org/api/1.0/series/49669/revisions/3/mbox/

Known issues


  Here are the changes found in Patchwork_10962 that come from known issues:

### IGT changes ###

 Issues hit 

  * igt@kms_pipe_crc_basic@read-crc-pipe-b:
- fi-byt-clapper: PASS -> FAIL [fdo#107362] +1

  
 Possible fixes 

  * igt@kms_frontbuffer_tracking@basic:
- fi-byt-clapper: FAIL [fdo#103167] -> PASS

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
- fi-byt-clapper: FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362


Participating hosts (50 -> 44)
--

  Missing(6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks 
fi-bsw-cyan fi-ctg-p8600 


Build changes
-

* Linux: CI_DRM_5224 -> Patchwork_10962

  CI_DRM_5224: 67ee0d0da79f5b32636d496fd2127da1eecf6262 @ 
git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4736: 285ebfb3b7adc56586031afa5150c4e5ad40c229 @ 
git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10962: e037e3a1eb538b0337c5334507e875f96d4e189d @ 
git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

e037e3a1eb53 drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10962/
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] [CI v15 04/17] drm/i915/dsc: Define & Compute VESA DSC params

2018-11-29 Thread Manasi Navare
From: Gaurav K Singh 

This patches does the following:

1. This patch defines all the DSC parameters as per the VESA
DSC specification. These are stored in the encoder and used
to compute the PPS parameters to be sent to the Sink.
2. Compute all the DSC parameters which are derived from DSC
state of intel_crtc_state.
3. Compute all parameters that are VESA DSC specific

This computation happens in the atomic check phase during
compute_config() to validate if display stream compression
can be enabled for the requested mode.

v8 (From Manasi):
* DEBUG_KMS instead of DRM_ERROR for user triggerable
errors (Ville)
v7: (From Manasi)
* Dont use signed int for rc_range_params (Manasi)
* Mask the range_bpg_offset to use only 6 bits
* Add SPDX identifier (Chris Wilson)
v6 (From Manasi):
* Add a check for line_buf_depth return value (Anusha)
* Remove DRM DSC constants to different patch (Manasi)
v5 (From Manasi):
* Add logic to limit the max line buf depth for DSC 1.1 to 13
as per DSC 1.1 spec
* Fix dim checkpatch warnings/checks

v4 (From Gaurav):
* Rebase on latest drm tip
* rename variable name(Manasi)
* Populate linebuf_depth variable(Manasi)

v3 (From Gaurav):
* Rebase my previous patches on top of Manasi's latest patch
series
* Using >>n rather than /2^n (Manasi)
* Change the commit message to explain what the patch is doing(Gaurav)

Fixed review comments from Ville:
* Don't use macro TWOS_COMPLEMENT
* Mention in comment about the source of RC params
* Return directly from case statements
* Using single asssignment for assigning rc_range_params
* Using <
Cc: Ville Syrjala 
Cc: Anusha Srivatsa 
Cc: Gaurav K Singh 
Signed-off-by: Gaurav K Singh 
Signed-off-by: Manasi Navare 
Co-developed-by: Manasi Navare 
Reviewed-by: Anusha Srivatsa 
---
 drivers/gpu/drm/i915/Makefile |   3 +-
 drivers/gpu/drm/i915/intel_dp.c   |   7 +
 drivers/gpu/drm/i915/intel_drv.h  |   4 +
 drivers/gpu/drm/i915/intel_vdsc.c | 456 ++
 4 files changed, 469 insertions(+), 1 deletion(-)
 create mode 100644 drivers/gpu/drm/i915/intel_vdsc.c

diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
index 0ff878c994e2..8370b9de6e4f 100644
--- a/drivers/gpu/drm/i915/Makefile
+++ b/drivers/gpu/drm/i915/Makefile
@@ -157,7 +157,8 @@ i915-y += dvo_ch7017.o \
  intel_sdvo.o \
  intel_tv.o \
  vlv_dsi.o \
- vlv_dsi_pll.o
+ vlv_dsi_pll.o \
+ intel_vdsc.o
 
 # Post-mortem debug and GPU hang state capture
 i915-$(CONFIG_DRM_I915_CAPTURE_ERROR) += i915_gpu_error.o
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index a2780733768a..3b612eb134e7 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -1953,6 +1953,13 @@ static bool intel_dp_dsc_compute_config(struct intel_dp 
*intel_dp,
return false;
}
}
+   if (intel_dp_compute_dsc_params(intel_dp, pipe_config) < 0) {
+   DRM_DEBUG_KMS("Cannot compute valid DSC parameters for Input 
Bpp = %d "
+ "Compressed BPP = %d\n",
+ pipe_config->pipe_bpp,
+ pipe_config->dsc_params.compressed_bpp);
+   return false;
+   }
pipe_config->dsc_params.compression_enable = true;
DRM_DEBUG_KMS("DP DSC computed with Input Bpp = %d "
  "Compressed Bpp = %d Slice Count = %d\n",
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index 5577f0f31b99..44a5b21799f5 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -1857,6 +1857,10 @@ uint16_t intel_dp_dsc_get_output_bpp(int link_clock, 
uint8_t lane_count,
 uint8_t intel_dp_dsc_get_slice_count(struct intel_dp *intel_dp, int mode_clock,
 int mode_hdisplay);
 
+/* intel_vdsc.c */
+int intel_dp_compute_dsc_params(struct intel_dp *intel_dp,
+   struct intel_crtc_state *pipe_config);
+
 static inline unsigned int intel_dp_unused_lane_mask(int lane_count)
 {
return ~((1 << lane_count) - 1) & 0xf;
diff --git a/drivers/gpu/drm/i915/intel_vdsc.c 
b/drivers/gpu/drm/i915/intel_vdsc.c
new file mode 100644
index ..5e0f9ac5667b
--- /dev/null
+++ b/drivers/gpu/drm/i915/intel_vdsc.c
@@ -0,0 +1,456 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright © 2018 Intel Corporation
+ *
+ * Author: Gaurav K Singh 
+ * Manasi Navare 
+ */
+
+#include 
+#include 
+#include "i915_drv.h"
+#include "intel_drv.h"
+
+enum ROW_INDEX_BPP {
+   ROW_INDEX_6BPP = 0,
+   ROW_INDEX_8BPP,
+   ROW_INDEX_10BPP,
+   ROW_INDEX_12BPP,
+   ROW_INDEX_15BPP,
+   MAX_ROW_INDEX
+};
+
+enum COLUMN_INDEX_BPC {
+   COLUMN_INDEX_8BPC = 0,
+   COLUMN_INDEX_10BPC,
+   COLUMN_INDEX_12BPC,
+   COLUMN_INDEX_14BPC,
+   COLUMN_INDEX_16BPC,
+   MAX_COLUMN_INDEX
+};
+
+#define 

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Remove whitelist application from ringbuffer backend (rev2)

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915: Remove whitelist application from ringbuffer backend (rev2)
URL   : https://patchwork.freedesktop.org/series/53243/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_5224 -> Patchwork_10958


Summary
---

  **FAILURE**

  Serious unknown changes coming with Patchwork_10958 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_10958, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: 
https://patchwork.freedesktop.org/api/1.0/series/53243/revisions/2/mbox/

Possible new issues
---

  Here are the unknown changes that may have been introduced in Patchwork_10958:

### IGT changes ###

 Possible regressions 

  * igt@i915_selftest@live_objects:
- fi-cfl-8109u:   PASS -> DMESG-WARN

  * {igt@runner@aborted}:
- fi-cfl-8109u:   NOTRUN -> FAIL

  
Known issues


  Here are the changes found in Patchwork_10958 that come from known issues:

### IGT changes ###

 Issues hit 

  * igt@i915_selftest@live_hangcheck:
- fi-skl-iommu:   PASS -> INCOMPLETE [fdo#108602]

  * igt@kms_flip@basic-flip-vs-wf_vblank:
- fi-bsw-n3050:   PASS -> FAIL [fdo#100368]

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
- fi-byt-clapper: PASS -> FAIL [fdo#103191] / [fdo#107362] +1

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- fi-cfl-8109u:   PASS -> DMESG-WARN [fdo#106107]

  * {igt@runner@aborted}:
- fi-skl-iommu:   NOTRUN -> FAIL [fdo#108602]

  
 Possible fixes 

  * igt@gem_exec_suspend@basic-s4-devices:
- fi-ivb-3520m:   FAIL [fdo#108880] -> PASS

  * igt@kms_frontbuffer_tracking@basic:
- fi-byt-clapper: FAIL [fdo#103167] -> PASS

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
- fi-byt-clapper: FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
  the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#106107]: https://bugs.freedesktop.org/show_bug.cgi?id=106107
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#108602]: https://bugs.freedesktop.org/show_bug.cgi?id=108602
  [fdo#108880]: https://bugs.freedesktop.org/show_bug.cgi?id=108880


Participating hosts (50 -> 43)
--

  Missing(7): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks 
fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 


Build changes
-

* Linux: CI_DRM_5224 -> Patchwork_10958

  CI_DRM_5224: 67ee0d0da79f5b32636d496fd2127da1eecf6262 @ 
git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4736: 285ebfb3b7adc56586031afa5150c4e5ad40c229 @ 
git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10958: 7228da7066cf1cd08aed0b8a793c7795f5026c43 @ 
git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

7228da7066cf drm/i915: Remove whitelist application from ringbuffer backend

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10958/
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/ddi: Check for unexpectedly disabled transcoders

2018-11-29 Thread Patchwork
== Series Details ==

Series: drm/i915/ddi: Check for unexpectedly disabled transcoders
URL   : https://patchwork.freedesktop.org/series/53256/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_5224 -> Patchwork_10959


Summary
---

  **SUCCESS**

  No regressions found.

  External URL: 
https://patchwork.freedesktop.org/api/1.0/series/53256/revisions/1/mbox/

Known issues


  Here are the changes found in Patchwork_10959 that come from known issues:

### IGT changes ###

 Issues hit 

  * igt@kms_frontbuffer_tracking@basic:
- fi-hsw-peppy:   PASS -> DMESG-WARN [fdo#102614]

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- fi-byt-clapper: PASS -> FAIL [fdo#103191] / [fdo#107362]

  
 Possible fixes 

  * igt@gem_exec_suspend@basic-s4-devices:
- fi-ivb-3520m:   FAIL [fdo#108880] -> PASS

  * igt@kms_frontbuffer_tracking@basic:
- fi-byt-clapper: FAIL [fdo#103167] -> PASS

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
- fi-byt-clapper: FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#108880]: https://bugs.freedesktop.org/show_bug.cgi?id=108880


Participating hosts (50 -> 44)
--

  Missing(6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks 
fi-bsw-cyan fi-ctg-p8600 


Build changes
-

* Linux: CI_DRM_5224 -> Patchwork_10959

  CI_DRM_5224: 67ee0d0da79f5b32636d496fd2127da1eecf6262 @ 
git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4736: 285ebfb3b7adc56586031afa5150c4e5ad40c229 @ 
git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10959: 21d18afdf5208f74d84deb414c00fcf80fc4edd1 @ 
git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

21d18afdf520 drm/i915/ddi: Check for unexpectedly disabled transcoders

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10959/
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


<    1   2   3