Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35;
Date:   Sun, 29 Oct 2023 18:17:06 +0100
From:   Wolfram Sang <wsa@kernel.org>
To:     Marius Hoch <mail@mariushoch.de>
Cc:     Jean Delvare <jdelvare@suse.com>, linux-i2c@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] i2c: i801: Force no IRQ for Dell Latitude E7450
Message-ID: <ZT6TknMc+34NCqPG@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
        Marius Hoch <mail@mariushoch.de>, Jean Delvare <jdelvare@suse.com>,
        linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20230514103634.235917-1-mail@mariushoch.de>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="LVHbfEbx9LzAVSb2"
Content-Disposition: inline
In-Reply-To: <20230514103634.235917-1-mail@mariushoch.de>
Precedence: bulk


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

On Sun, May 14, 2023 at 12:36:32PM +0200, Marius Hoch wrote:
> The Dell Latitude E7450 uses IRQ 18 for the accelerometer,
> but also claims that the SMBus uses IRQ 18. This will
> result in:
>=20
> i801_smbus 0000:00:1f.3: PCI INT C: failed to register GSI
> i801_smbus 0000:00:1f.3: Failed to enable SMBus PCI device (-16)
> i801_smbus: probe of 0000:00:1f.3 failed with error -16
>=20
> Force the SMBus IRQ to IRQ_NOTCONNECTED in this case, so that
> we fall back to polling, which also seems to be what the (very
> dated) Windows 7 drivers on the Dell Latitude E7450 do.
>=20
> This was tested on Dell Latitude E7450.
>=20
> I chose to explicitly list all affected devices here, but
> alternatively it would be possible to do this programmatically:
> If the initial pcim_enable_device fails and we're on (any)
> Dell Latitude, re-try with IRQ_NOTCONNECTED.
>=20
> Marius Hoch (2):
>   i2c: i801: Force no IRQ for Dell Latitude E7450
>   i2c: i801: Force no IRQ for further Dell Latitudes
>=20
>  drivers/i2c/busses/i2c-i801.c | 34 ++++++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)

Where are we with this patch set? Other solution found already?
Discussion stalled? Too much other things going on?


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

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmU+k5IACgkQFA3kzBSg
KbafUxAAgd58sc/w7TNN3XnqkARACN+uSB+NhVdhfF7g2NJbNSpUzXWttkaBD8L8
fqlX8gD9sNHs4cBL0MlCnUPDoA67flTb/q96Sk6zadnAL4eNW4v1bQpxHK1StNqR
8bEeEYAyoUGX6Xe2rdYiESHjtFNy2YCMX6pzAvLJn+yHwTrMcrYFU/VtOHi7HfA/
qiTUZJbLfrl1OXxqcuEBUyYB2WhXed/SRsO0NdKhkpwn90V/s4EwrlVXjGpuUKdn
0mI+OrCTt8umBQDn023FWtvUX8lWww4KQOo2szsMD/deCt4YzsYLO00Aj1z/T+GY
YvYYo4R1W9zVZnbXq7w+nihYIApiaEN3QiICzbo4YvubRqyMeJ1B+vSutIrLVFKS
gT2T90sMtWKxXjS/zAFSCB63dgabml8QB1o1SqdsLHL9l7vrnKK7TKXDrzo5I+Jg
z/2mdCsxyaPHMjHRhk9dKQ3BtOHPSXsKajePtkgj1RYuOpSbI4EMt92l+luVgCUK
s5miafDazxyru0b8F7zccy0FvMwXhJSMHChlfzjVbAvjwr/s79bzPEPuRNRoqJFB
QXeODrTouBci68cqnV+eq3KL334+2mNB/yiVIWyckrcrmVdzXye+f1HshKNFwL2y
XtPHFHG1lDWLVdbIq3O6kEm+Skr/5cDWSerAT8q/g5a1HaIJzM0=
=yPh2
-----END PGP SIGNATURE-----

--LVHbfEbx9LzAVSb2--