From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1756637AbbJ2IXU (ORCPT <rfc822;w@1wt.eu>);
	Thu, 29 Oct 2015 04:23:20 -0400
Received: from mga03.intel.com ([134.134.136.65]:1521 "EHLO mga03.intel.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1756541AbbJ2IXP (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Thu, 29 Oct 2015 04:23:15 -0400
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.20,213,1444719600"; 
   d="scan'208";a="822107646"
Date: Thu, 29 Oct 2015 10:22:59 +0200
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: William Breathitt Gray <vilhelm.gray@gmail.com>,
        Andreas Bofjall <andreas@gazonk.org>,
        Simon Guinot <simon.guinot@sequanux.org>,
        Diego Elio =?iso-8859-1?Q?Petten=F2?= <flameeyes@flameeyes.eu>,
        Daniel Krueger <daniel.krueger@systec-electronic.com>,
        Bruno Randolf <br1@einfach.org>,
        Vivien Didelot <vivien.didelot@savoirfairelinux.com>,
        Alexandre Courbot <gnurou@gmail.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>
Subject: Re: [PATCH v10] gpio: Add GPIO support for the ACCES 104-IDIO-16
Message-ID: <20151029082259.GD1548@lahna.fi.intel.com>
References: <20151019165914.GA22867@sophia>
 <CACRpkdbQyed0S0Y6MSt35LuOekFprvbPMDpsGku-cBXBt=9j0Q@mail.gmail.com>
 <20151027142343.GI1548@lahna.fi.intel.com>
 <CACRpkdbs8US2m1+hEmXbJ9JJT15-iRHnO2L4bk5qeNRgb2vbKg@mail.gmail.com>
 <20151027165134.GK1548@lahna.fi.intel.com>
 <CACRpkdZBnmE2F7i2PYuxvO4cpYsGnrcGr0HMKg9WAPVMGnDBTw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CACRpkdZBnmE2F7i2PYuxvO4cpYsGnrcGr0HMKg9WAPVMGnDBTw@mail.gmail.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Oct 28, 2015 at 04:11:41PM +0100, Linus Walleij wrote:
> On Tue, Oct 27, 2015 at 5:51 PM, Mika Westerberg
> <mika.westerberg@linux.intel.com> wrote:
> > On Tue, Oct 27, 2015 at 05:42:23PM +0100, Linus Walleij wrote:
> >> On Tue, Oct 27, 2015 at 3:23 PM, Mika Westerberg
> >> <mika.westerberg@linux.intel.com> wrote:
> >> > On Tue, Oct 27, 2015 at 11:06:58AM +0100, Linus Walleij wrote:
> >> >> > +menu "ISA GPIO drivers"
> >> >> > +
> >> >> > +config GPIO_104_IDIO_16
> >> >> > +       tristate "ACCES 104-IDIO-16 GPIO support"
> >> >> > +       depends on X86
> >> >> > +       help
> >> >> > +         Enables GPIO support for the ACCES 104-IDIO-16 family.
> >> >> > +
> >> >> > +endmenu
> >> >>
> >> >> I have five other drivers using port-based I/O:
> >> >>
> >> >> gpio-f7188x.c
> >> >> gpio-it87.c
> >> >> gpio-sch.c
> >> >> gpio-sch311x.c
> >> >> gpio-ts5500.c
> >> >>
> >> >> Do you think it makes sense to move all of these in under
> >> >> the "ISA GPIO drivers" menu as well? I guess they all require
> >> >> ISA after all.
> >> >>
> >> >> (Added their maintainers to the To: line so I can get some input
> >> >> on this.)
> >> >
> >> > Not all of them actually require ISA. For example gpio-sch is part of an
> >> > MFD device which is a PCI device. PCI bus also has notion of I/O ports.
> >>
> >> Hm, is there a more fitting symbol we should sort it under, which
> >> means thingofabob-that-has-portmapped I/O?
> >
> > Well, there's already "Memory mapped GPIO drivers" so how about "I/O
> > mapped GPIO drivers"?
> 
> Sure, "Port-mapped I/O GPIO drivers"
> 
> But what should I put as "depends" for that submenu, that is maybe
> the real question.
> 
> Would (ISA || PCI) suffice?

x86 has I/O space regardless of ISA or PCI (even though most probably it
also has ISA and PCI). Looking at the drivers in question they seem to
be x86 drivers so maybe "depends on X86"?