[Intel-gfx] [PATCH igt] igt/kms_setmode: Clear all connectors before starting the test

[Chris Wilson]

On Mon, Oct 24, 2016 at 01:11:49PM +0300, Ville Syrjälä wrote:
> On Mon, Oct 24, 2016 at 11:04:35AM +0200, Daniel Vetter wrote:
> > On Sat, Oct 22, 2016 at 09:32:41AM +0100, Chris Wilson wrote:
> > > Before we start trying random combinations of connectors and CRTCs, we
> > > should first ensure we have a blank slate so that if we only change a
> > > subset of the CRTC we do not conflict with a residual setup on the other
> > > CRTC.
> > > 
> > > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > > ---
> > >  tests/kms_setmode.c | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > > 
> > > diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
> > > index 24fb34c..df958f0 100644
> > > --- a/tests/kms_setmode.c
> > > +++ b/tests/kms_setmode.c
> > > @@ -268,6 +268,10 @@ static void setup_crtcs(drmModeRes *resources, struct connector_config *cconf,
> > >  	int i;
> > >  	int encoder_usage_count[resources->count_encoders];
> > >  
> > > +	for (i = 0; i < resources->count_crtcs; i++)
> > > +		drmModeSetCrtc(drm_fd, resources->crtcs[i],
> > > +			       0, 0, 0, 0, 0, NULL);
> > 
> > Shouldn't this be in some modeset helper library, or is kms_setmode not
> > using that one?
> 
> Like kmstest_unset_all_crtcs() ?

Done.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre