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: > https://lists.freedesktop.org/archives/intel-gfx/2016-February/088043.html > > 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. :-) > > > Matt
Sorry for that, completely missed on the first 3/4 comments. You're right the limited range multiplication was wrong (input[i] vs result[i]...). Fixed that and simplified it as you suggested. Thanks a lot! - Lionel
