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;
Date:   Sat, 14 May 2022 16:06:41 +0200
From:   Wolfram Sang <wsa@kernel.org>
To:     Niklas Schnelle <schnelle@linux.ibm.com>
Cc:     Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
        linux-pci@vger.kernel.org, Arnd Bergmann <arnd@kernel.org>,
        "open list:I2C SUBSYSTEM HOST DRIVERS" <linux-i2c@vger.kernel.org>
Subject: Re: [RFC v2 12/39] i2c: add HAS_IOPORT dependencies
Message-ID: <Yn+3cZ02UDP19XGu@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
        Niklas Schnelle <schnelle@linux.ibm.com>,
        Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
        linux-pci@vger.kernel.org, Arnd Bergmann <arnd@kernel.org>,
        "open list:I2C SUBSYSTEM HOST DRIVERS" <linux-i2c@vger.kernel.org>
References: <20220429135108.2781579-1-schnelle@linux.ibm.com>
 <20220429135108.2781579-22-schnelle@linux.ibm.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="1Ae8AEsrKJseqCtm"
Content-Disposition: inline
In-Reply-To: <20220429135108.2781579-22-schnelle@linux.ibm.com>
Precedence: bulk


--1Ae8AEsrKJseqCtm
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Apr 29, 2022 at 03:50:19PM +0200, Niklas Schnelle wrote:
> In a future patch HAS_IOPORT=3Dn will result in inb()/outb() and friends
> not being declared. We thus need to add HAS_IOPORT as dependency for
> those drivers using them.
>=20
> Co-developed-by: Arnd Bergmann <arnd@kernel.org>
> Signed-off-by: Niklas Schnelle <schnelle@linux.ibm.com>

What has changed since RFC v1?

> @@ -1232,6 +1233,7 @@ config I2C_CP2615
>  config I2C_PARPORT
>  	tristate "Parallel port adapter"
>  	depends on PARPORT
> +	depends on HAS_IOPORT
>  	select I2C_ALGOBIT
>  	select I2C_SMBUS
>  	help

Didn't you want to drop this one?


--1Ae8AEsrKJseqCtm
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmJ/t3AACgkQFA3kzBSg
KbbldxAAijJWDKJn8oOjI9OPknBQkZs2iRuzoaBa6ZCu999lpIIAc+qAGmcGU67D
X/Qqcolemb14Pz2vDFjR9Sh1XH3UxwZ6/EA9tzwLf/LxMGC+PI9bzEi+3Li78GBx
XX0ZXenzcJoxBOihybOGxRnMgBfgflXOhfL/jpaD1Wdh6D51nS0Ygc7abGnEnZ2t
LbCeVUOeMSfAAolitmruD3uboSY65agsXrCtO0M3q/fU2CCZS5B+B9kGAYteB6BC
3ySbZqA9IPbPFgRjsAhVfcUkIHMrm5IBAG64FhacqFG+DcoHin8W0UbCBV2OByTE
khy6rI1VhJ3PmwmvN3X6Nbsi6OH3gur+MgqwnwyrgwZ5ILjxMaN2Uy7Vbr5B7MMh
YGVA0H7hX38DH4MaAsVCYr8JZ/sp42iydw1rhEVgNFqkGK+6dRtmkxkNGQh4P6Rb
mtr7Caxf+haa8YnKXcEMkHjDmkjpYAeJAhK1mUC6D/slsUlGM75FsJ/NdDZ1Ccob
Iq9ZUe0eZHCTDaegfrSdI1DmIYSbT6ZupJN40Wbt/Rv41HvjB8GGPyUI0y0zGpSx
tGbDB0wsm6ZvEtzOhAg6tiSJW/rjh4I3zuuutPYKYGJgva/JxITYWXRRg5qnqfW4
Ja3oXwiYCpDPT2ZsIqdegh5FTayAdecztDIp8YeYp0eVqRDabO4=
=ojBv
-----END PGP SIGNATURE-----

--1Ae8AEsrKJseqCtm--