[Mesa-dev] [PATCH 9/9] panfrost: Enable sRGB

Tomeu Vizoso tomeu at tomeuvizoso.net
Wed Jun 19 14:23:27 UTC 2019


On Wed, 19 Jun 2019 at 07:44, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
>
> On Tue, 18 Jun 2019 at 17:00, Alyssa Rosenzweig
> <alyssa.rosenzweig at collabora.com> wrote:
> >
> > Now that sRGB formats are supported for both rendering and sampling,
> > advertise support.
> >
> > Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
>
> Hi there,
>
> this patch seems to have caused the following regressions:
>
> dEQP-GLES2.functional.fbo.render.recreate_colorbuffer.no_rebind_rbo_rgb565
> dEQP-GLES2.functional.fbo.render.resize.tex2d_rgb
>
> For more details, see https://gitlab.freedesktop.org/tomeu/mesa/pipelines/43396
>
> Should we revert it for now?

Actually, I think this is the commit that introduced these regressions:

panfrost: Figure out job requirements in pan_job.c

Cheers,

Tomeu

> Cheers,
>
> Tomeu
>
> > ---
> >  src/gallium/drivers/panfrost/pan_screen.c | 4 ----
> >  1 file changed, 4 deletions(-)
> >
> > diff --git a/src/gallium/drivers/panfrost/pan_screen.c b/src/gallium/drivers/panfrost/pan_screen.c
> > index 9cd98cd9051..70bff565930 100644
> > --- a/src/gallium/drivers/panfrost/pan_screen.c
> > +++ b/src/gallium/drivers/panfrost/pan_screen.c
> > @@ -450,10 +450,6 @@ panfrost_is_format_supported( struct pipe_screen *screen,
> >          if (sample_count > 1)
> >                  return FALSE;
> >
> > -       /* sRGB colorspace is not supported (yet?) */
> > -       if (format_desc->colorspace == UTIL_FORMAT_COLORSPACE_SRGB)
> > -               return FALSE;
> > -
> >          /* Format wishlist */
> >          if (format == PIPE_FORMAT_Z24X8_UNORM || format == PIPE_FORMAT_X8Z24_UNORM)
> >                  return FALSE;
> > --
> > 2.20.1
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list