[igt-dev] [PATCH i-g-t 1/3] igt/kms_fbcon_fbt: Handle ENODEV when checking i915_edp_psr_status for chipset support

[Daniel Vetter]

On Thu, Feb 22, 2018 at 07:48:53AM +0000, Lofstedt, Marta wrote:
> 
> 
> > -----Original Message-----
> > From: Chris Wilson [mailto:chris at chris-wilson.co.uk]
> > Sent: Thursday, February 22, 2018 9:47 AM
> > To: Lofstedt, Marta <marta.lofstedt at intel.com>; igt-
> > dev at lists.freedesktop.org
> > Subject: Re: [igt-dev] [PATCH i-g-t 1/3] igt/kms_fbcon_fbt: Handle ENODEV
> > when checking i915_edp_psr_status for chipset support
> > 
> > Quoting Marta Lofstedt (2018-02-22 07:43:41)
> > > If the machine doesn't support PSR, it will return -ENODEV from
> > > i915_edp_psr_status, which we want to interpret as unsupported.
> > > This is in line with what Chris just fixed for FBC.
> > >
> > > Signed-off-by: Marta Lofstedt <marta.lofstedt at intel.com>
> > > ---
> > >  tests/kms_fbcon_fbt.c | 2 ++
> > >  1 file changed, 2 insertions(+)
> > >
> > > diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c index
> > > 1b236f8b..6aa171e6 100644
> > > --- a/tests/kms_fbcon_fbt.c
> > > +++ b/tests/kms_fbcon_fbt.c
> > > @@ -164,6 +164,8 @@ static bool psr_supported_on_chipset(int fd)  {
> > >         char buf[256];
> > >
> > > +       if (*buf == '\0') /* !HAS_PSR -> -ENODEV*/
> > > +               return false;
> > >         igt_debugfs_read(fd, "i915_edp_psr_status", buf);
> > 
> > After the read :)
> 
> Yeah, was way too trigger happy here

Since Chris wrote the same patch from kms_frontbuffer_tracking ... should
we extract these into helpers in lib/ somewhere?
-Daniel

> 
> > 
> > >         return strstr(buf, "Sink_Support: yes\n");  }
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch