From mboxrd@z Thu Jan 1 00:00:00 1970 From: William Breathitt Gray Subject: Re: [PATCH v2 2/4] gpio: Allow PC/104 devices on X86_64 Date: Fri, 3 Jun 2016 17:12:44 -0400 Message-ID: <20160603211244.GA6831@sophia> References: <20160603205703.GC11253@amd> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20160603205703.GC11253@amd> Sender: linux-watchdog-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Pavel Machek Cc: gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org, akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-next-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Guenter Roeck , Linus Walleij List-Id: linux-gpio@vger.kernel.org On Fri, Jun 03, 2016 at 10:57:03PM +0200, Pavel Machek wrote: >Hi! > >> With the introduction of the ISA_BUS_API Kconfig option, ISA-style >> drivers may be built for X86_64 architectures. This patch changes the >> ISA Kconfig option dependency of the PC/104 drivers to ISA_BUS_API, thus >> allowing them to build for X86_64 as they are expected to. >> >> Cc: Guenter Roeck >> Cc: Linus Walleij >> Signed-off-by: William Breathitt Gray >> --- >> drivers/gpio/Kconfig | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig >> index 48da857..dc6da77 100644 >> --- a/drivers/gpio/Kconfig >> +++ b/drivers/gpio/Kconfig >> @@ -530,7 +530,7 @@ menu "Port-mapped I/O GPIO drivers" >> >> config GPIO_104_DIO_48E >> tristate "ACCES 104-DIO-48E GPIO support" >> - depends on ISA >> + depends on ISA_BUS_API >> select GPIOLIB_IRQCHIP >> help >> Enables GPIO support for the ACCES 104-DIO-48E series > >Should we do "depends on PC104" here, because that is what it really >means, and have PC104 enabled when ISA_BUS_API is enabled or something >like that? Since the functionality remains the same, I'm a bit indifferent to that change; as long as the driver builds for systems in which it's intended to be used, I'm satisfied. Differentiating between PC/104 and ISA may be a pointless endeavor though since both buses appear the same to software. But if it is better to differentiate between devices as such, then I see little harm in adding a PC104 Kconfig option which follows the ISA_BUS_API Kconfig option. William Breathitt Gray -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html