linux-kernel - Re: [PATCH 1/2] input: misc: Add generic input driver to read encoded GPIO lines

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <f069755c-8312-c58a-2a52-9c56648299d8@ti.com>
Date:	Wed, 17 Aug 2016 12:12:24 +0530
From:	Vignesh R <vigneshr@...com>
To:	Rob Herring <robh@...nel.org>
CC:	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Mark Rutland <mark.rutland@....com>,
	Tony Lindgren <tony@...mide.com>,
	Russell King <linux@...linux.org.uk>,
	Arnd Bergmann <arnd@...db.de>,
	John Stultz <john.stultz@...aro.org>,
	S Twiss <stwiss.opensource@...semi.com>,
	Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@...aro.org>,
	Moritz Fischer <moritz.fischer@...us.com>,
	Daniel Hung-yu Wu <hywu@...gle.com>,
	Grant Grundler <grundler@...omium.org>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 1/2] input: misc: Add generic input driver to read encoded
 GPIO lines



On Saturday 13 August 2016 12:29 AM, Rob Herring wrote:
> On Fri, Aug 12, 2016 at 10:22:28AM +0530, Vignesh R wrote:
>> Add a driver to read group of GPIO lines and provide its status as a
>> numerical value as input event to the system. This will help in
>> intefacing devices, that can be connected over GPIOs, that provide input
>> to the system by driving GPIO lines connected to them like a rotary dial
>> or a switch.
>>
>> For example, a rotary switch can be connected to four GPIO lines. The
>> status of the GPIO lines reflect the actual position of the rotary
>> switch dial. For example, if dial points to 9, then the four GPIO lines
>> connected to the switch will read HLLH(0b'1001 = 9). This value
>> can be reported as an ABS_* event to the input subsystem.
>>
>> Signed-off-by: Vignesh R <vigneshr@...com>
>> ---
>>  .../devicetree/bindings/input/gpio-decoder.txt     |  23 ++++
>>  drivers/input/misc/Kconfig                         |  12 ++
>>  drivers/input/misc/Makefile                        |   1 +
>>  drivers/input/misc/gpio_decoder.c                  | 128 +++++++++++++++++++++
>>  4 files changed, 164 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/input/gpio-decoder.txt
>>  create mode 100644 drivers/input/misc/gpio_decoder.c
>>
>> diff --git a/Documentation/devicetree/bindings/input/gpio-decoder.txt b/Documentation/devicetree/bindings/input/gpio-decoder.txt
>> new file mode 100644
>> index 000000000000..efd947f52cb3
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/input/gpio-decoder.txt
>> @@ -0,0 +1,23 @@
>> +* GPIO Decoder DT bindings
>> +
>> +Required Properties:
>> +- compatible: should be "gpio-decoder"
>> +- gpios: a spec of gpios (atleast two) to be decoded to a number with
>> +  first entry representing the MSB.
>> +
>> +Optional Properties:
>> +- gpio-decoder,max-value: Maximum possible value that can be reported by
> 
> Just "decoder-max-value".

Agreed. I will wait for any comments for the driver part, before sending
the next version.

> 
> With that, the binding looks fine to me:
> 
> Acked-by: Rob Herring <robh@...nel.org>
> 

Thanks!

-- 
Regards
Vignesh