[Intel-gfx] [PATCH 0/6] Pipe level color management

[Daniel Stone]

Hi,

On 22 January 2016 at 15:04, Daniel Stone <daniel at fooishbar.org> wrote:
> On 21 January 2016 at 15:03, Lionel Landwerlin
> <lionel.g.landwerlin at intel.com> wrote:
>> Hi,
>>
>> This serie introduces pipe level color management through a set of properties
>> attached to the CRTC. It also provides an implementation for some Intel
>> platforms.
>>
>> This serie is based of a previous set of patches by Shashank Sharma and takes
>> into account of the comments by Daniel Stone & Daniel Vetter.
>
> This is a lot more tractable than previous series, thanks!

One subtlety which did just come up on IRC and should probably be
documented, is the interaction with the 'Broadcast Mode' property.
Currently, Broadcast Mode logically sits on top of colour management:
all tables passed into CM are assumed to be for 0-255 colour ranges,
and setting limited BM instead clamps these tables at apply time to
16-235 (by manually adjusting them in the driver).

It'd be nice to document this, so someone doesn't implement this the
other way around, or assuming that CM trumps BM, or ...

Cheers,
Daniel