Re: [PATCH v6 2/3] dt-bindings: iio: proximity: Add TYHX HX9023S

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 2024/6/23 02:04, Krzysztof Kozlowski wrote:
On 22/06/2024 14:35, Yasin Lee wrote:
On 2024/6/22 18:51, Conor Dooley wrote:
On Sat, Jun 22, 2024 at 01:56:42PM +0800, Yasin Lee wrote:
On 2024/6/21 18:12, Krzysztof Kozlowski wrote:

Hi ,Krzysztof
Thank you for your reply. I have some questions inline.

Best regards,
Yasin

On 21/06/2024 09:40, Yasin Lee wrote:
A capacitive proximity sensor

Signed-off-by: Yasin Lee <yasin.lee.x@xxxxxxxxx>
---
    .../bindings/iio/proximity/tyhx,hx9023s.yaml       | 115 +++++++++++++++++++++
    1 file changed, 115 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/proximity/tyhx,hx9023s.yaml b/Documentation/devicetree/bindings/iio/proximity/tyhx,hx9023s.yaml
new file mode 100644
index 000000000000..beca70ce7609
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/proximity/tyhx,hx9023s.yaml
@@ -0,0 +1,115 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/proximity/tyhx,hx9023s.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TYHX HX9023S capacitive proximity sensor
+
+maintainers:
+  - Yasin Lee <yasin.lee.x@xxxxxxxxx>
+
+description: |
+  TYHX HX9023S proximity sensor. Datasheet can be found here:
+    http://www.tianyihexin.com/ueditor/php/upload/file/20240614/1718336303992081.pdf
+
+allOf:
+  - $ref: /schemas/iio/iio.yaml#
Which part of iio.yaml binding do you use here? I cannot find anything,
so this looks wrong.	

I will remove this reference.


+
+properties:
+  compatible:
+    const: tyhx,hx9023s
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    description:
+      Generated by device to announce preceding read request has finished
+      and data is available or that a close/far proximity event has happened.
+    maxItems: 1
+
+  vdd-supply: true
+
+  "#address-cells":
+    const: 1
+
+  "#size-cells":
+    const: 0
+
+patternProperties:
+  "^channel@[0-4]$":
+    $ref: /schemas/iio/adc/adc.yaml
+    type: object
+
+    properties:
+      reg:
+        minimum: 0
+        maximum: 4
+        description: The channel number.
+
+      input-channel:
Isn't this duplicating single-channel property?

Where is this property defined (which common schema)?

|input-channel| is indeed intended for single-ended configuration, but I
couldn't find a definition

or reference for |single-channel| anywhere. If possible, should I rename
|input-channel| to |single-channel|?
Single-channel is new, it should be the next branch of the iio tree and
in linux-next.
Hi Conor,

Thank you for informing me. I plan to temporarily add a prefix to this
attribute to distinguish it and update it in the future. Is this the
correct approach?
No, because there is no need. You are supposed to work on maintainer
tree (linux-next works usually as well).

Best regards,
Krzysztof

Hi Krzysztof,

Understood, Thank you for the reply, I will reference single-channel directly as I do with diff-channel.

Best regards,

Yasin





[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux