Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <cb6aa00b1901abb572e69e218a5500f2cd1561ce.camel@linux.ibm.com>
Subject: Re: [PATCH v4 05/41] counter: add HAS_IOPORT dependencies
From:   Niklas Schnelle <schnelle@linux.ibm.com>
To:     William Breathitt Gray <william.gray@linaro.org>
Cc:     Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Bjorn Helgaas <bhelgaas@google.com>,
        Uwe =?ISO-8859-1?Q?Kleine-K=F6nig?= 
        <u.kleine-koenig@pengutronix.de>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Alan Stern <stern@rowland.harvard.edu>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        Geert Uytterhoeven <geert@linux-m68k.org>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
        linux-pci@vger.kernel.org, Arnd Bergmann <arnd@kernel.org>,
        linux-iio@vger.kernel.org
Date:   Mon, 22 May 2023 12:42:15 +0200
In-Reply-To: <ZGeF1K0Yxu9lTgN2@fedora>
References: <20230516110038.2413224-1-schnelle@linux.ibm.com>
         <20230516110038.2413224-6-schnelle@linux.ibm.com> <ZGbQYzXK8InMqkxu@fedora>
         <6f4d672ba7136f2b01ea9ee69687b16168eddb8d.camel@linux.ibm.com>
         <231dcebc57c2e43ba65d007b60d3d446d9ed71c8.camel@linux.ibm.com>
         <abc02dc2af7563ae26bf0d0ddd927d9b4a21dda3.camel@linux.ibm.com>
         <ZGeF1K0Yxu9lTgN2@fedora>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.1 (3.48.1-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

On Fri, 2023-05-19 at 10:21 -0400, William Breathitt Gray wrote:
> On Fri, May 19, 2023 at 03:39:57PM +0200, Niklas Schnelle wrote:
> > On Fri, 2023-05-19 at 15:38 +0200, Niklas Schnelle wrote:
> > > On Fri, 2023-05-19 at 15:17 +0200, Niklas Schnelle wrote:
> > > > On Thu, 2023-05-18 at 21:26 -0400, William Breathitt Gray wrote:
> > > > > On Tue, May 16, 2023 at 01:00:01PM +0200, Niklas Schnelle wrote:
> > > > > > In a future patch HAS_IOPORT=3Dn will result in inb()/outb() an=
d friends
> > > > > > not being declared. We thus need to add HAS_IOPORT as dependenc=
y for
> > > > > > those drivers using them.
> > > > > >=20
> > > > > > Co-developed-by: Arnd Bergmann <arnd@kernel.org>
> > > > > > Signed-off-by: Arnd Bergmann <arnd@kernel.org>
> > > > > > Signed-off-by: Niklas Schnelle <schnelle@linux.ibm.com>
> > > > >=20
> > > > > Hi Niklas,
> > > > >=20
> > > > > The change itself is fine, but please update the description to r=
eflect
> > > > > that this is adding a depends on HAS_IOPORT_MAP rather than HAS_I=
OPORT,
> > > > > along with the reason why it's needed (i.e. devm_ioport_map() is =
used).
> > > > >=20
> > > > > Thanks,
> > > > >=20
> > > > > William Breathitt Gray
> > > > >=20
> > > > >=20
> > > >=20
> > > > Right, this clearly needs adjustment. I went with the following com=
mit
> > > > message for v5:
> > > >=20
> > > > "counter: add HAS_IOPORT_MAP dependency
> > > >=20
> > > > The 104_QUAD_8 counter driver uses devm_ioport_map() without depend=
ing
> > > > on HAS_IOPORT_MAP. This causes compilation to fail on platforms suc=
h as
> > > > s390 which do not support I/O port mapping. Add the missing
> > > > HAS_IOPORT_MAP dependency to fix this."
> > > >=20
> > >=20
> > > Just noticed this isn't entirely correct. As devm_ioport_map() has an
> > > empty stub for HAS_IOPORT_MAP=3Dn this doesn't lead to a compile erro=
r it
> > > just doesn't work. Will reword to "This causes the driver to not be
> > > useable on platforms ..."
> >=20
> > s/useable/usable/
>=20
> 104_QUAD_8 has an explicit dependency on PC104 and X86, so I don't think
> it would ever be used outside of x86 platforms. Does it still make sense
> to have the HAS_IOPORT_MAP dependency in this case?
>=20
> William Breathitt Gray

Well, yes and no, you're right that it doesn't really cause compile
issues despite the "|| COMPILE_TEST" albeit the code could never work.
Still, I'd add the dependency. At the very least it serves as
documentation and maybe in the future someone will want to remove those
empty stubs for HAS_IOPORT_MAP=3Dn.

Thanks
Niklas