Re: [PATCH] drm: Use named initializers for arrays of i2c_device_data

From: Laurent Pinchart

Date: Mon May 18 2026 - 06:10:20 EST

Hi Uwe,

Thank you for the patch.

On Mon, May 18, 2026 at 12:04:01PM +0200, Uwe Kleine-König (The Capable Hub) wrote:
> While being less compact, using named initializers allows to more easily
> see which members of the structs are assigned which value without having
> to lookup the declaration of the struct. And it's also more robust
> against changes to the struct definition.
>
> The mentioned robustness is relevant for a planned change to struct
> i2c_device_id that replaces .driver_data by an anonymous union.
>
> While touching all these arrays, unify usage of whitespace in the list
> terminator and drop trailing commas there.
>
> This patch doesn't modify the compiled arrays, only their representation
> in source form benefits. The former was confirmed with x86 and arm64
> builds.
>
> Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@xxxxxxxxxxxx>

Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>

> ---
> Hello,
>
> the mentioned change to i2c_device_id is the following:
>
> diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h
> index 23ff24080dfd..aebd3a5e90af 100644
> --- a/include/linux/mod_devicetable.h
> +++ b/include/linux/mod_devicetable.h
> @@ -477,7 +477,11 @@ struct rpmsg_device_id {
>
> struct i2c_device_id {
> char name[I2C_NAME_SIZE];
> - kernel_ulong_t driver_data; /* Data private to the driver */
> + union {
> + /* Data private to the driver */
> + kernel_ulong_t driver_data;
> + const void *driver_data_ptr;
> + };
> };
>
> /* pci_epf */
>
> and this requires that .driver_data is assigned via a named initializer
> for static data. This requirement isn't a bad one because named
> initializers are also much better readable than list initializers.
>
> The union added to struct i2c_device_id enables further cleanups like:
>
> diff --git a/drivers/regulator/ad5398.c b/drivers/regulator/ad5398.c
> index 0123ca8157a8..84272ba65d08 100644
> --- a/drivers/regulator/ad5398.c
> +++ b/drivers/regulator/ad5398.c
> @@ -207,8 +207,8 @@ struct ad5398_current_data_format {
> static const struct ad5398_current_data_format df_10_4_120 = {10, 4, 0, 120000};
>
> static const struct i2c_device_id ad5398_id[] = {
> - { .name = "ad5398", .driver_data = (kernel_ulong_t)&df_10_4_120 },
> - { .name = "ad5821", .driver_data = (kernel_ulong_t)&df_10_4_120 },
> + { .name = "ad5398", .driver_data_ptr = &df_10_4_120 },
> + { .name = "ad5821", .driver_data_ptr = &df_10_4_120 },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, ad5398_id);
> @@ -219,8 +219,7 @@ static int ad5398_probe(struct i2c_client *client)
> struct regulator_init_data *init_data = dev_get_platdata(&client->dev);
> struct regulator_config config = { };
> struct ad5398_chip_info *chip;
> - const struct ad5398_current_data_format *df =
> - (struct ad5398_current_data_format *)id->driver_data;
> + const struct ad5398_current_data_format *df = id->driver_data_ptr;
>
> chip = devm_kzalloc(&client->dev, sizeof(*chip), GFP_KERNEL);
> if (!chip)
>
> that are an improvement for readability (again!) and it keeps some
> properties of the pointers (here: being const) without having to pay
> attention for that. (I didn't find a nice driver below drivers/gpu that
> benefits, so this is "only" a regulator driver example.)
>
> My additional motivation for this effort is CHERI[1]. This is a hardware
> extension that uses 128 bit pointers but unsigned long is still 64 bit.
> So with CHERI you cannot store pointers in unsigned long variables.
>
> I was unsure if I should split the patch. I guess doing all of
> drivers/gpu/drm/bridge together is fine, please tell if I should split
> off drivers/gpu/drm/nouveau/ and/or drivers/gpu/drm/sitronix/.
>
> Best regards
> Uwe
>
> [1] https://cheri-alliance.org/discover-cheri/
> https://lwn.net/Articles/1037974/
>
> drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 10 +++++-----
> drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 2 +-
> drivers/gpu/drm/bridge/analogix/anx7625.c | 4 ++--
> drivers/gpu/drm/bridge/chipone-icn6211.c | 4 ++--
> drivers/gpu/drm/bridge/chrontel-ch7033.c | 2 +-
> drivers/gpu/drm/bridge/ite-it6263.c | 2 +-
> drivers/gpu/drm/bridge/ite-it6505.c | 2 +-
> drivers/gpu/drm/bridge/lontium-lt8713sx.c | 2 +-
> drivers/gpu/drm/bridge/lontium-lt8912b.c | 4 ++--
> drivers/gpu/drm/bridge/lontium-lt9211.c | 4 ++--
> drivers/gpu/drm/bridge/lontium-lt9611.c | 4 ++--
> drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 2 +-
> .../gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 8 ++++----
> drivers/gpu/drm/bridge/nxp-ptn3460.c | 4 ++--
> drivers/gpu/drm/bridge/parade-ps8622.c | 8 ++++----
> drivers/gpu/drm/bridge/sii902x.c | 2 +-
> drivers/gpu/drm/bridge/sii9234.c | 2 +-
> drivers/gpu/drm/bridge/sil-sii8620.c | 2 +-
> drivers/gpu/drm/bridge/tc358767.c | 2 +-
> drivers/gpu/drm/bridge/tc358768.c | 4 ++--
> drivers/gpu/drm/bridge/tc358775.c | 4 ++--
> drivers/gpu/drm/bridge/tda998x_drv.c | 2 +-
> drivers/gpu/drm/bridge/ti-dlpc3433.c | 2 +-
> drivers/gpu/drm/bridge/ti-sn65dsi83.c | 6 +++---
> drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++--
> drivers/gpu/drm/bridge/ti-tfp410.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/i2c/ch7006_drv.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/i2c/sil164_drv.c | 2 +-
> drivers/gpu/drm/sitronix/st7571-i2c.c | 4 ++--
> 29 files changed, 51 insertions(+), 51 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> index 6bd76c1fb007..e960ddd96ac4 100644
> --- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
> @@ -1470,11 +1470,11 @@ static const struct adv7511_chip_info adv7535_chip_info = {
> };
>
> static const struct i2c_device_id adv7511_i2c_ids[] = {
> - { "adv7511", (kernel_ulong_t)&adv7511_chip_info },
> - { "adv7511w", (kernel_ulong_t)&adv7511_chip_info },
> - { "adv7513", (kernel_ulong_t)&adv7511_chip_info },
> - { "adv7533", (kernel_ulong_t)&adv7533_chip_info },
> - { "adv7535", (kernel_ulong_t)&adv7535_chip_info },
> + { .name = "adv7511", .driver_data = (kernel_ulong_t)&adv7511_chip_info },
> + { .name = "adv7511w", .driver_data = (kernel_ulong_t)&adv7511_chip_info },
> + { .name = "adv7513", .driver_data = (kernel_ulong_t)&adv7511_chip_info },
> + { .name = "adv7533", .driver_data = (kernel_ulong_t)&adv7533_chip_info },
> + { .name = "adv7535", .driver_data = (kernel_ulong_t)&adv7535_chip_info },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, adv7511_i2c_ids);
> diff --git a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c
> index f3fe47b12edc..f0040b124b8f 100644
> --- a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c
> +++ b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c
> @@ -766,7 +766,7 @@ static void anx6345_i2c_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id anx6345_id[] = {
> - { "anx6345" },
> + { .name = "anx6345" },
> { /* sentinel */ }
> };
> MODULE_DEVICE_TABLE(i2c, anx6345_id);
> diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/bridge/analogix/anx7625.c
> index 54b02242d6db..8b5748af8c55 100644
> --- a/drivers/gpu/drm/bridge/analogix/anx7625.c
> +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c
> @@ -2992,8 +2992,8 @@ static void anx7625_i2c_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id anx7625_id[] = {
> - { "anx7625" },
> - {}
> + { .name = "anx7625" },
> + { }
> };
>
> MODULE_DEVICE_TABLE(i2c, anx7625_id);
> diff --git a/drivers/gpu/drm/bridge/chipone-icn6211.c b/drivers/gpu/drm/bridge/chipone-icn6211.c
> index 814713c5bea9..d389270f5463 100644
> --- a/drivers/gpu/drm/bridge/chipone-icn6211.c
> +++ b/drivers/gpu/drm/bridge/chipone-icn6211.c
> @@ -787,8 +787,8 @@ static struct mipi_dsi_driver chipone_dsi_driver = {
> };
>
> static const struct i2c_device_id chipone_i2c_id[] = {
> - { "chipone,icn6211" },
> - {},
> + { .name = "chipone,icn6211" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, chipone_i2c_id);
>
> diff --git a/drivers/gpu/drm/bridge/chrontel-ch7033.c b/drivers/gpu/drm/bridge/chrontel-ch7033.c
> index 54d49d4882c8..c036337e3064 100644
> --- a/drivers/gpu/drm/bridge/chrontel-ch7033.c
> +++ b/drivers/gpu/drm/bridge/chrontel-ch7033.c
> @@ -598,7 +598,7 @@ static const struct of_device_id ch7033_dt_ids[] = {
> MODULE_DEVICE_TABLE(of, ch7033_dt_ids);
>
> static const struct i2c_device_id ch7033_ids[] = {
> - { "ch7033" },
> + { .name = "ch7033" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, ch7033_ids);
> diff --git a/drivers/gpu/drm/bridge/ite-it6263.c b/drivers/gpu/drm/bridge/ite-it6263.c
> index 4f3ebb7af4d4..49af808e4862 100644
> --- a/drivers/gpu/drm/bridge/ite-it6263.c
> +++ b/drivers/gpu/drm/bridge/ite-it6263.c
> @@ -913,7 +913,7 @@ static const struct of_device_id it6263_of_match[] = {
> MODULE_DEVICE_TABLE(of, it6263_of_match);
>
> static const struct i2c_device_id it6263_i2c_ids[] = {
> - { "it6263" },
> + { .name = "it6263" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, it6263_i2c_ids);
> diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c
> index a094803ba7aa..f6c3d4257e3c 100644
> --- a/drivers/gpu/drm/bridge/ite-it6505.c
> +++ b/drivers/gpu/drm/bridge/ite-it6505.c
> @@ -3652,7 +3652,7 @@ static void it6505_i2c_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id it6505_id[] = {
> - { "it6505" },
> + { .name = "it6505" },
> { }
> };
>
> diff --git a/drivers/gpu/drm/bridge/lontium-lt8713sx.c b/drivers/gpu/drm/bridge/lontium-lt8713sx.c
> index 18fac6a46db4..3b13b28eab6e 100644
> --- a/drivers/gpu/drm/bridge/lontium-lt8713sx.c
> +++ b/drivers/gpu/drm/bridge/lontium-lt8713sx.c
> @@ -570,7 +570,7 @@ static void lt8713sx_remove(struct i2c_client *client)
> }
>
> static struct i2c_device_id lt8713sx_id[] = {
> - { "lontium,lt8713sx", 0 },
> + { .name = "lontium,lt8713sx" },
> { /* sentinel */ }
> };
>
> diff --git a/drivers/gpu/drm/bridge/lontium-lt8912b.c b/drivers/gpu/drm/bridge/lontium-lt8912b.c
> index 8a0b48efca58..d22e4ec8f644 100644
> --- a/drivers/gpu/drm/bridge/lontium-lt8912b.c
> +++ b/drivers/gpu/drm/bridge/lontium-lt8912b.c
> @@ -815,8 +815,8 @@ static const struct of_device_id lt8912_dt_match[] = {
> MODULE_DEVICE_TABLE(of, lt8912_dt_match);
>
> static const struct i2c_device_id lt8912_id[] = {
> - { "lt8912" },
> - {}
> + { .name = "lt8912" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, lt8912_id);
>
> diff --git a/drivers/gpu/drm/bridge/lontium-lt9211.c b/drivers/gpu/drm/bridge/lontium-lt9211.c
> index 03fc8fd10f20..df9ad691d2cf 100644
> --- a/drivers/gpu/drm/bridge/lontium-lt9211.c
> +++ b/drivers/gpu/drm/bridge/lontium-lt9211.c
> @@ -772,8 +772,8 @@ static void lt9211_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id lt9211_id[] = {
> - { "lontium,lt9211" },
> - {},
> + { .name = "lontium,lt9211" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, lt9211_id);
>
> diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c
> index 4517aee83332..ea393207060c 100644
> --- a/drivers/gpu/drm/bridge/lontium-lt9611.c
> +++ b/drivers/gpu/drm/bridge/lontium-lt9611.c
> @@ -1226,8 +1226,8 @@ static void lt9611_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id lt9611_id[] = {
> - { "lontium,lt9611" },
> - {}
> + { .name = "lontium,lt9611" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, lt9611_id);
>
> diff --git a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
> index 11aab07d88df..a08132fe1b50 100644
> --- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
> +++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
> @@ -893,7 +893,7 @@ static void lt9611uxc_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id lt9611uxc_id[] = {
> - { "lontium,lt9611uxc" },
> + { .name = "lontium,lt9611uxc" },
> { /* sentinel */ }
> };
>
> diff --git a/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c b/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c
> index c9e6505cbd88..89d1c0796f19 100644
> --- a/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c
> +++ b/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c
> @@ -310,8 +310,8 @@ static void stdp4028_ge_b850v3_fw_remove(struct i2c_client *stdp4028_i2c)
> }
>
> static const struct i2c_device_id stdp4028_ge_b850v3_fw_i2c_table[] = {
> - { "stdp4028_ge_fw" },
> - {}
> + { .name = "stdp4028_ge_fw" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, stdp4028_ge_b850v3_fw_i2c_table);
>
> @@ -357,8 +357,8 @@ static void stdp2690_ge_b850v3_fw_remove(struct i2c_client *stdp2690_i2c)
> }
>
> static const struct i2c_device_id stdp2690_ge_b850v3_fw_i2c_table[] = {
> - { "stdp2690_ge_fw" },
> - {}
> + { .name = "stdp2690_ge_fw" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, stdp2690_ge_b850v3_fw_i2c_table);
>
> diff --git a/drivers/gpu/drm/bridge/nxp-ptn3460.c b/drivers/gpu/drm/bridge/nxp-ptn3460.c
> index 7acb11f16dc1..ae0becdc388a 100644
> --- a/drivers/gpu/drm/bridge/nxp-ptn3460.c
> +++ b/drivers/gpu/drm/bridge/nxp-ptn3460.c
> @@ -318,8 +318,8 @@ static void ptn3460_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id ptn3460_i2c_table[] = {
> - { "ptn3460" },
> - {}
> + { .name = "ptn3460" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, ptn3460_i2c_table);
>
> diff --git a/drivers/gpu/drm/bridge/parade-ps8622.c b/drivers/gpu/drm/bridge/parade-ps8622.c
> index f879a1df077d..b3fc2b3cf93b 100644
> --- a/drivers/gpu/drm/bridge/parade-ps8622.c
> +++ b/drivers/gpu/drm/bridge/parade-ps8622.c
> @@ -528,10 +528,10 @@ static void ps8622_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id ps8622_i2c_table[] = {
> - /* Device type, max_lane_count */
> - {"ps8622", 1},
> - {"ps8625", 2},
> - {},
> + /* Device type, driver_data holds the maximal lane_count */
> + { .name = "ps8622", .driver_data = 1 },
> + { .name = "ps8625", .driver_data = 2 },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, ps8622_i2c_table);
>
> diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c
> index 12497f5ce4ff..938f245a2fda 100644
> --- a/drivers/gpu/drm/bridge/sii902x.c
> +++ b/drivers/gpu/drm/bridge/sii902x.c
> @@ -1237,7 +1237,7 @@ static const struct of_device_id sii902x_dt_ids[] = {
> MODULE_DEVICE_TABLE(of, sii902x_dt_ids);
>
> static const struct i2c_device_id sii902x_i2c_ids[] = {
> - { "sii9022" },
> + { .name = "sii9022" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, sii902x_i2c_ids);
> diff --git a/drivers/gpu/drm/bridge/sii9234.c b/drivers/gpu/drm/bridge/sii9234.c
> index bb1bed03eb5b..a9dce523a9c6 100644
> --- a/drivers/gpu/drm/bridge/sii9234.c
> +++ b/drivers/gpu/drm/bridge/sii9234.c
> @@ -945,7 +945,7 @@ static const struct of_device_id sii9234_dt_match[] = {
> MODULE_DEVICE_TABLE(of, sii9234_dt_match);
>
> static const struct i2c_device_id sii9234_id[] = {
> - { "SII9234" },
> + { .name = "SII9234" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, sii9234_id);
> diff --git a/drivers/gpu/drm/bridge/sil-sii8620.c b/drivers/gpu/drm/bridge/sil-sii8620.c
> index 982306eb4f0a..6daf745fc608 100644
> --- a/drivers/gpu/drm/bridge/sil-sii8620.c
> +++ b/drivers/gpu/drm/bridge/sil-sii8620.c
> @@ -2370,7 +2370,7 @@ static const struct of_device_id sii8620_dt_match[] = {
> MODULE_DEVICE_TABLE(of, sii8620_dt_match);
>
> static const struct i2c_device_id sii8620_id[] = {
> - { "sii8620" },
> + { .name = "sii8620" },
> { }
> };
>
> diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
> index 4097fef4b86b..1158ae151da0 100644
> --- a/drivers/gpu/drm/bridge/tc358767.c
> +++ b/drivers/gpu/drm/bridge/tc358767.c
> @@ -2611,7 +2611,7 @@ static void tc_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id tc358767_i2c_ids[] = {
> - { "tc358767" },
> + { .name = "tc358767" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, tc358767_i2c_ids);
> diff --git a/drivers/gpu/drm/bridge/tc358768.c b/drivers/gpu/drm/bridge/tc358768.c
> index fbdc44e16229..aef0c7de21e1 100644
> --- a/drivers/gpu/drm/bridge/tc358768.c
> +++ b/drivers/gpu/drm/bridge/tc358768.c
> @@ -1249,8 +1249,8 @@ static const struct regmap_config tc358768_regmap_config = {
> };
>
> static const struct i2c_device_id tc358768_i2c_ids[] = {
> - { "tc358768" },
> - { "tc358778" },
> + { .name = "tc358768" },
> + { .name = "tc358778" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, tc358768_i2c_ids);
> diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c
> index 366b12db0e7c..ea3e9b693c04 100644
> --- a/drivers/gpu/drm/bridge/tc358775.c
> +++ b/drivers/gpu/drm/bridge/tc358775.c
> @@ -727,8 +727,8 @@ static void tc_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id tc358775_i2c_ids[] = {
> - { "tc358765", TC358765, },
> - { "tc358775", TC358775, },
> + { .name = "tc358765", .driver_data = TC358765 },
> + { .name = "tc358775", .driver_data = TC358775 },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, tc358775_i2c_ids);
> diff --git a/drivers/gpu/drm/bridge/tda998x_drv.c b/drivers/gpu/drm/bridge/tda998x_drv.c
> index d9b388165de1..9471c0208fea 100644
> --- a/drivers/gpu/drm/bridge/tda998x_drv.c
> +++ b/drivers/gpu/drm/bridge/tda998x_drv.c
> @@ -2059,7 +2059,7 @@ MODULE_DEVICE_TABLE(of, tda998x_dt_ids);
> #endif
>
> static const struct i2c_device_id tda998x_ids[] = {
> - { "tda998x" },
> + { .name = "tda998x" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, tda998x_ids);
> diff --git a/drivers/gpu/drm/bridge/ti-dlpc3433.c b/drivers/gpu/drm/bridge/ti-dlpc3433.c
> index b07f7c9d5890..e61548d122e4 100644
> --- a/drivers/gpu/drm/bridge/ti-dlpc3433.c
> +++ b/drivers/gpu/drm/bridge/ti-dlpc3433.c
> @@ -389,7 +389,7 @@ static void dlpc3433_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id dlpc3433_id[] = {
> - { "ti,dlpc3433" },
> + { .name = "ti,dlpc3433" },
> { /* sentinel */ }
> };
> MODULE_DEVICE_TABLE(i2c, dlpc3433_id);
> diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi83.c b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> index 17a885244e1e..9eb8ea5c164b 100644
> --- a/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> +++ b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> @@ -1063,9 +1063,9 @@ static void sn65dsi83_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id sn65dsi83_id[] = {
> - { "ti,sn65dsi83", MODEL_SN65DSI83 },
> - { "ti,sn65dsi84", MODEL_SN65DSI84 },
> - {},
> + { .name = "ti,sn65dsi83", .driver_data = MODEL_SN65DSI83 },
> + { .name = "ti,sn65dsi84", .driver_data = MODEL_SN65DSI84 },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, sn65dsi83_id);
>
> diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> index 98d64ad791d0..d64c4c0bcbaf 100644
> --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> @@ -2118,8 +2118,8 @@ static int ti_sn65dsi86_probe(struct i2c_client *client)
> }
>
> static const struct i2c_device_id ti_sn65dsi86_id[] = {
> - { "ti,sn65dsi86" },
> - {}
> + { .name = "ti,sn65dsi86" },
> + { }
> };
> MODULE_DEVICE_TABLE(i2c, ti_sn65dsi86_id);
>
> diff --git a/drivers/gpu/drm/bridge/ti-tfp410.c b/drivers/gpu/drm/bridge/ti-tfp410.c
> index 3b6b0e92cf89..85dc5e77b712 100644
> --- a/drivers/gpu/drm/bridge/ti-tfp410.c
> +++ b/drivers/gpu/drm/bridge/ti-tfp410.c
> @@ -435,7 +435,7 @@ static void tfp410_i2c_remove(struct i2c_client *client)
> }
>
> static const struct i2c_device_id tfp410_i2c_ids[] = {
> - { "tfp410" },
> + { .name = "tfp410" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, tfp410_i2c_ids);
> diff --git a/drivers/gpu/drm/nouveau/dispnv04/i2c/ch7006_drv.c b/drivers/gpu/drm/nouveau/dispnv04/i2c/ch7006_drv.c
> index 976ed59a86e4..96dabdb38c4b 100644
> --- a/drivers/gpu/drm/nouveau/dispnv04/i2c/ch7006_drv.c
> +++ b/drivers/gpu/drm/nouveau/dispnv04/i2c/ch7006_drv.c
> @@ -486,7 +486,7 @@ static int ch7006_encoder_init(struct i2c_client *client,
> }
>
> static const struct i2c_device_id ch7006_ids[] = {
> - { "ch7006" },
> + { .name = "ch7006" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, ch7006_ids);
> diff --git a/drivers/gpu/drm/nouveau/dispnv04/i2c/sil164_drv.c b/drivers/gpu/drm/nouveau/dispnv04/i2c/sil164_drv.c
> index 7f402b901977..2324507eb5f3 100644
> --- a/drivers/gpu/drm/nouveau/dispnv04/i2c/sil164_drv.c
> +++ b/drivers/gpu/drm/nouveau/dispnv04/i2c/sil164_drv.c
> @@ -414,7 +414,7 @@ sil164_encoder_init(struct i2c_client *client,
> }
>
> static const struct i2c_device_id sil164_ids[] = {
> - { "sil164" },
> + { .name = "sil164" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, sil164_ids);
> diff --git a/drivers/gpu/drm/sitronix/st7571-i2c.c b/drivers/gpu/drm/sitronix/st7571-i2c.c
> index 44bc94be33d6..be6fbed8adaa 100644
> --- a/drivers/gpu/drm/sitronix/st7571-i2c.c
> +++ b/drivers/gpu/drm/sitronix/st7571-i2c.c
> @@ -134,8 +134,8 @@ static const struct of_device_id st7571_of_match[] = {
> MODULE_DEVICE_TABLE(of, st7571_of_match);
>
> static const struct i2c_device_id st7571_id[] = {
> - { "st7567", 0 },
> - { "st7571", 0 },
> + { .name = "st7567" },
> + { .name = "st7571" },
> { }
> };
> MODULE_DEVICE_TABLE(i2c, st7571_id);
>
> base-commit: 254f49634ee16a731174d2ae34bc50bd5f45e731
> prerequisite-patch-id: 7779c63f16ef6f7247cdb71c89e66b27e299eb74
> prerequisite-patch-id: 6f920b6f8c31dc0ad1689200c37680755c20ce8b
> prerequisite-patch-id: 1fd68e883664147052540eea19769ea9e92d0138
> prerequisite-patch-id: fff07090df18a39a361bbb091a3f17223b4606b4
> prerequisite-patch-id: a935aab66aa9896437ab9d757ef9fdc859d22495
> prerequisite-patch-id: d0d54f7acecd560cdeb6ea0c0e5ae77a50695d68
> prerequisite-patch-id: d7876560b3bb9b05ac462d0a9b09b50efeb9b5e1
> prerequisite-patch-id: 65531f0504ac1885c6c4ed6be0e6a206a9795d5b
> prerequisite-patch-id: ec83e7e18d66da9ca677b5c8180a22bf3717c8fb
> prerequisite-patch-id: e33193d1a91f5819128db924c080caf1c5198667
> prerequisite-patch-id: c8862be402a445f30b9f5c91b07afdc840e7e21f
> prerequisite-patch-id: 33c001d1b8ecaf57ebe53c321d82d671bc82b647
> prerequisite-patch-id: a59a578f49eb5147623aab9fdcacba405b9c8353
> prerequisite-patch-id: 5eef512b8a5a1ec1848a939928168dbb719ca72e
> prerequisite-patch-id: 04746bfdfe146af71d0c41e225978fb42ce977f5
> prerequisite-patch-id: f8aeb0b768ae718aa0dab188fbad671aa4c76501
> prerequisite-patch-id: 8e0b48a5d5f8bb91933d440eda8f065590e5ca97
> prerequisite-patch-id: 5ca261e980415013434edc099b6f741b7a96c7a8
> prerequisite-patch-id: 912c3a7f7fa847c59d499e004965b805a63c8836
> prerequisite-patch-id: acf784c7c03d3cd14a8a19610a6d8995e2b5da6d
> prerequisite-patch-id: 09a95efff7875781ceed199c60204a2603960f4f
> prerequisite-patch-id: 8051fbf0a60e5e9d34f5e7623a5159d32fbc511d
> prerequisite-patch-id: c4897ca0762d01cd5e8cb6e761e5ad95eb9de918
> prerequisite-patch-id: eb0120090b9fd1e1f668919cf1d81cdf1b684b20
> prerequisite-patch-id: 239c0e25dae78755bb553eb9049190f6ed6c056a
> prerequisite-patch-id: d988f27ca0fac3a2b06cc8e465aef5e0a4f09bd3
> prerequisite-patch-id: 8fceffb98fa11bf1e709a51022afa05eb12236bc
> prerequisite-patch-id: 3575869dc475ff7191d314632d496cc67aabed53
> prerequisite-patch-id: 8916058071bcbde1a4d06c6a8981c443132168f7
> prerequisite-patch-id: 7f29acd91e1c8b8d4f21152cbcb8a8b0bdb6f963
> prerequisite-patch-id: 65cf39c97cc086b6a89965079cccb896f089a0cb
> prerequisite-patch-id: 915b7a2276c5a38944107c84bb282271d0bf28f4
> prerequisite-patch-id: 76d499a624150588f7ef07dc087e70eca6f33419

--
Regards,

Laurent Pinchart