On Sat, Oct 06, 2018 at 12:07:15PM +0200, Paul Kocialkowski wrote:
> Contrary to what is implemented in the Linux kernel, the U-Boot driver
> supporting the sunxi TCON was assuming that the TCON0_IO_POL bits
> indicate active-low signals when set for the hsync and vsync signals.
> 
> After looking at it with a logic analyzer, it appears that the signals
> are active-low with the bits unset and active-high with the bits set.
> 
> Fix this by looking for the right flag (high instead of low) and
> inverting the sync bits in the VIDEO_LCD_MODE declarations.
> 
> Signed-off-by: Paul Kocialkowski <cont...@paulk.fr>

For the first three patches,
Acked-by: Maxime Ripard <maxime.rip...@bootlin.com>

It would be good to adjust our wiki as well to not have more of these
inversions creeping in.

Thanks!
Maxime

-- 
Maxime Ripard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to