<div dir="ltr">Dear Chanh Nguyen,<br><br>Thank you for your response.<br><br>Chanh Nguyen <<a href="mailto:chanh@amperemail.onmicrosoft.com">chanh@amperemail.onmicrosoft.com</a>> æ–¼ 2024å¹´11月6æ—¥ é€±ä¸‰ ä¸‹åˆ12:58寫道:<br>><br>><br>><br>> On 06/11/2024 09:39, Eason Yang wrote:<br>> > This adds a binding specification for the Nuvoton NCT7201/NCT7202<br>> > family of ADCs.<br>> ><br>> > Signed-off-by: Eason Yang <<a href="mailto:j2anfernee@gmail.com">j2anfernee@gmail.com</a>><br>> > ---<br>> > Â  .../bindings/iio/adc/nuvoton,nct720x.yaml Â  Â  | 47 +++++++++++++++++++<br>> > Â  MAINTAINERS Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  | Â 1 +<br>> > Â  2 files changed, 48 insertions(+)<br>> > Â  create mode 100644 Documentation/devicetree/bindings/iio/adc/nuvoton,nct720x.yaml<br>> ><br>> > diff --git a/Documentation/devicetree/bindings/iio/adc/nuvoton,nct720x.yaml b/Documentation/devicetree/bindings/iio/adc/nuvoton,nct720x.yaml<br>> > new file mode 100644<br>> > index 000000000000..3052039af10e<br>> > --- /dev/null<br>> > +++ b/Documentation/devicetree/bindings/iio/adc/nuvoton,nct720x.yaml<br>> > @@ -0,0 +1,47 @@<br>> > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)<br>> > +%YAML 1.2<br>> > +---<br>> > +$id: <a href="http://devicetree.org/schemas/iio/adc/nuvoton,nct720x.yaml#">http://devicetree.org/schemas/iio/adc/nuvoton,nct720x.yaml#</a><br>> > +$schema: <a href="http://devicetree.org/meta-schemas/core.yaml#">http://devicetree.org/meta-schemas/core.yaml#</a><br>> > +<br>> > +title: Nuvoton nct7202 and similar ADCs<br>> > +<br>> > +maintainers:<br>> > + Â - Eason Yang <<a href="mailto:yhyang2@nuvoton.com">yhyang2@nuvoton.com</a>><br>> > +<br>> > +description: |<br>> > + Â  Family of ADCs with i2c interface.<br>> > +<br>> > +properties:<br>> > + Â compatible:<br>> > + Â  Â enum:<br>> > + Â  Â  Â - nuvoton,nct7201<br>> > + Â  Â  Â - nuvoton,nct7202<br>> > +<br>> > + Â reg:<br>> > + Â  Â maxItems: 1<br>> > +<br>> > + Â read-vin-data-size:<br>><br>> Is it generic property or vendor property? I tried to find in the<br>> <a href="https://github.com/torvalds/linux/tree/master/Documentation/devicetree/bindings">https://github.com/torvalds/linux/tree/master/Documentation/devicetree/bindings</a><br>> , but it seems this property hasn't been used on other devices.<br>><br>> If it is vendor property, then I think it should include a vendor<br>> prefix. For examples:<br>><br>> <a href="https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/adi%2Cad7780.yaml#L50">https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/adi%2Cad7780.yaml#L50</a><br>> <a href="https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/fsl%2Cvf610-adc.yaml#L42">https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/fsl%2Cvf610-adc.yaml#L42</a><br>> <a href="https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/st%2Cstmpe-adc.yaml#L22">https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/st%2Cstmpe-adc.yaml#L22</a><br><br>I would add a vendor prefix for it.<br><br>><br>><br>> > + Â  Â description: number of data bits per read vin<br>> > + Â  Â $ref: /schemas/types.yaml#/definitions/uint32<br>> > + Â  Â enum: [8, 16]<br>> > +<br>> > +required:<br>> > + Â - compatible<br>> > + Â - reg<br>> > + Â - read-vin-data-size<br>> > +<br>> > +additionalProperties: false<br>> > +<br>> > +examples:<br>> > + Â - |<br>> > + Â  Â i2c {<br>> > + Â  Â  Â  Â #address-cells = <1>;<br>> > + Â  Â  Â  Â #size-cells = <0>;<br>> > +<br>> > + Â  Â  Â  Â nct7202@1d {<br>><br>> I think the Node name should follow<br>> <a href="https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation">https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation</a><br>><br>><br>> For some examples that were merged before<br>><br>> <a href="https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/adi%2Cad7091r5.yaml#L102">https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/adi%2Cad7091r5.yaml#L102</a><br>> <a href="https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/maxim%2Cmax1238.yaml#L73">https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/maxim%2Cmax1238.yaml#L73</a><br>> <a href="https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/ti%2Cadc081c.yaml#L49">https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/ti%2Cadc081c.yaml#L49</a><br>><br><br>I would change it for the node naming.<br><br>> > + Â  Â  Â  Â  Â  Â compatible = "nuvoton,nct7202";<br>> > + Â  Â  Â  Â  Â  Â reg = <0x1d>;<br>> > + Â  Â  Â  Â  Â  Â read-vin-data-size = <8>;<br>> > + Â  Â  Â  Â };<br>> > + Â  Â };<br>> > diff --git a/MAINTAINERS b/MAINTAINERS<br>> > index 91d0609db61b..68570c58e7aa 100644<br>> > --- a/MAINTAINERS<br>> > +++ b/MAINTAINERS<br>> > @@ -2746,6 +2746,7 @@ L: Â  Â  Â <a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a> (moderated for non-subscribers)<br>> > Â  S: Â Supported<br>> > Â  F: Â Documentation/devicetree/bindings/*/*/*npcm*<br>> > Â  F: Â Documentation/devicetree/bindings/*/*npcm*<br>> > +F: Â  Documentation/devicetree/bindings/iio/adc/nuvoton,nct720x.yaml<br>> > Â  F: Â Documentation/devicetree/bindings/rtc/nuvoton,nct3018y.yaml<br>> > Â  F: Â arch/arm/boot/dts/nuvoton/nuvoton-npcm*<br>> > Â  F: Â arch/arm/mach-npcm/<br>></div>