[PATCH 4/5] drm/i915: Implement color management on bdw/skl/bxt/kbl
lionel.g.landwerlin at intel.com
Tue Feb 23 14:42:20 UTC 2016
On 23/02/16 00:52, Matt Roper wrote:
> On Mon, Feb 22, 2016 at 02:18:10PM +0000, Lionel Landwerlin wrote:
>> Patch based on a previous series by Shashank Sharma.
>> v2: Do not read GAMMA_MODE register to figure what mode we're in
>> v3: Program PREC_PAL_GC_MAX to clamp pixel values > 1.0
>> Add documentation on how the Broadcast RGB property is affected by CTM
>> v4: Update contributors
>> v5: Refactor degamma/gamma LUTs load into a single function
>> v6: Fix missing intel_crtc variable (bisect issue)
> Not sure if you saw me feedback on v4 or not:
> It looks like most of those comments still apply here. If you disagree
> with my comments, that's fine too, I just wanted to make sure it didn't
> get overlooked. :-)
Sorry for that, completely missed on the first 3/4 comments.
You're right the limited range multiplication was wrong (input[i] vs
Fixed that and simplified it as you suggested.
Thanks a lot!
More information about the dri-devel