On Mon, Jun 26, 2017 at 10:27 PM, Wang Yafei [off-list ref] wrote:

On Mon, Jun 26, 2017 at 09:16:40AM -0500, Rob Herring wrote:

quoted

On Mon, Jun 26, 2017 at 8:22 AM, Wang Yafei [off-list ref] wrote:

quoted

Hi Rob,

Thank you for the review.

On Fri, Jun 23, 2017 at 03:37:06PM -0500, Rob Herring wrote:

quoted

On Mon, Jun 19, 2017 at 06:40:32PM +0800, Wang Yafei wrote:

quoted

Signed-off-by: Wang Yafei <redacted>
---
 .../bindings/input/touchscreen/goodix-gtx5.txt     | 75 ++++++++++++++++++++++
 1 file changed, 75 insertions(+)
 create mode 100755 Documentation/devicetree/bindings/input/touchscreen/goodix-gtx5.txt

[...]

quoted

quoted

quoted

quoted

+           compatible = "goodix,gtx5";
+           reg = <0x14>;
+           interrupt-parent = <&msm_gpio>;
+           interrupts = <13 0x2800>;
+           vtouch-supply = <&pm8916_l15>;
+           reset-gpios = <&msm_gpio 12 0x0>;
+           irq-gpios = <&msm_gpio 13 0x2800>;
+           irq-flags = <1>; /* 1:trigger rising, 2:trigger falling;*/
+           touchscreen-max-id = <10>;
+           touchscreen-size-x = <400>;
+           touchscreen-size-y = <400>;
+           touchscreen-max-w = <400>;
+           touchscreen-max-pressure = <255>;
+           touchscreen-swapped-x-y;
+           touchscreen-key-map = <172 158>; /*KEY_HOMEPAGE, KEY_BACK*/
+           sensor0 {

How many sensors can there be? That needs to be documented.

Why do you need a sub-node?

sensor is an internal call, we use senesorX to represent the touchpanel
is produced by which factory. A panel factory will have it corresponding
config data to get the best touch results. It will delete the sub-node
and change to the following format:
        panel0,normal-cfg = [...]  /* panelX,cfg : X is no more than 10*/

panel0 is not a vendor and normal doesn't tell us anything. How about
just "goodix,panel-cfg-X"?

Can I use "goodix,normal-cfg-X" rather than "goodix,panel-cfg-X"? The
string "normal-cfg" is meaningful to driver.

Okay.

Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html