On Wed, Jan 04, 2023 at 02:34:13PM +0100, Martin Zaťovič wrote: > Add a bus driver for Wiegand protocol. The bus driver handles > Wiegand controller and Wiegand device managemement and driver > matching. The bus driver defines the structures for Wiegand > controllers and Wiegand devices. Wiegand controller structure > contains the format and payload_len fields to add support for > different format messages. This should be handled by the > controller driver. > > Each Wiegand controller should be associated with one Wiegand > device, as Wiegand is typically a point-to-point bus. > > Signed-off-by: Martin Zaťovič <m.zatovic1@xxxxxxxxx> > --- > MAINTAINERS | 2 + > drivers/bus/Kconfig | 5 + > drivers/bus/Makefile | 1 + > drivers/bus/wiegand.c | 768 ++++++++++++++++++++++++++++++++++++++++ > include/linux/wiegand.h | 228 ++++++++++++ > 5 files changed, 1004 insertions(+) > create mode 100644 drivers/bus/wiegand.c > create mode 100644 include/linux/wiegand.h > > diff --git a/MAINTAINERS b/MAINTAINERS > index db9624d93af0..b6e68e92f0e3 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -22432,6 +22432,8 @@ WIEGAND BUS DRIVER > M: Martin Zaťovič <m.zatovic1@xxxxxxxxx> > S: Maintained > F: Documentation/devicetree/bindings/wiegand/wiegand-controller.yaml > +F: drivers/bus/wiegand.c > +F: include/linux/wiegand.h > > WILOCITY WIL6210 WIRELESS DRIVER > L: linux-wireless@xxxxxxxxxxxxxxx > diff --git a/drivers/bus/Kconfig b/drivers/bus/Kconfig > index 7bfe998f3514..360e55abc311 100644 > --- a/drivers/bus/Kconfig > +++ b/drivers/bus/Kconfig > @@ -241,6 +241,11 @@ config VEXPRESS_CONFIG > Platform configuration infrastructure for the ARM Ltd. > Versatile Express. > > +config WIEGAND > + tristate "Wiegand Bus driver" > + help > + Driver for low-level 2-wire serial protocol communication. We need more information here in the help text please. thanks, greg k-h