[igt-dev] [PATCH i-g-t 20/20] tests/psr: Add more PSR subtests to fast feedback list

[Dhinakaran Pandiyan]

On Tue, 2018-04-24 at 15:32 +0000, Saarinen, Jani wrote:
> HI, 
> 
> > -----Original Message-----
> > From: igt-dev [mailto:igt-dev-bounces at lists.freedesktop.org] On Behalf Of Petri
> > Latvala
> > Sent: tiistai 24. huhtikuuta 2018 12.34
> > To: Pandiyan, Dhinakaran <dhinakaran.pandiyan at intel.com>; Vivi, Rodrigo
> > <rodrigo.vivi at intel.com>
> > Cc: igt-dev at lists.freedesktop.org
> > Subject: Re: [igt-dev] [PATCH i-g-t 20/20] tests/psr: Add more PSR subtests to
> > fast feedback list
> > 
> > 
> > 
> > On 04/24/2018 02:58 AM, Dhinakaran Pandiyan wrote:
> > >
> > >
> > > On Mon, 2018-04-23 at 16:06 -0700, Rodrigo Vivi wrote:
> > >> On Sat, Apr 14, 2018 at 06:13:40PM -0700, Dhinakaran Pandiyan wrote:
> > >>> On Fri, 2018-04-13 at 12:16 +0300, Petri Latvala wrote:
> > >>>> On Wed, Apr 11, 2018 at 07:14:36PM +0300, Pandiyan, Dhinakaran wrote:
> > >>>>> Yeah, that is expected. The kernel patches are not in yet.
> > >>>> The requirement for using --sink-crc is going away when kernel side
> > >>>> patches land? Or how is this planned?
> > >>>>
> > >>> Irrespective of whether or when the kernel patches land, --sink-crc
> > >>> will not be used for CI. --sink-crc is being retained for developers
> > >>> to manually use it for debug and possibly fix the code in future.
> > >>>
> > >>> Having said that, the kernel patches adding time stamp support
> > >>> should land soon, after which the tests will not skip.
> > >> Ok, I finally looked to all patches on this series and I support all of them.
> > >>
> > >> My only concern is actually with patch 14 that is around this discussion here...
> > >>
> > >> For me it seems that or you run with interactive debug on or with
> > >> sink_crc and CI is not running with any so it will all skip.
> > >>
> > >> Unless I missunderstood something on that patch...
> > >>
> > >> If CI still runs without requiring any extra option I'm in favor of
> > >> the whole series and I'm tending to merge it soon...
> > >>
> > > CI will not need any options. Timestamps are supported by default on
> > > all platforms except CHV/VLV. We don't have those platforms in CI and
> > > I am planning to nuke PSR support for them.
> > >
> > 
> > These machines are VLV:
> > fi-byt-j1900
> > fi-byt-n2820
> Also CHV=BSW => fi-bsw-n3050
> 

No psr panels on them.


> > 
> > 
> > --
> > Petri Latvala
> > 
> > _______________________________________________
> > igt-dev mailing list
> > igt-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/igt-dev
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev