On Tue, Nov 11, 2014 at 05:19:46PM +0200, Ville Syrjälä wrote:
> On Tue, Nov 11, 2014 at 04:00:12PM +0100, Daniel Vetter wrote:
> > On Wed, Nov 05, 2014 at 02:26:08PM -0800, Jesse Barnes wrote:
> > > This is useful for checking things later.
> > >
> > > v2:
> > > - fix hsw infoframe enabled check (Ander)
> > >
> > > Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>
> > > ---
> > > drivers/gpu/drm/i915/intel_drv.h | 4 +++
> > > drivers/gpu/drm/i915/intel_hdmi.c | 62 +++++++++++++++++++++++++++++++++++++++
> > > 2 files changed, 66 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
> > > index d53ac23..8aa80e1 100644
> > > --- a/drivers/gpu/drm/i915/intel_drv.h
> > > +++ b/drivers/gpu/drm/i915/intel_drv.h
> > > @@ -292,6 +292,9 @@ struct intel_crtc_config {
> > > * between pch encoders and cpu encoders. */
> > > bool has_pch_encoder;
> > >
> > > + /* Are we sending infoframes on the attached port */
> > > + bool has_infoframe;
> >
> > The cross-checking of this new hw state is missing. I've added that while
> > applying.
>
> Where is the compute_config part of this? I can't see it in any patch,
> but maybe I'm just blind.
Yeah, seems to be missing, thanks for catching this. Jesse/Ander, can you
pls supply a fixup quickly - I'd need to drop the series until this one
otherwise I think.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
