[PATCH] drm/omapdrm: Remove double gamma table write in omap_crtc_atomic_flush()

2016-09-01 Thread Tomi Valkeinen
On 31/08/16 20:17, Jyri Sarha wrote:
> Remove double gamma table write in omap_crtc_atomic_flush().
> Fixes commit 492a426a2fc53
> ("drm/omapdrm: Implement gamma_lut atomic crtc properties")
> 
> Signed-off-by: Jyri Sarha 
> ---
>  drivers/gpu/drm/omapdrm/omap_crtc.c | 13 -
>  1 file changed, 13 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c 
> b/drivers/gpu/drm/omapdrm/omap_crtc.c
> index 180f644..bdbbe3d 100644
> --- a/drivers/gpu/drm/omapdrm/omap_crtc.c
> +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c
> @@ -411,19 +411,6 @@ static void omap_crtc_atomic_flush(struct drm_crtc *crtc,
>   dispc_mgr_set_gamma(omap_crtc->channel, lut, length);
>   }
>  
> - if (crtc->state->color_mgmt_changed) {
> - struct drm_color_lut *lut = NULL;
> - uint length = 0;
> -
> - if (crtc->state->gamma_lut) {
> - lut = (struct drm_color_lut *)
> - crtc->state->gamma_lut->data;
> - length = crtc->state->gamma_lut->length /
> - sizeof(*lut);
> - }
> - dispc_mgr_set_gamma(omap_crtc->channel, lut, length);
> - }
> -
>   if (dispc_mgr_is_enabled(omap_crtc->channel)) {
>  
>   DBG("%s: GO", omap_crtc->name);

Thanks, I've picked this up for 4.9.

 Tomi

-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: 



[PATCH] drm/omapdrm: Remove double gamma table write in omap_crtc_atomic_flush()

2016-08-31 Thread Jyri Sarha
Remove double gamma table write in omap_crtc_atomic_flush().
Fixes commit 492a426a2fc53
("drm/omapdrm: Implement gamma_lut atomic crtc properties")

Signed-off-by: Jyri Sarha 
---
 drivers/gpu/drm/omapdrm/omap_crtc.c | 13 -
 1 file changed, 13 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c 
b/drivers/gpu/drm/omapdrm/omap_crtc.c
index 180f644..bdbbe3d 100644
--- a/drivers/gpu/drm/omapdrm/omap_crtc.c
+++ b/drivers/gpu/drm/omapdrm/omap_crtc.c
@@ -411,19 +411,6 @@ static void omap_crtc_atomic_flush(struct drm_crtc *crtc,
dispc_mgr_set_gamma(omap_crtc->channel, lut, length);
}

-   if (crtc->state->color_mgmt_changed) {
-   struct drm_color_lut *lut = NULL;
-   uint length = 0;
-
-   if (crtc->state->gamma_lut) {
-   lut = (struct drm_color_lut *)
-   crtc->state->gamma_lut->data;
-   length = crtc->state->gamma_lut->length /
-   sizeof(*lut);
-   }
-   dispc_mgr_set_gamma(omap_crtc->channel, lut, length);
-   }
-
if (dispc_mgr_is_enabled(omap_crtc->channel)) {

DBG("%s: GO", omap_crtc->name);
-- 
1.9.1