Received-SPF: pass (google.com: domain of linux-kernel+bounces-162042-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Mon, 29 Apr 2024 12:21:05 +0200
From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>, 
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>, linux-kernel@vger.kernel.org, 
	Krzysztof Kozlowski <krzk@kernel.org>, Wolfram Sang <wsa+renesas@sang-engineering.com>, 
	linux-i2c@vger.kernel.org, kernel@pengutronix.de
Subject: Re: [PATCH RFC] i2c: Add a void pointer to i2c_device_id
Message-ID: <youkuwbynndjpcoux2zaxwjp5gquj647leub3bat37a4wtho6p@ypir6ay3vhaw>
References: <20240426213832.915485-2-u.kleine-koenig@pengutronix.de>
 <Zi9gRVVw7qbKSL5k@smile.fi.intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="yfivxb4d4fdixn6l"
Content-Disposition: inline
In-Reply-To: <Zi9gRVVw7qbKSL5k@smile.fi.intel.com>


--yfivxb4d4fdixn6l
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hello,

On Mon, Apr 29, 2024 at 11:54:29AM +0300, Andy Shevchenko wrote:
> On Fri, Apr 26, 2024 at 11:38:33PM +0200, Uwe Kleine-K=F6nig wrote:
> > Traditionally a struct i2c_device_id has a kernel_ulong_t member to
> > store some chip variant data. Some drivers use it to store an enum,
> > others to store a pointer. In the latter case there is some ugly(?)
> > casting involved. To improve that, add a void pointer in an anonymous
> > union together with the integer driver_data.
> >=20
> > This way a i2c_device_id requires usage of a designated initializer when
> > the driver_data or data member should be initialized, but IMHO that's
> > fine and you might even consider that an advantage.
>=20
> ...
>=20
> >  static const struct i2c_device_id wlf_gf_module_id[] =3D {
> > -	{ "wlf-gf-module", 0 },
> > +	{ "wlf-gf-module", },
>=20
> In such cases the inner comma is redundant as well.

I would tend to keep the comma, but no strong opinion on my side.
If another member init is added later, the line has to be touched
anyhow, but in the layout:

	... =3D {
		{
			"wlf-gf-module",
		},
		{ }
	}

I'd keep it for sure.


> >  	{ }
> >  };
>=20
> ...
>=20
> In general idea might be okay, but I always have the same Q (do we have it
> being clarified in the documentation, btw): is an ID table the ABI or not?
> In another word, how should we treat the changes there, because ID tables
> are being used by the user space tools.

Note that the layout doesn't change and the traditional interpretation
of the data still works fine. Or do you see something that I miss?

Best regards
Uwe

--=20
Pengutronix e.K.                           | Uwe Kleine-K=F6nig            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

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

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

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmYvdJAACgkQj4D7WH0S
/k5SrAf/UQu0T91ViSkNd60kQV/AoTviGlvx8F5UlQbv/Vqu3BH9V2qGdpKpBhZh
rp4lcTa2z6SegBoH6yctMYp9vfPWE1MB0URyTm2Hvzj9zCxOB7bu2gtuAzYeKvLG
UyAzHCqa9K+AwWgSAb/rH3aqy3pHwxL3JoB+FSgcu75SBgpPkRukZgJDrCB7gWzb
AUlb0Z3LrSjRUGSVvxxSxNy5S69CSDMQ20cvK8of7mno8uL0MYJ/RKCRfx+XGsvm
V0HZwZgSI+D/oNCIKLtB9DoH/3W9xGCZIgnfBDWbtKGizCt9A9bDujaR8cYAdGrR
dM19ko2kATgu3pL2e/Jf1ly3N97j1Q==
=eVcX
-----END PGP SIGNATURE-----

--yfivxb4d4fdixn6l--