Hi Tomi,
On Tuesday 18 March 2014 08:33:55 Tomi Valkeinen wrote:
On 17/03/14 16:22, Laurent Pinchart wrote:
+Example +-------
+lcd0: display {
- compatible = "tpo,taal", "panel-dsi-cm";
- label = "lcd0";
- gpios = <&gpio4 6 GPIO_ACTIVE_HIGH>; /* 102, reset */
If the panel uses a TE GPIO but no reset GPIO, do you plan to express this with a "hole" for the reset GPIO ? e.g. something like
gpios = <0>, <&gpio4 6 GPIO_ACTIVE_HIGH>;
Yes.
Wouldn't it be better to split the gpios property into "reset-gpios" and "te- gpios" ?
Yes, I can change it. I don't have a strong preference.
I've gotten similar comments for other bindings also, so I guess the preferred way is to use named "-gpios" properties for everything except the case where you really have multiple gpios with the same purpose?
That's my understanding, yes.
The gpio binding documentation doesn't give much guidance on this.