From: Rojhalat Ibrahim <imr@rtschenk.de>
To: Gerhard Sittig <gsi@denx.de>
Cc: Alexandre Courbot <gnurou@gmail.com>,
Linus Walleij <linus.walleij@linaro.org>,
"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
Grant Likely <grant.likely@linaro.org>,
Mark Brown <broonie@kernel.org>, Wolfram Sang <wsa@the-dreams.de>
Subject: Re: [PATCH 1/2][v5] gpiolib: allow simultaneous setting of multiple GPIO outputs
Date: Thu, 31 Jul 2014 10:42:03 +0200 [thread overview]
Message-ID: <12452684.GmmEHEPXzu@pcimr> (raw)
In-Reply-To: <20140731071235.GH3407@book.gsilab.sittig.org>
On Thursday 31 July 2014 09:12:35 Gerhard Sittig wrote:
> On Thu, 2014-07-31 at 11:35 +0900, Alexandre Courbot wrote:
> >
> > I'd be surprised if at least *one* such driver was not in mainline
> > though. Had a quick look but could not find anything that would be an
> > obvious candidate to use these new functions.
>
> What about drivers/mtd/nand/gpio.c, the GPIO bitbanged NAND
> controller driver? Most of the latch, control, and data lines
> groups are set at the same time. It should be a perfect example.
>
Well, not so perfect, since AFAIUI only a few control signals are managed
by GPIOs (Chip Enable, Command Latch Enable, Address Latch Enable), but not
the actual data lines. But granted, at least at one place in the driver three
lines are set at the same time.
There are a few other drivers which also do that:
- drivers/tty/serial/serial_mctrl_gpio.c controls multiple modem lines via GPIO
- drivers/net/phy/mdio-mux-gpio.c a GPIO controlled MDIO bus multiplexer driver
- drivers/regulator/gpio-regulator.c controls voltage regulators
All of the above do set multiple GPIOs at the same time, but in all cases
performance does not really matter. Moreover they all seem a bit obscure to me.
Nevertheless, IMHO they are still better candidates for using the new functions
than the i2c-gpio or spi-gpio drivers.
So, Linus, would a patched version of one of the mentioned drivers qualify as
a user that would allow you to merge my patch?
Rojhalat
next prev parent reply other threads:[~2014-07-31 8:42 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-16 13:51 [PATCH 1/2][v5] gpiolib: allow simultaneous setting of multiple GPIO outputs Rojhalat Ibrahim
2014-06-21 7:08 ` Alexandre Courbot
2014-06-21 20:24 ` Mark Brown
2014-07-07 15:23 ` Linus Walleij
2014-07-07 16:12 ` Rojhalat Ibrahim
2014-07-09 10:17 ` Linus Walleij
2014-07-30 9:12 ` Rojhalat Ibrahim
2014-07-30 10:41 ` Mark Brown
2014-07-30 11:04 ` Rojhalat Ibrahim
2014-07-31 2:35 ` Alexandre Courbot
2014-07-31 7:12 ` Gerhard Sittig
2014-07-31 8:42 ` Rojhalat Ibrahim [this message]
2014-08-06 12:45 ` Gerhard Sittig
2014-08-06 19:39 ` Mark Brown
2014-08-07 9:52 ` Rojhalat Ibrahim
2014-08-29 5:21 ` Linus Walleij
2014-08-29 8:15 ` Wolfram Sang
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=12452684.GmmEHEPXzu@pcimr \
--to=imr@rtschenk.de \
--cc=broonie@kernel.org \
--cc=gnurou@gmail.com \
--cc=grant.likely@linaro.org \
--cc=gsi@denx.de \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=wsa@the-dreams.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).