From: Rob Herring <robh@kernel.org>
To: Giulio Benetti <giulio.benetti@benettiengineering.com>
Cc: "Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Jonathan Neuschäfer" <j.neuschaefer@gmx.net>,
"Henrik Rydberg" <rydberg@bitmath.org>,
linux-input@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 2/3] dt-bindings: touchscreen: Add HY46XX bindings
Date: Thu, 8 Apr 2021 15:21:37 -0500 [thread overview]
Message-ID: <20210408202137.GA1890401@robh.at.kernel.org> (raw)
In-Reply-To: <20210407174909.1475150-3-giulio.benetti@benettiengineering.com>
On Wed, Apr 07, 2021 at 07:49:08PM +0200, Giulio Benetti wrote:
> This adds device tree bindings for the Hycon HY46XX touchscreen series.
>
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
> V1->V2:
> As suggested by Rob Herring:
> * fixed $id: address
> * added "hycon," in front of every custom property
> * changed all possible property to boolean type
> * removed proximity-sensor-switch property since it's not handled in driver
> V2->V3:
> As suggested by Jonathan Neuschäfer:
> * fixed some typo
> * fixed description indentation
> * improved boolean properties descriptions
> * improved hycon,report-speed description
> V3->V4:
> * fixed binding compatible string in example as suggested by Jonathan Neuschäfer
> ---
> .../input/touchscreen/hycon,hy46xx.yaml | 120 ++++++++++++++++++
> MAINTAINERS | 6 +
> 2 files changed, 126 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/input/touchscreen/hycon,hy46xx.yaml
>
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/hycon,hy46xx.yaml b/Documentation/devicetree/bindings/input/touchscreen/hycon,hy46xx.yaml
> new file mode 100644
> index 000000000000..8860613a12ad
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/input/touchscreen/hycon,hy46xx.yaml
> @@ -0,0 +1,120 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/input/touchscreen/hycon,hy46xx.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Hycon HY46XX series touchscreen controller bindings
> +
> +description: |
> + There are 6 variants of the chip for various touch panel sizes and cover lens material
> + Glass: 0.3mm--4.0mm
> + PET/PMMA: 0.2mm--2.0mm
> + HY4613(B)-N048 < 6"
> + HY4614(B)-N068 7" .. 10.1"
> + HY4621-NS32 < 5"
> + HY4623-NS48 5.1" .. 7"
> + Glass: 0.3mm--8.0mm
> + PET/PMMA: 0.2mm--4.0mm
> + HY4633(B)-N048 < 6"
> + HY4635(B)-N048 < 7" .. 10.1"
> +
> +maintainers:
> + - Giulio Benetti <giulio.benetti@benettiengineering.com>
> +
> +allOf:
> + - $ref: touchscreen.yaml#
> +
> +properties:
> + compatible:
> + enum:
> + - hycon,hycon-hy4613
> + - hycon,hycon-hy4614
> + - hycon,hycon-hy4621
> + - hycon,hycon-hy4623
> + - hycon,hycon-hy4633
> + - hycon,hycon-hy4635
As suggested earlier, drop the 2nd 'hycon'.
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + reset-gpios:
> + maxItems: 1
> +
> + vcc-supply: true
> +
> + hycon,threshold:
> + description: Allows setting the sensitivity in the range from 0 to 255.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + minimum: 0
> + maximum: 255
> +
> + hycon,glove-enable:
> + type: boolean
> + description: Allows enabling glove setting.
> +
> + hycon,report-speed:
> + description: Allows setting the report speed in Hertz.
If in Hertz, use standard unit suffix.
> + $ref: /schemas/types.yaml#/definitions/uint32
And then you can drop this.
> + minimum: 0
0Hz doesn't seem to useful?
> + maximum: 255
> +
> + hycon,power-noise-enable:
hycon,noise-filter-enable
No one wants to enable power noise. :)
> + type: boolean
> + description: Allows enabling power noise filter.
> +
> + hycon,filter-data:
> + description: Allows setting the filtering data before reporting touch
> + in the range from 0 to 5.
This is averaging samples? Sounds like something common perhaps.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + minimum: 0
> + maximum: 5
> +
> + hycon,gain:
> + description: Allows setting the sensitivity distance in the range from 0 to 5.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + minimum: 0
> + maximum: 5
> +
> + hycon,edge-offset:
> + description: Allows setting the edge compensation in the range from 0 to 16.
> + $ref: /schemas/types.yaml#/definitions/uint32
> + minimum: 0
> + maximum: 16
> +
> + touchscreen-size-x: true
> + touchscreen-size-y: true
> + touchscreen-fuzz-x: true
> + touchscreen-fuzz-y: true
> + touchscreen-inverted-x: true
> + touchscreen-inverted-y: true
> + touchscreen-swapped-x-y: true
> + interrupt-controller: true
> +
> +additionalProperties: false
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> +
> +examples:
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + hycon-hy4633@1c {
touchscreen@1c
> + compatible = "hycon,hycon-hy4633";
> + reg = <0x1c>;
> + interrupt-parent = <&gpio2>;
> + interrupts = <5 IRQ_TYPE_EDGE_FALLING>;
> + reset-gpios = <&gpio2 6 GPIO_ACTIVE_LOW>;
> + };
> + };
> +
> +...
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c80ad735b384..d022ff09e609 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -8243,6 +8243,12 @@ S: Maintained
> F: mm/hwpoison-inject.c
> F: mm/memory-failure.c
>
> +HYCON HY46XX TOUCHSCREEN SUPPORT
> +M: Giulio Benetti <giulio.benetti@benettiengineering.com>
> +L: linux-input@vger.kernel.org
> +S: Maintained
> +F: Documentation/devicetree/bindings/input/touchscreen/hycon,hy46xx.yaml
> +
> HYGON PROCESSOR SUPPORT
> M: Pu Wen <puwen@hygon.cn>
> L: linux-kernel@vger.kernel.org
> --
> 2.25.1
>
next prev parent reply other threads:[~2021-04-08 20:21 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20210305163834.70924-1-giulio.benetti@benettiengineering.com>
2021-03-05 16:38 ` [PATCH 1/3] dt-bindings: Add Hycon Technology vendor prefix Giulio Benetti
2021-03-05 17:25 ` Jonathan Neuschäfer
2021-03-05 17:33 ` Giulio Benetti
2021-03-05 19:32 ` Giulio Benetti
2021-03-05 16:38 ` [PATCH 2/3] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-03-05 19:34 ` Giulio Benetti
2021-03-06 19:28 ` Rob Herring
2021-03-06 19:41 ` Rob Herring
2021-04-01 18:37 ` Giulio Benetti
2021-04-01 22:24 ` Giulio Benetti
2021-04-01 23:03 ` [PATCH v2 0/3] Input: add Hycon HY46XX Touchscreen controller Giulio Benetti
2021-04-01 23:03 ` [PATCH v2 1/3] dt-bindings: Add Hycon Technology vendor prefix Giulio Benetti
2021-04-01 23:03 ` [PATCH v2 2/3] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-04-02 8:36 ` Jonathan Neuschäfer
2021-04-02 15:17 ` Giulio Benetti
2021-04-01 23:03 ` [PATCH v2 3/3] Input: add driver for the Hycon HY46XX touchpanel series Giulio Benetti
2021-04-02 8:59 ` Jonathan Neuschäfer
2021-04-02 15:23 ` Giulio Benetti
2021-04-02 16:16 ` [PATCH v3 0/3] Input: add Hycon HY46XX Touchscreen controller Giulio Benetti
2021-04-02 16:16 ` [PATCH v3 1/3] dt-bindings: Add Hycon Technology vendor prefix Giulio Benetti
2021-04-02 16:16 ` [PATCH v3 2/3] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-04-06 13:24 ` Rob Herring
2021-04-07 17:57 ` Giulio Benetti
2021-04-07 18:56 ` Rob Herring
2021-04-07 19:17 ` Giulio Benetti
2021-04-06 13:37 ` Jonathan Neuschäfer
2021-04-06 14:07 ` Giulio Benetti
2021-04-07 17:49 ` [PATCH v4 0/3] Input: add Hycon HY46XX Touchscreen controller Giulio Benetti
2021-04-07 17:49 ` [PATCH v4 1/3] dt-bindings: Add Hycon Technology vendor prefix Giulio Benetti
2021-04-08 20:16 ` Rob Herring
2021-04-07 17:49 ` [PATCH v4 2/3] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-04-08 20:21 ` Rob Herring [this message]
2021-04-11 11:37 ` Giulio Benetti
2021-04-11 11:48 ` [PATCH v5 0/3] Input: add Hycon HY46XX Touchscreen controller Giulio Benetti
2021-04-11 11:48 ` [PATCH v5 1/3] dt-bindings: Add Hycon Technology vendor prefix Giulio Benetti
2021-04-12 15:05 ` Rob Herring
2021-04-12 15:12 ` Giulio Benetti
2021-04-13 13:35 ` Rob Herring
2021-04-13 14:44 ` [PATCH v7 0/3] Input: add Hycon HY46XX Touchscreen controller Giulio Benetti
2021-04-13 14:44 ` [PATCH v7 1/3] dt-bindings: Add Hycon Technology vendor prefix Giulio Benetti
2021-04-14 5:44 ` Dmitry Torokhov
2021-04-13 14:44 ` [PATCH v7 2/3] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-04-13 16:36 ` Rob Herring
2021-04-14 5:44 ` Dmitry Torokhov
2021-04-13 14:44 ` [PATCH v7 3/3] Input: add driver for the Hycon HY46XX touchpanel series Giulio Benetti
2021-04-14 5:44 ` Dmitry Torokhov
2021-04-14 6:46 ` Peter Hutterer
2021-04-14 11:22 ` Giulio Benetti
2021-04-14 17:26 ` Dmitry Torokhov
2021-04-15 6:16 ` Peter Hutterer
2021-04-14 11:24 ` Giulio Benetti
2021-04-11 11:48 ` [PATCH v5 2/3] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-04-12 13:20 ` Rob Herring
2021-04-12 14:46 ` Rob Herring
2021-04-12 14:49 ` Giulio Benetti
2021-04-12 15:23 ` [PATCH v6 0/2] Input: add Hycon HY46XX Touchscreen controller Giulio Benetti
2021-04-12 15:23 ` [PATCH v6 1/2] dt-bindings: touchscreen: Add HY46XX bindings Giulio Benetti
2021-04-13 15:07 ` Rob Herring
2021-04-13 15:27 ` Giulio Benetti
2021-04-13 17:58 ` Rob Herring
2021-04-13 18:34 ` Giulio Benetti
2021-04-12 15:24 ` [PATCH v6 2/2] Input: add driver for the Hycon HY46XX touchpanel series Giulio Benetti
2021-04-11 11:48 ` [PATCH v5 3/3] " Giulio Benetti
2021-04-07 17:49 ` [PATCH v4 " Giulio Benetti
2021-04-02 16:16 ` [PATCH v3 " Giulio Benetti
2021-03-05 16:38 ` [PATCH " Giulio Benetti
2021-03-05 19:00 ` Dmitry Torokhov
2021-03-05 19:31 ` Giulio Benetti
2021-03-05 19:54 ` Giulio Benetti
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210408202137.GA1890401@robh.at.kernel.org \
--to=robh@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=giulio.benetti@benettiengineering.com \
--cc=j.neuschaefer@gmx.net \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rydberg@bitmath.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).