[PATCHv8][ 4/7] staging: imx-drm: Use de-active and pixelclk-active display-timings.

Lothar Waßmann LW at KARO-electronics.de
Thu Mar 6 01:58:43 EST 2014


Denis Carikli wrote:
> If de-active and/or pixelclk-active properties were set in the
> display-timings DT node, they were not used.
> Instead the data-enable and the pixel data clock polarity
> were hardcoded.
> This change is needed for making the eukrea-cpuimx51
>   QVGA display work.
I just tried this patch on our hardware and found that the pixelclock
polarity is inverse to what the documentation says.

Your patch sets the 'clk_pol' variable in positive logic, while it is
interpreted in negative logic when converted to the final register
value in drivers/staging/imx-drm/ipu-v3/ipu-di.c:

	if (!(sig->clk_pol))

IMO this should be
	if (sig->clk_pol)

Did you actually measure the resulting clock signal and LCD data?

Lothar Waßmann

Ka-Ro electronics GmbH | Pascalstraße 22 | D - 52076 Aachen
Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10
Geschäftsführer: Matthias Kaussen
Handelsregistereintrag: Amtsgericht Aachen, HRB 4996

www.karo-electronics.de | info at karo-electronics.de

More information about the linux-arm-kernel mailing list