On 2017-12-23 07:07 AM, Mario Kleiner wrote: > The hardware gamma luts get bypassed at color > depth 30 anyway, so skip their setup. > > Also skip drmmode_crtc_gamma_set() for a screen > with depth 30. > > Signed-off-by: Mario Kleiner <[email protected]> > --- > src/drmmode_display.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/src/drmmode_display.c b/src/drmmode_display.c > index 7ad3235..e5f9dbe 100644 > --- a/src/drmmode_display.c > +++ b/src/drmmode_display.c > @@ -1275,6 +1275,10 @@ drmmode_crtc_gamma_set(xf86CrtcPtr crtc, uint16_t > *red, uint16_t *green, > RADEONInfoPtr info = RADEONPTR(scrn); > int i; > > + /* Hw gamma lut's are bypassed at color depth 30 */ > + if (scrn->depth == 30) > + return;
It's better to set xf86CrtcFuncsRec::gamma_set = NULL in this case, to prevent the X server from wasting work calculating gamma tables that will never be used. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
