Re: [PATCH igt] igt/kms_setmode: Clear all connectors before starting the test

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxxxx>
> > > ---
> > >  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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux