On Wed, Mar 01, 2023 at 03:28:32PM +0100, Martin Zaťovič wrote: > Wiegand bus is defined by a Wiegand controller node. This node > can contain one or more device nodes for devices attached to > the controller(it is advised to only connect one device as Wiegand > is a point-to-point bus). > > Wiegand controller needs to specify several attributes such as > the pulse length in order to function properly. These attributes > are documented here. > > Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > Signed-off-by: Martin Zaťovič <m.zatovic1@xxxxxxxxx> > --- > .../bindings/wiegand/wiegand-controller.yaml | 39 +++++++++++++++++++ > MAINTAINERS | 5 +++ > 2 files changed, 44 insertions(+) > create mode 100644 Documentation/devicetree/bindings/wiegand/wiegand-controller.yaml > > diff --git a/Documentation/devicetree/bindings/wiegand/wiegand-controller.yaml b/Documentation/devicetree/bindings/wiegand/wiegand-controller.yaml > new file mode 100644 > index 000000000000..df985cb3045a > --- /dev/null > +++ b/Documentation/devicetree/bindings/wiegand/wiegand-controller.yaml > @@ -0,0 +1,39 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/wiegand/wiegand-controller.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Wiegand Generic Controller Common Properties > + > +maintainers: > + - Martin Zaťovič <martin.zatovic@xxxxxxxxxxxxxxxxxx> > + > +description: > + Wiegand busses can be described with a node for the Wiegand controller device > + and a set of child nodes for each SPI slave on the bus. > + > +properties: > + $nodename: > + pattern: "^wiegand(@.*|-[0-9a-f])?$" > + > + pulse-len-us: > + description: | Don't need '|' here and elsewhere. With that fixed, Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > + Length of the low pulse in microseconds. > + > + interval-len-us: > + description: | > + Length of a whole bit (both the pulse and the high phase) in microseconds. > + > + frame-gap-us: > + description: | > + Length of the last bit of a frame (both the pulse and the high phase) in > + microseconds. > + > +required: > + - compatible > + - pulse-len-us > + - interval-len-us > + - frame-gap-us > + > +additionalProperties: true > diff --git a/MAINTAINERS b/MAINTAINERS > index b0db911207ba..1f6f6d236f0c 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -22512,6 +22512,11 @@ L: linux-rtc@xxxxxxxxxxxxxxx > S: Maintained > F: drivers/rtc/rtc-sd3078.c > > +WIEGAND BUS DRIVER > +M: Martin Zaťovič <m.zatovic1@xxxxxxxxx> > +S: Maintained > +F: Documentation/devicetree/bindings/wiegand/wiegand-controller.yaml > + > WIIMOTE HID DRIVER > M: David Rheinsberg <david.rheinsberg@xxxxxxxxx> > L: linux-input@xxxxxxxxxxxxxxx > -- > 2.39.2 >
