[PATCH 2/9] drm/imx: ipuv3-plane: disable local alpha for planes without alpha channel

Ying Liu gnuiyl at gmail.com
Wed Oct 19 07:17:42 UTC 2016


On Wed, Oct 19, 2016 at 12:07 AM, Philipp Zabel <p.zabel at pengutronix.de> wrote:
> Without this patch, after enabling the overlay plane with an RGBA
> framebuffer, switching to a framebuffer without alpha channel would
> cause the plane to vanish, since the pixel local alpha is constant
> zero in that case. Disable local alpha again when setting an opaque
> framebuffer.
>
> Signed-off-by: Philipp Zabel <p.zabel at pengutronix.de>

Looking at the big picture, we've got some hard coding for DP CSC&alpha :(
However, looking at this patch itself, it is acceptable to me.

Regards,
Liu Ying

> ---
>  drivers/gpu/drm/imx/ipuv3-plane.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ipuv3-plane.c
> index 3a03fd8..d4a771c 100644
> --- a/drivers/gpu/drm/imx/ipuv3-plane.c
> +++ b/drivers/gpu/drm/imx/ipuv3-plane.c
> @@ -434,6 +434,7 @@ static void ipu_plane_atomic_update(struct drm_plane *plane,
>                         ipu_dp_set_global_alpha(ipu_plane->dp, false, 0, false);
>                         break;
>                 default:
> +                       ipu_dp_set_global_alpha(ipu_plane->dp, true, 0, true);
>                         break;
>                 }
>         }
> --
> 2.9.3
>


More information about the dri-devel mailing list