On Fri, Sep 13, 2024 at 10:52:29AM +0300, Antoniu Miclaus wrote: > Add device tree bindings for ad7877. > > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx> > --- > changes in v4: > - add bindings patch first > - update commit title/body based on review. > - add units for pen-down-acc-interval property. > - add default value for adi,acquisition-time-us and adi,vref-delay-us property > .../input/touchscreen/adi,ad7877.yaml | 106 ++++++++++++++++++ > 1 file changed, 106 insertions(+) > create mode 100644 Documentation/devicetree/bindings/input/touchscreen/adi,ad7877.yaml > > diff --git a/Documentation/devicetree/bindings/input/touchscreen/adi,ad7877.yaml b/Documentation/devicetree/bindings/input/touchscreen/adi,ad7877.yaml > new file mode 100644 > index 000000000000..c472f50f7a59 > --- /dev/null > +++ b/Documentation/devicetree/bindings/input/touchscreen/adi,ad7877.yaml > @@ -0,0 +1,106 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/input/touchscreen/adi,ad7877.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog Devices AD7877 Touch Screen Controller > + > +maintainers: > + - Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx> > + > +description: | > + Analog Devices Touch Screen Controller > + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7877.pdf > + > +allOf: > + - $ref: touchscreen.yaml# > + - $ref: /schemas/spi/spi-peripheral-props.yaml# > + > +properties: > + compatible: > + enum: > + - adi,ad7877 > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + spi-max-frequency: > + description: AD7877 SPI bus clock frequency. > + minimum: 10000 > + maximum: 20000000 > + > + adi,stopacq-polarity: > + description: The polarity of the signal applied to the STOPACQ pin. > + $ref: /schemas/types.yaml#/definitions/string > + enum: [low, high] Missing default. > + > + adi,first-conv-delay-ns: > + description: Delay in ns before the first conversion. > + enum: [500, 128000, 1000000, 8000000] Missing default. Reviewed-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx> Best regards, Krzysztof
