Hi Jonathan, On Saturday 21 September 2013 14:42:37 Jonathan Cameron wrote: > The device tree list has moved, so I've changed the cc. > > Few comments inline. Basically I'd go for longer more > descriptive names when the abreviation isn't a really > well known common one. > > Note I'm not all that familiar with device tree conventions > so may be barking up the wrong tree ;) > > > [...] > > @@ -9,13 +10,38 @@ Optional properties: > > - fsl,lradc-touchscreen-wires: Number of wires used to connect the > > touchscreen to LRADC. Valid value is either 4 or 5. If this property is > > not present, then the touchscreen is - > > disabled. > > Do we want that lradc prefix on these other properties? The question is, if we want to keep it. This is the currently used binding. > > + disabled. 5 wires is valid for i.MX28 SoC only. > > +- fsl,ave-ctrl: number of samples per direction to calculate an average value. > > + Allowed value is 1 ... 31, default is 4 > > This naming seems less than informative, what about fsl,average_num_samples > of average_count (ave isn't all that obvious an abrevaition of average). I used already existing names from "Documentation/devicetree/bindings/input/touchscreen/stmpe.txt". > > +- fsl,ave-delay: delay between consecutive samples. Allowed value is > > + 1 ... 2047. It is used if 'fsl,ave-ctrl' > 1, counts at > > + 2 kHz and its default is 2 (= 1 ms) > > fsl,average_intersample_delay (I assume any limit of device tree property > names is longer than that?) If the parameter description in "Documentation/devicetree/bindings" is useful, we don't need long parameter names. If there is no useful parameter description, we will need long names ;) Regards, Juergen -- Pengutronix e.K. | Juergen Beisert | Linux Solutions for Science and Industry | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html