Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754566AbbEOJVV (ORCPT ); Fri, 15 May 2015 05:21:21 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:16230 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754462AbbEOJVQ (ORCPT ); Fri, 15 May 2015 05:21:16 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Fri, 15 May 2015 02:18:44 -0700 Date: Fri, 15 May 2015 11:21:05 +0200 From: Thierry Reding To: Javier Martinez Canillas CC: Olof Johansson , David Airlie , "Ajay Kumar" , Daniel Vetter , Krzysztof Kozlowski , Doug Anderson , , , Subject: Re: [PATCH 1/1] drm/bridge: ptn3460: Fix I2C ID table to match the reported modalias Message-ID: <20150515092103.GF4984@ulmo.nvidia.com> References: <1431613889-32737-1-git-send-email-javier.martinez@collabora.co.uk> MIME-Version: 1.0 In-Reply-To: <1431613889-32737-1-git-send-email-javier.martinez@collabora.co.uk> X-NVConfidentiality: public User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [10.2.70.208] X-ClientProxiedBy: UKMAIL102.nvidia.com (10.26.138.15) To UKMAIL101.nvidia.com (10.26.138.13) Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="1Y7d0dPL928TPQbc" Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2549 Lines: 71 --1Y7d0dPL928TPQbc Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, May 14, 2015 at 04:31:29PM +0200, Javier Martinez Canillas wrote: > I2C drivers that support OF, have both an I2C and OF device ID tables > that are used to fill the supported module aliases. But currently the > I2C core only uses the OF table to match a device with a driver and > the aliases information are always reported in the form i2c:. >=20 > The client->name is used as the name postfix and when booting with OF > this is obtained with of_modalias_node() which drops the compatible > string vendor prefix. >=20 > So for I2C drivers, the I2C and OF device ID tables should be keep in > sync in order to make module auto-loading to work but the I2C device > entries shouldn't have the vendor prefix since that is not reported. >=20 > Before this patch: >=20 > MODALIAS=3Di2c:ptn3460 >=20 > $ modinfo | grep alias > alias: i2c:nxp,ptn3460 > alias: of:N*T*Cnxp,ptn3460* >=20 > After this patch: >=20 > MODALIAS=3Di2c:ptn3460 >=20 > $ modinfo | grep alias > alias: i2c:ptn3460 > alias: of:N*T*Cnxp,ptn3460* >=20 > Signed-off-by: Javier Martinez Canillas > --- > drivers/gpu/drm/bridge/ptn3460.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied, thanks. Thierry --1Y7d0dPL928TPQbc Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVVbp9AAoJEN0jrNd/PrOh0X0QAKdZMamtIk3nWGAX+m3VuR/Q hoMzSHdVr3IKW4Qja5LVvLP7hdYtWQ4ECN28gztwt+IhFGe7CjcyZZCrcwhO1mLn oX3zv63ZpZIYDZ6wF/jQYEzNCqzYMEyMGcnSFbQx8i5xAxFIIHDB7ObHpvezciUB L5ZKdDdGKQ1ZYYqWf0/b2XOZSvuNudvsD8f5sneyRW/Ng4dwHYGORp8USslhYYMY 4edsVmf1EaAAQa5cw/i3IJKLqPeOnmNZDJ0Lpn25/Y36M28XUhBw4FIqCLwXASA4 aq+6VNAU6wXeFzwPMJyc/WWhNUFpf8YSCIkEcdkGQSQwCEBTRBM7ovf9jUEfNrPb jhl/MzRDx34oihKKyReo7RWpv/XiRXUSQoxNDxo/r9LL99pfr3CJ6rWF0NuDfUUs WfW6VcidKoG4zaGDjCRnqaD/Z+rUjYI6wYh+Rf5p6c7GXwwNK1k33kvPdbp6qV06 mOMgIt0eAPWjVWEcjlNC8JKRz2mLzBIIBHUnUbWYq38ahUp9ptUbcs01CUNz6qZ0 TtUWX4M/Yql9CJwyRTVLOaafo2ZAM9TRwz+N/OQvYIyGaJg8Ob6LeQu321oYfz/r WhyOA0FetmgmoB0GTtgS+fnpO47u1fCrksimVIn1oBGvePE9UUPST8eESg34lsty sPMm2QvltU1be/EJJQqu =p3e9 -----END PGP SIGNATURE----- --1Y7d0dPL928TPQbc-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/