From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Linus Walleij <linus.walleij@linaro.org>,
Guenter Roeck <linux@roeck-us.net>,
"Maciej S . Szmigiero" <mail@maciej.szmigiero.name>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] gpio: winbond: fix ISA_BUS_API dependency
Date: Thu, 11 Jan 2018 08:00:45 -0500 [thread overview]
Message-ID: <20180111130045.GA23562@sophia> (raw)
In-Reply-To: <20180111083600.4121805-1-arnd@arndb.de>
On Thu, Jan 11, 2018 at 09:35:15AM +0100, Arnd Bergmann wrote:
>This particular driver appears to be different from the other ISA_BUS_API
>based drivers, in that it is not normally an add-on card (ISA or PC104)
>but instead is an LPC-attached component on the mainboard. We already
>support other functionality provided by this chip, at least
>drivers/watchdog/w83627hf_wdt.c and drivers/hwmon/w83627ehf.c, plus
>there is a discovery function for this hardware in
>drivers/parport/parport_pc.c.
>
>If we want to use this driver without having to enable CONFIG_EXPERT,
>it might be better to not use the isa_bus_type for it, but rather
>turn it into a platform_driver, acpi_driver or add an MFD for it that
>is shared with the wdt and hwmon portions and does the probing.
I think there is some merit to reinvestigate an MFD driver at a later
point; Super I/O chips by their design typically control various
multiple devices, which I believe fits in well with the MFD paradigm.
Although other existing Super I/O drivers in the kernel do not make use
of MFD, this doesn't necessarily mean it is not apt -- it may be
appropriate to refactor those drivers as well to take advantage of MFD.
As far as this particular patch is concerned however, I'm going to
suggest waiting for the ISA_BUS_API Kconfig fixes
((https://patchwork.ozlabs.org/project/linux-gpio/list/?series=20657)
to be pulled-in; applying this patch now will just require a revert
later once the patchset makes it into the tree.
William Breathitt Gray
prev parent reply other threads:[~2018-01-11 13:00 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-11 8:35 [PATCH] gpio: winbond: fix ISA_BUS_API dependency Arnd Bergmann
2018-01-11 9:18 ` Linus Walleij
2018-01-11 12:40 ` Maciej S. Szmigiero
2018-01-11 12:42 ` William Breathitt Gray
2018-01-11 13:00 ` William Breathitt Gray [this message]
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=20180111130045.GA23562@sophia \
--to=vilhelm.gray@gmail.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=arnd@arndb.de \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=mail@maciej.szmigiero.name \
/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).