Re: [PATCH v5 1/2] dt-bindings: input: Document Imagis ISA1200 haptic motor driver

[Svyatoslav Ryhel]

чт, 14 трав. 2026 р. о 17:55 Rob Herring <robh@xxxxxxxxxx> пише:
>
> On Tue, May 12, 2026 at 01:24:44PM +0300, Svyatoslav Ryhel wrote:
> > Document the Imagis ISA1200 haptic motor driver, used primarily in mobile
> > handheld devices and capable of supporting up to two motors.
> >
> > The exact datasheet for the ISA1200 is not available; all data was modeled
> > based on available downstream kernel sources for various devices and
> > fragments of information scattered across the internet.
> >
> > Signed-off-by: Svyatoslav Ryhel <clamor95@xxxxxxxxx>
> > ---
> >  .../bindings/input/imagis,isa1200.yaml        | 140 ++++++++++++++++++
> >  1 file changed, 140 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/input/imagis,isa1200.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/input/imagis,isa1200.yaml b/Documentation/devicetree/bindings/input/imagis,isa1200.yaml
> > new file mode 100644
> > index 000000000000..bbe6f99d39c1
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/input/imagis,isa1200.yaml
> > @@ -0,0 +1,140 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/input/imagis,isa1200.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Imagis ISA1200 haptic motor driver
> > +
> > +maintainers:
> > +  - Svyatoslav Ryhel <clamor95@xxxxxxxxx>
> > +  - Linus Walleij <linusw@xxxxxxxxxx>
> > +
> > +description:
> > +  The ISA1200 is a high-performance enhanced haptic motor driver designed
> > +  for mobile hand-held devices. It supports various voltages for both ERM
> > +  (Eccentric Rotating Mass) and LRA (Linear Resonant Actuator) type
> > +  actuators. Thanks to an embedded LDO, battery power can be used directly
> > +  in handheld applications.
> > +
> > +properties:
> > +  compatible:
> > +    const: imagis,isa1200
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  control-gpios:
> > +    description:
> > +      One or two GPIOs flagged as active high linked to HEN and LEN pins
>
>        minItems: 1
>

In theory there may be no GPIOs and both pins can be hooked to power.
This is unlikely scenario but since it is possible I did not set
minItems and did not make control-gpios a required property.

> With that,
>
> Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
>
> > +    maxItems: 2