Re: [PATCH v2 3/4] drm/vc4: Add color transformation matrix (CTM) support

2018-03-30 Thread Eric Anholt
Stefan Schake writes: > The hardware supports a CTM with S0.9 values. We therefore only allow > a value of 1.0 or fractional only and reject all others with integer > parts. This restriction is mostly inconsequential in practice since > commonly used transformation matrices

[PATCH v2 3/4] drm/vc4: Add color transformation matrix (CTM) support

2018-03-24 Thread Stefan Schake
The hardware supports a CTM with S0.9 values. We therefore only allow a value of 1.0 or fractional only and reject all others with integer parts. This restriction is mostly inconsequential in practice since commonly used transformation matrices have all scalars <= 1.0. Signed-off-by: Stefan