On 4/22/22 09:51, Patrice CHOTARD wrote: > 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 <[email protected]> >> --- >> >> 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 <[email protected]> > > Thanks > Patrice Applied to u-boot-stm32 Thanks Patrice

