From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Cc: "Kuppuswamy Sathyanarayanan"
<sathyanarayanan.kuppuswamy@linux.intel.com>,
linux-aspeed@lists.ozlabs.org,
"David Daney" <david.daney@cavium.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
alsa-devel@alsa-project.org, "Peter Tyser" <ptyser@xes-inc.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@st.com>,
"Jonathan Hunter" <jonathanh@nvidia.com>,
"Fabio Estevam" <festevam@gmail.com>,
linux-stm32@st-md-mailman.stormreply.com,
"Marek Vasut" <marek.vasut+renesas@gmail.com>,
"Florian Fainelli" <f.fainelli@gmail.com>,
"Kevin Hilman" <khilman@kernel.org>,
"Michal Simek" <michal.simek@xilinx.com>,
"Marek Behun" <marek.behun@nic.cz>,
"Bartosz Golaszewski" <bgolaszewski@baylibre.com>,
"Ludovic Desroches" <ludovic.desroches@microchip.com>,
bcm-kernel-feedback-list@broadcom.com,
"Joel Stanley" <joel@jms.id.au>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Bamvor Jian Zhang" <bamv2005@gmail.com>,
"NXP Linux Team" <linux-imx@nxp.com>,
linux-pwm@vger.kernel.org,
"Grygorii Strashko" <grygorii.strashko@ti.com>,
"Charles Keepax" <ckeepax@opensource.cirrus.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
linux-omap@vger.kernel.org, mazziesaccount@gmail.com,
"Ray Jui" <rjui@broadcom.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
linux-gpio@vger.kernel.org,
"Richard Fitzgerald" <rf@opensource.cirrus.com>,
"Santosh Shilimkar" <ssantosh@kernel.org>,
linux-tegra@vger.kernel.org,
"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
linux-arm-kernel@lists.infradead.org,
"Scott Branden" <sbranden@broadcom.com>,
"Masahiro Yamada" <yamada.masahiro@socionext.com>,
"Andrew Jeffery" <andrew@aj.id.au>,
"Enrico Weigelt, metux IT consult" <info@metux.net>,
linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
"Thorsten Scherer" <t.scherer@eckelmann.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
patches@opensource.cirrus.com, "Shawn Guo" <shawnguo@kernel.org>
Subject: Re: [alsa-devel] [PATCH v2 0/2] Add definition for GPIO direction
Date: Wed, 6 Nov 2019 09:39:38 -0500 [thread overview]
Message-ID: <20191106143938.GA3176@icarus> (raw)
In-Reply-To: <cover.1573029228.git.matti.vaittinen@fi.rohmeurope.com>
On Wed, Nov 06, 2019 at 10:51:06AM +0200, Matti Vaittinen wrote:
> The patch series adds definitions for GPIO line directions.
>
> For occasional GPIO contributor like me it is always a pain to remember
> whether 1 or 0 was used for GPIO direction INPUT/OUTPUT. Judging the
> fact that I removed few comments like:
>
> /* Return 0 if output, 1 if input */
> /* This means "out" */
> return 1; /* input */
> return 0; /* output */
>
> it seems at least some others may find it hard to remember too. Adding
> defines for these values helps us who really have good - but short
> duration - memory :]
>
> This idea comes from RFC series for ROHM BD71828 PMIC and was initially
> discussed with Linus Walleij here:
> https://lore.kernel.org/lkml/c06725c3dd34118a324907137758d8b85b3d4043.camel@fi.rohmeurope.com/
> but as this has no dependencies to BD71828 work (which probably takes a
> while) I decided to make it independent series.
>
> Patches are compile-tested only. I have no HW to really test them. Thus I'd
> appreciate carefull review. This work is mainly about converting zeros
> and ones to the new defines but it wouldn't be first time I get it
> wrong.
>
> Changelog v2:
> - squashed patches 2-61 into patch 2.
> - dropped patch 62 (can be applied later)
> - rebased (cherry-picked) changes on top of GPIO devel branch.
>
> Patch 1:
> - adds the defines
> Patch 2:
> - convert drivers to use new defines
>
> This patch series is based on GPIO tree devel branch.
Just a tip for future patchsets in case you didn't know, you case
specify the base of your patchset by using the "--base" option:
https://git-scm.com/docs/git-format-patch#_base_tree_information
This will append a "base-commit" line with the commit hash at the end of
the first message so that it's unabiguous which base you're using
(useful in case the branch changes before your patch is reviewed).
William Breathitt Gray
>
> ---
>
> Matti Vaittinen (2):
> gpio: Add definition for GPIO direction
> gpio: Use new GPIO_LINE_DIRECTION
>
> drivers/gpio/gpio-104-dio-48e.c | 5 ++++-
> drivers/gpio/gpio-104-idi-48.c | 2 +-
> drivers/gpio/gpio-104-idio-16.c | 4 ++--
> drivers/gpio/gpio-74xx-mmio.c | 5 ++++-
> drivers/gpio/gpio-amd-fch.c | 2 +-
> drivers/gpio/gpio-aspeed.c | 7 +++----
> drivers/gpio/gpio-bcm-kona.c | 6 +++---
> drivers/gpio/gpio-bd70528.c | 8 +++++---
> drivers/gpio/gpio-bd9571mwv.c | 4 +++-
> drivers/gpio/gpio-dln2.c | 6 +++---
> drivers/gpio/gpio-exar.c | 5 ++++-
> drivers/gpio/gpio-f7188x.c | 5 ++++-
> drivers/gpio/gpio-gpio-mm.c | 5 ++++-
> drivers/gpio/gpio-htc-egpio.c | 5 ++++-
> drivers/gpio/gpio-ich.c | 5 ++++-
> drivers/gpio/gpio-kempld.c | 5 ++++-
> drivers/gpio/gpio-lp873x.c | 2 +-
> drivers/gpio/gpio-lp87565.c | 5 ++++-
> drivers/gpio/gpio-madera.c | 5 ++++-
> drivers/gpio/gpio-max3191x.c | 2 +-
> drivers/gpio/gpio-merrifield.c | 5 ++++-
> drivers/gpio/gpio-mmio.c | 21 +++++++++++++++------
> drivers/gpio/gpio-mockup.c | 11 +++--------
> drivers/gpio/gpio-moxtet.c | 4 ++--
> drivers/gpio/gpio-mvebu.c | 5 ++++-
> drivers/gpio/gpio-mxs.c | 5 ++++-
> drivers/gpio/gpio-omap.c | 6 ++++--
> drivers/gpio/gpio-pca953x.c | 5 ++++-
> drivers/gpio/gpio-pci-idio-16.c | 4 ++--
> drivers/gpio/gpio-pcie-idio-24.c | 9 ++++++---
> drivers/gpio/gpio-pisosr.c | 2 +-
> drivers/gpio/gpio-pl061.c | 5 ++++-
> drivers/gpio/gpio-raspberrypi-exp.c | 5 ++++-
> drivers/gpio/gpio-rcar.c | 5 ++++-
> drivers/gpio/gpio-reg.c | 3 ++-
> drivers/gpio/gpio-sa1100.c | 5 ++++-
> drivers/gpio/gpio-sama5d2-piobu.c | 7 ++++---
> drivers/gpio/gpio-sch.c | 5 ++++-
> drivers/gpio/gpio-sch311x.c | 5 ++++-
> drivers/gpio/gpio-siox.c | 4 ++--
> drivers/gpio/gpio-stmpe.c | 5 ++++-
> drivers/gpio/gpio-tc3589x.c | 5 ++++-
> drivers/gpio/gpio-tegra.c | 5 ++++-
> drivers/gpio/gpio-tegra186.c | 4 ++--
> drivers/gpio/gpio-thunderx.c | 5 ++++-
> drivers/gpio/gpio-tpic2810.c | 2 +-
> drivers/gpio/gpio-tps65086.c | 2 +-
> drivers/gpio/gpio-tps65912.c | 4 ++--
> drivers/gpio/gpio-tps68470.c | 6 +++---
> drivers/gpio/gpio-tqmx86.c | 5 ++++-
> drivers/gpio/gpio-ts4900.c | 5 ++++-
> drivers/gpio/gpio-twl4030.c | 10 +++++-----
> drivers/gpio/gpio-twl6040.c | 3 +--
> drivers/gpio/gpio-uniphier.c | 5 ++++-
> drivers/gpio/gpio-wcove.c | 7 +++++--
> drivers/gpio/gpio-ws16c48.c | 5 ++++-
> drivers/gpio/gpio-xgene.c | 5 ++++-
> drivers/gpio/gpio-xra1403.c | 5 ++++-
> drivers/gpio/gpio-xtensa.c | 4 ++--
> drivers/gpio/gpio-zynq.c | 7 +++++--
> include/linux/gpio/driver.h | 3 +++
> 61 files changed, 214 insertions(+), 102 deletions(-)
>
> --
> 2.21.0
>
>
> --
> Matti Vaittinen, Linux device drivers
> ROHM Semiconductors, Finland SWDC
> Kiviharjunlenkki 1E
> 90220 OULU
> FINLAND
>
> ~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
> Simon says - in Latin please.
> ~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
> Thanks to Simon Glass for the translation =]
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2019-11-06 15:11 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-06 8:51 [alsa-devel] [PATCH v2 0/2] Add definition for GPIO direction Matti Vaittinen
2019-11-06 8:51 ` [alsa-devel] [PATCH v2 1/2] gpio: " Matti Vaittinen
2019-11-07 8:20 ` Linus Walleij
2019-11-06 8:54 ` [alsa-devel] [PATCH v2 2/2] gpio: Use new GPIO_LINE_DIRECTION Matti Vaittinen
2019-11-06 14:22 ` Andy Shevchenko
2019-11-06 14:43 ` William Breathitt Gray
2019-11-06 17:35 ` Kuppuswamy Sathyanarayanan
2019-11-06 12:08 ` [alsa-devel] [PATCH v2 0/2] Add definition for GPIO direction Sebastian Reichel
2019-11-06 12:25 ` Vaittinen, Matti
2019-11-06 14:24 ` andriy.shevchenko
2019-11-07 8:22 ` Linus Walleij
2019-11-06 14:39 ` William Breathitt Gray [this message]
2019-11-06 17:28 ` Vaittinen, Matti
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=20191106143938.GA3176@icarus \
--to=vilhelm.gray@gmail.com \
--cc=alexandre.torgue@st.com \
--cc=alsa-devel@alsa-project.org \
--cc=andrew@aj.id.au \
--cc=andriy.shevchenko@linux.intel.com \
--cc=bamv2005@gmail.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=bgolaszewski@baylibre.com \
--cc=ckeepax@opensource.cirrus.com \
--cc=david.daney@cavium.com \
--cc=f.fainelli@gmail.com \
--cc=festevam@gmail.com \
--cc=grygorii.strashko@ti.com \
--cc=info@metux.net \
--cc=joel@jms.id.au \
--cc=jonathanh@nvidia.com \
--cc=kernel@pengutronix.de \
--cc=khilman@kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-aspeed@lists.ozlabs.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux-tegra@vger.kernel.org \
--cc=ludovic.desroches@microchip.com \
--cc=marek.behun@nic.cz \
--cc=marek.vasut+renesas@gmail.com \
--cc=matti.vaittinen@fi.rohmeurope.com \
--cc=mazziesaccount@gmail.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=michal.simek@xilinx.com \
--cc=patches@opensource.cirrus.com \
--cc=ptyser@xes-inc.com \
--cc=rf@opensource.cirrus.com \
--cc=rjui@broadcom.com \
--cc=s.hauer@pengutronix.de \
--cc=sathyanarayanan.kuppuswamy@linux.intel.com \
--cc=sbranden@broadcom.com \
--cc=shawnguo@kernel.org \
--cc=ssantosh@kernel.org \
--cc=t.scherer@eckelmann.de \
--cc=thierry.reding@gmail.com \
--cc=u.kleine-koenig@pengutronix.de \
--cc=yamada.masahiro@socionext.com \
/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).