[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <87fu3l4uti.fsf@anholt.net>
Date: Mon, 23 Apr 2018 11:54:49 -0700
From: Eric Anholt <eric@...olt.net>
To: Stefan Schake <stschake@...il.com>
Cc: airlied@...ux.ie, dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org, Stefan Schake <stschake@...il.com>
Subject: Re: [PATCH v4 1/2] drm/vc4: Add CTM support
Stefan Schake <stschake@...il.com> writes:
> The hardware has a single block for applying a CTM prior to gamma lut.
> It can be fed with pixels from one of our CRTC at a time and uses a
> matrix with S0.9 scalars. Use private atomic state to reject attempts
> from userland to apply CTM for more than one CRTC at a time and reject
> matrices with scalars that we can't approximate without integer bits.
Looks good! I think you got all the transition cases we were concerned
about before.
Reviewed and applied these two to drm-misc-next. Thanks!
Download attachment "signature.asc" of type "application/pgp-signature" (833 bytes)
Powered by blists - more mailing lists