HI Yannick

On 4/6/22 10:37, Yannick FERTRE wrote:
> Wrong DISPLAY_FLAGS used to set the data enable polarity.
> 
> Signed-off-by: Yannick FERTRE <yannick.fer...@foss.st.com>
> ---
> 
>  drivers/video/stm32/stm32_ltdc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/video/stm32/stm32_ltdc.c 
> b/drivers/video/stm32/stm32_ltdc.c
> index e741e74739..9269d4bfb4 100644
> --- a/drivers/video/stm32/stm32_ltdc.c
> +++ b/drivers/video/stm32/stm32_ltdc.c
> @@ -255,7 +255,7 @@ static void stm32_ltdc_set_mode(struct stm32_ltdc_priv 
> *priv,
>               val |= GCR_HSPOL;
>       if (timings->flags & DISPLAY_FLAGS_VSYNC_HIGH)
>               val |= GCR_VSPOL;
> -     if (timings->flags & DISPLAY_FLAGS_DE_HIGH)
> +     if (timings->flags & DISPLAY_FLAGS_DE_LOW)
>               val |= GCR_DEPOL;
>       if (timings->flags & DISPLAY_FLAGS_PIXDATA_NEGEDGE)
>               val |= GCR_PCPOL;

Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>

Thanks 
Patrice

Reply via email to