From: Linus Walleij <linus.walleij@linaro.org>
To: linux-gpio@vger.kernel.org, Alexandre Courbot <acourbot@nvidia.com>
Cc: "Linus Walleij" <linus.walleij@linaro.org>, "Michael Büsch" <m@bues.ch>
Subject: [PATCH 01/23] gpio: remove deps on ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
Date: Wed, 20 Apr 2016 10:57:59 +0200 [thread overview]
Message-ID: <1461142701-21096-2-git-send-email-linus.walleij@linaro.org> (raw)
In-Reply-To: <1461142701-21096-1-git-send-email-linus.walleij@linaro.org>
The GPIOLIB symbol currently require that
ARCH_WANT_OPTIONAL_GPIOLIB or ARCH_REQUIRE_GPIOLIB is selected
to be selectable.
The ARCH_REQUIRE_GPIOLIB does only one thing: select GPIOLIB.
This is just confusing: architectures that want GPIOLIB should
be able to configure it in no matter what, and those who
require it should just select GPIOLIB.
It also creates problems for drivers that need to state
"select GPIOLIB" to get dependencies: those depend on the
selected architecture to select
ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB first, and will cause
compile errors for the few archs that state neither.
These intermediary symbols need to go.
As a first step, remove the dependencies so that:
- ARCH_WANT_OPTIONAL_GPIOLIB becomes a noop (GPIOLIB will be
available for everyone) and
- "select ARCH_REQUIRE_GPIOLIB" can be replaced by just
"select GPIOLIB"
After this patch we can follow up with patches cleaning up the
architectures one-by one and eventually remove the
ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB symbols altogether.
Reported-by: Michael Hennerich <michael.hennerich@analog.com>
Cc: Michael Büsch <m@bues.ch>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
drivers/gpio/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index f73f26b43532..a68d83808f37 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -33,7 +33,6 @@ config ARCH_REQUIRE_GPIOLIB
menuconfig GPIOLIB
bool "GPIO Support"
- depends on ARCH_WANT_OPTIONAL_GPIOLIB || ARCH_REQUIRE_GPIOLIB
help
This enables GPIO support through the generic GPIO library.
You only need to enable this, if you also want to enable
--
2.4.11
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-04-20 8:58 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-20 8:57 [PATCH 00/23] Kill ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
2016-04-20 8:57 ` Linus Walleij [this message]
2016-04-20 8:58 ` [PATCH 02/23] arc: select GPIOLIB directly Linus Walleij
2016-04-22 5:16 ` Vineet Gupta
2016-04-26 12:09 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 03/23] ARM: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
2016-04-20 8:58 ` [PATCH 04/23] arm64: " Linus Walleij
2016-04-20 8:58 ` [PATCH 05/23] avr32: do away with ARCH_REQUIRE_GPIOLIB Linus Walleij
2016-04-20 9:09 ` Hans-Christian Noren Egtvedt
2016-04-26 11:38 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 06/23] blackfin: " Linus Walleij
2016-04-20 8:58 ` [PATCH 07/23] cris: " Linus Walleij
2016-04-21 14:05 ` Jesper Nilsson
2016-04-26 12:04 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 08/23] hexagon: update TODO list Linus Walleij
2016-04-20 8:58 ` [PATCH 09/23] m68k: do away with ARCH_REQUIRE_GPIOLIB Linus Walleij
2016-04-21 1:12 ` Greg Ungerer
2016-04-26 12:02 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 10/23] MIPS: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
2016-05-10 8:05 ` Ralf Baechle
2016-04-20 8:58 ` [PATCH 11/23] openrisc: do away with ARCH_REQUIRE_GPIOLIB Linus Walleij
2016-04-20 8:58 ` [PATCH 12/23] powerpc: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
2016-04-20 8:58 ` [PATCH 13/23] sh: " Linus Walleij
2016-04-20 8:58 ` [PATCH 14/23] unicore: do away with ARCH_REQUIRE_GPIOLIB Linus Walleij
2016-04-20 8:58 ` [PATCH 15/23] x86: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
2016-04-20 8:58 ` [PATCH 16/23] alpha: remove ARCH_WANT_OPTIONAL_GPIOLIB Linus Walleij
2016-04-20 16:15 ` Matt Turner
2016-04-26 11:59 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 17/23] ia64: " Linus Walleij
2016-04-20 8:58 ` [PATCH 18/23] metag: " Linus Walleij
[not found] ` <1461142701-21096-19-git-send-email-linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2016-04-20 9:26 ` James Hogan
[not found] ` <20160420092602.GX7859-4bYivNCBEGTR3KXKvIWQxtm+Uo4AYnCiHZ5vskTnxNA@public.gmane.org>
2016-04-26 11:40 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 19/23] microblaze: " Linus Walleij
2016-04-20 8:58 ` [PATCH 20/23] nios2: " Linus Walleij
2016-04-21 7:46 ` Ley Foon Tan
2016-04-26 12:03 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 21/23] sparc: " Linus Walleij
2016-04-21 18:58 ` David Miller
2016-04-26 12:07 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 22/23] xtensa: " Linus Walleij
2016-04-20 19:00 ` Max Filippov
2016-04-26 12:00 ` Linus Walleij
2016-04-20 8:58 ` [PATCH 23/23] gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB 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=1461142701-21096-2-git-send-email-linus.walleij@linaro.org \
--to=linus.walleij@linaro.org \
--cc=acourbot@nvidia.com \
--cc=linux-gpio@vger.kernel.org \
--cc=m@bues.ch \
/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).