lkml.org 
[lkml]   [2012]   [Dec]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] gpio: export 'debounce' attribute if supported by the gpio chip
From
On Sun, Dec 9, 2012 at 6:07 PM, Guenter Roeck <groeck-dsl@sbcglobal.net> wrote:

> The gpio pins I am dealing with are provided by an FPGA which is used on various
> boards. While the gpio access registers are always the same, the actual usage is
> board specific. This means I either need to write ugly code, or use the gpio
> subsystem to provide access to the gpio pins. Ugly code is out of the question,
> which means I'll need gpio support.
>
> Anyway, I want to keep things simple, not add unnecessary complexity. Having to
> go through the input subsystem just to be able to support debounce on a couple
> of input pins doesn't really sound simple. Guess I'll have to find another
> solution if the patch is not accepted. Maybe I'll add a "debounce" property to
> the gpio driver's of properties.

I would like you to seriously consider both gpio-input and the new
drivers/extcon/extcon-gpio.c driver (maybe adding debounce into that
driver). The latter also has a sysfs interface.

Your usecases seem to be EXTCON_MECHANICAL. If not, new classes
can surely be added.

There is some risk that sysfs makes everything that is a GPIO line
appear as GPIO instead of what it really is, which is not helpful
for userspace which will have to keep track of all the routing of the
electronics.

The latter might be comfortable if you're designing and maintaining the
whole system with firmware and so on, but from a newcomer
examining the system in sysfs it's not helpful, and we're designing
that ABI for a purpose.

(And I share Grant's hesitant stance on extending the GPIO sysfs.)

Yours,
Linus Walleij


\
 
 \ /
  Last update: 2012-12-10 12:01    [W:0.069 / U:1.512 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site