From: florian.vaussard@epfl•ch (Florian Vaussard)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 1/2] gpiolib: add gpio_export_with_name
Date: Thu, 30 May 2013 11:19:33 +0200 [thread overview]
Message-ID: <51A719A5.50906@epfl.ch> (raw)
In-Reply-To: <20121126135911.2F1D33E187C@localhost>
Hello Grant,
On 11/26/2012 02:59 PM, Grant Likely wrote:
> On Wed, 21 Nov 2012 11:14:08 +0100, Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft•com> wrote:
>> allow to specify a name to an exported gpio
>>
>> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft•com>
>
> The gpio sysfs ABI is already horrible, racy, and unsafe. Really, we
> need a proper chrdev interface for controlling gpios. Sysfs is fine for
> poking around and experimenting, but we cannot provide any fine grained
> access control, locking or faster IO with the one-file-per-gpio sysfs
> model. So, no, I don't think this is a good idea to extend gpiolib in
> this way.
>
Would it make sense to provide only the DT binding to export a GPIO,
without changing the sysfs ABI? Even if work is progressing towards
having gpio-controlled reset pins [1], some boards still need GPIOs to
be exported to userspace for other functionalities.
Regards,
Florian
[1] http://article.gmane.org/gmane.linux.power-management.general/31364
next prev parent reply other threads:[~2013-05-30 9:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-21 10:12 [PATCH 0/2] gpio: dt: add gpio-export support Jean-Christophe PLAGNIOL-VILLARD
2012-11-21 10:14 ` [PATCH 1/2] gpiolib: add gpio_export_with_name Jean-Christophe PLAGNIOL-VILLARD
2012-11-21 10:14 ` [PATCH 2/2] gpiolib-of: ad gpio-export support Jean-Christophe PLAGNIOL-VILLARD
2012-11-26 13:59 ` [PATCH 1/2] gpiolib: add gpio_export_with_name Grant Likely
2012-11-28 20:19 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-18 6:04 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-19 13:11 ` Grant Likely
2013-05-30 9:19 ` Florian Vaussard [this message]
2013-05-30 20:03 ` Linus Walleij
2013-05-30 20:38 ` Florian Vaussard
2013-05-30 21:11 ` Linus Walleij
2013-05-31 8:17 ` Florian Vaussard
2012-11-21 14:43 ` [PATCH 0/2] gpio: dt: add gpio-export support Linus Walleij
2012-11-24 18:48 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-01 16:56 ` Linus Walleij
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51A719A5.50906@epfl.ch \
--to=florian.vaussard@epfl$(echo .)ch \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox