Thread (11 messages) 11 messages, 7 authors, 2012-10-08

Re: [PATCH 1/2 v6] of: add helper to parse display timings

From: Robert Schwebel <hidden>
Date: 2012-10-05 07:17:40
Also in: dri-devel, linux-devicetree, linux-media

Possibly related (same subject, not in this thread)

On Thu, Oct 04, 2012 at 11:35:35PM +0200, Guennadi Liakhovetski wrote:
quoted
+optional properties:
+ - hsync-active-high (bool): Hsync pulse is active high
+ - vsync-active-high (bool): Vsync pulse is active high
For the above two we also considered using bool properties but eventually
settled down with integer ones:

- hsync-active = <1>

for active-high and 0 for active low. This has the added advantage of
being able to omit this property in the .dts, which then doesn't mean,
that the polarity is active low, but rather, that the hsync line is not
used on this hardware. So, maybe it would be good to use the same binding
here too?
Philipp, this is the same argumentation as we discussed yesterday for
the dual-link LVDS option, so that one could be modelled in a similar
way.

rsc
-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help