[PATCH 7/7] Staging/iio/adc/touchscreen/MXS: provide devicetree adaption

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 ;)

quoted

[...]

@@ -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.

quoted

+                               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".

quoted

+- 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/  |