[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Do not redefine the has_csr parameter.

[Imre Deak]

On Fri, Aug 17, 2018 at 07:40:11PM +0100, Chris Wilson wrote:
> Quoting Rodrigo Vivi (2018-08-17 19:24:14)
> > On Fri, Aug 17, 2018 at 05:59:59PM -0000, Patchwork wrote:
> > > == Series Details ==
> > > 
> > > Series: drm/i915: Do not redefine the has_csr parameter.
> > > URL   : https://patchwork.freedesktop.org/series/48408/
> > > State : success
> > > 
> > > == Summary ==
> > > 
> > > = CI Bug Log - changes from CI_DRM_4686 -> Patchwork_9974 =
> > > 
> > > == Summary - WARNING ==
> > > 
> > >   Minor unknown changes coming with Patchwork_9974 need to be verified
> > >   manually.
> > >   
> > >   If you think the reported changes have nothing to do with the changes
> > >   introduced in Patchwork_9974, 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/48408/revisions/1/mbox/
> > > 
> > > == Possible new issues ==
> > > 
> > >   Here are the unknown changes that may have been introduced in Patchwork_9974:
> > > 
> > >   === IGT changes ===
> > > 
> > >     ==== Possible regressions ====
> > > 
> > >     igt at pm_rpm@basic-rte:
> > >       {fi-icl-u}:         PASS -> FAIL
> > 
> > Is this related?
> 
> Yes. The platform now HAS_CSR but no DMC firmware so runtime-pm has to
> be disabled. Allowing the platform to enter runtime suspend with
> programming the dmc as before the patch is an undefined hazard.

Right. We could convert the result to be skip in this case, not sure if
it's worth the effort.

In any case the patch is good to be merged.

> -Chris
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx