On Tue, 20 Jun 2017 11:42:28 +0200
Daniel Vetter <daniel.vet...@ffwll.ch> wrote:

> It's not obvious that when using the new color manager stuff you still
> need to wire up the legacy helper to get legacy LUT support. Improve
> this with more links. drm_crtc_funcs->gamma_set already explains this
> properly.
> 
> Cc: Peter Rosin <p...@axentia.se>
> Cc: Boris Brezillon <boris.brezil...@free-electrons.com>

Reviewed-by: Boris Brezillon <boris.brezil...@free-electrons.com>

> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
> ---
>  drivers/gpu/drm/drm_color_mgmt.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_color_mgmt.c 
> b/drivers/gpu/drm/drm_color_mgmt.c
> index 3eda500fc005..fe0982708e95 100644
> --- a/drivers/gpu/drm/drm_color_mgmt.c
> +++ b/drivers/gpu/drm/drm_color_mgmt.c
> @@ -128,6 +128,9 @@ EXPORT_SYMBOL(drm_color_lut_extract);
>   * optional. The gamma and degamma properties are only attached if
>   * their size is not 0 and ctm_property is only attached if has_ctm is
>   * true.
> + *
> + * Drivers should use drm_atomic_helper_legacy_gamma_set() to implement the
> + * legacy &drm_crtc_funcs.gamma_set callback.
>   */
>  void drm_crtc_enable_color_mgmt(struct drm_crtc *crtc,
>                               uint degamma_lut_size,

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to