On Wed, May 18, 2016 at 02:23:18PM +0200, Robin van der Gracht wrote: > Added a driver for the Holtek HT16K33 LED controller with keyscan. > > Signed-off-by: Robin van der Gracht <robin@xxxxxxxxxxx> > --- > Changes in v6: > - Fixed minor typo in documentation 'KEY_F0' -> 'KEY_F10' > > .../devicetree/bindings/display/ht16k33.txt | 42 ++ > drivers/auxdisplay/Kconfig | 9 + > drivers/auxdisplay/Makefile | 1 + > drivers/auxdisplay/ht16k33.c | 563 +++++++++++++++++++++ > 4 files changed, 615 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/ht16k33.txt > create mode 100644 drivers/auxdisplay/ht16k33.c > > diff --git a/Documentation/devicetree/bindings/display/ht16k33.txt b/Documentation/devicetree/bindings/display/ht16k33.txt > new file mode 100644 > index 0000000..5c11320 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/ht16k33.txt > @@ -0,0 +1,42 @@ > +Holtek ht16k33 RAM mapping 16*8 LED controller driver with keyscan > +------------------------------------------------------------------------------- > + > +Required properties: > +- compatible: "holtek,ht16k33" > +- reg: I2C slave address of the chip. > +- interrupt-parent: A phandle pointing to the interrupt controller > + serving the interrupt for this chip. > +- interrupts: Interrupt specification for the key pressed interrupt. > +- refresh-rate-hz: Display update interval in HZ. > +- debounce-delay-ms: Debouncing interval time in microseconds. The units and description still don't match. I acked the last one. Please add acks when sending new versions (unless of course you don't fix any trivial things noted). Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html