| Message ID | 20241202114731.1005549-2-u.kleine-koenig@baylibre.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] drm: Drop explicit initialization of struct i2c_device_id::driver_data to 0 | expand |
Am 02.12.24 um 12:47 schrieb Uwe Kleine-König: > These drivers don't use the driver_data member of struct i2c_device_id, > so don't explicitly initialize this member. > > This prepares putting driver_data in an anonymous union which requires > either no initialization or named designators. But it's also a nice > cleanup on its own. > > While add it, also remove commas after the sentinel entries. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de> > --- > Hello, > > changes since (implicit) v1 > (https://lore.kernel.org/dri-devel/20240516123837.81664-2-u.kleine-koenig@pengutronix.de): > > - Update author email address > - Rebase (trivially) to latest next > - Also convert drivers/gpu/drm/bridge/ite-it6263.c (which is new since > the series was orginally sent in May) > > Best regards > Uwe > > drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 2 +- > drivers/gpu/drm/bridge/analogix/anx7625.c | 2 +- > 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-lt8912b.c | 4 ++-- > drivers/gpu/drm/bridge/lontium-lt9611.c | 2 +- > drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 2 +- > drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 8 ++++---- > drivers/gpu/drm/bridge/nxp-ptn3460.c | 4 ++-- > drivers/gpu/drm/bridge/sii902x.c | 4 ++-- > drivers/gpu/drm/bridge/sii9234.c | 4 ++-- > drivers/gpu/drm/bridge/sil-sii8620.c | 4 ++-- > drivers/gpu/drm/bridge/tc358767.c | 2 +- > drivers/gpu/drm/bridge/tc358768.c | 4 ++-- > drivers/gpu/drm/bridge/ti-dlpc3433.c | 2 +- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++-- > drivers/gpu/drm/bridge/ti-tfp410.c | 2 +- > drivers/gpu/drm/i2c/ch7006_drv.c | 2 +- > drivers/gpu/drm/i2c/sil164_drv.c | 2 +- > drivers/gpu/drm/i2c/tda9950.c | 4 ++-- > drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- > 22 files changed, 33 insertions(+), 33 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c > index b754947e3e00..83d711ee3a2e 100644 > --- a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c > +++ b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c > @@ -793,7 +793,7 @@ static void anx6345_i2c_remove(struct i2c_client *client) > } > > static const struct i2c_device_id anx6345_id[] = { > - { "anx6345", 0 }, > + { "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 a2675b121fe4..3427322f986a 100644 > --- a/drivers/gpu/drm/bridge/analogix/anx7625.c > +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c > @@ -2795,7 +2795,7 @@ static void anx7625_i2c_remove(struct i2c_client *client) > } > > static const struct i2c_device_id anx7625_id[] = { > - {"anx7625", 0}, > + { "anx7625" }, > {} > }; > > diff --git a/drivers/gpu/drm/bridge/chrontel-ch7033.c b/drivers/gpu/drm/bridge/chrontel-ch7033.c > index c83486cf6b15..da17f0978a79 100644 > --- a/drivers/gpu/drm/bridge/chrontel-ch7033.c > +++ b/drivers/gpu/drm/bridge/chrontel-ch7033.c > @@ -597,7 +597,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", 0 }, > + { "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 cbabd4e20d3e..1e4b20abc335 100644 > --- a/drivers/gpu/drm/bridge/ite-it6263.c > +++ b/drivers/gpu/drm/bridge/ite-it6263.c > @@ -878,7 +878,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", 0 }, > + { "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 008d86cc562a..0faad10ba8e4 100644 > --- a/drivers/gpu/drm/bridge/ite-it6505.c > +++ b/drivers/gpu/drm/bridge/ite-it6505.c > @@ -3497,7 +3497,7 @@ static void it6505_i2c_remove(struct i2c_client *client) > } > > static const struct i2c_device_id it6505_id[] = { > - { "it6505", 0 }, > + { "it6505" }, > { } > }; > > diff --git a/drivers/gpu/drm/bridge/lontium-lt8912b.c b/drivers/gpu/drm/bridge/lontium-lt8912b.c > index e265ab3c8c92..52da204f5740 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", 0}, > - {}, > + { "lt8912" }, > + {} > }; > MODULE_DEVICE_TABLE(i2c, lt8912_id); > > diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c > index 1b31fdebe164..b3f5e69fb39b 100644 > --- a/drivers/gpu/drm/bridge/lontium-lt9611.c > +++ b/drivers/gpu/drm/bridge/lontium-lt9611.c > @@ -1236,7 +1236,7 @@ static void lt9611_remove(struct i2c_client *client) > } > > static struct i2c_device_id lt9611_id[] = { > - { "lontium,lt9611", 0 }, > + { "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 4d1d40e1f1b4..57f66e4f7843 100644 > --- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c > +++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c > @@ -914,7 +914,7 @@ static void lt9611uxc_remove(struct i2c_client *client) > } > > static struct i2c_device_id lt9611uxc_id[] = { > - { "lontium,lt9611uxc", 0 }, > + { "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 37f1acf5c0f8..a3dcee62e7a5 100644 > --- a/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c > +++ b/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c > @@ -318,8 +318,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", 0}, > - {}, > + { "stdp4028_ge_fw" }, > + {} > }; > MODULE_DEVICE_TABLE(i2c, stdp4028_ge_b850v3_fw_i2c_table); > > @@ -365,8 +365,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", 0}, > - {}, > + { "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 e77aab965fcf..44e36ae66db4 100644 > --- a/drivers/gpu/drm/bridge/nxp-ptn3460.c > +++ b/drivers/gpu/drm/bridge/nxp-ptn3460.c > @@ -319,8 +319,8 @@ static void ptn3460_remove(struct i2c_client *client) > } > > static const struct i2c_device_id ptn3460_i2c_table[] = { > - {"ptn3460", 0}, > - {}, > + { "ptn3460" }, > + {} > }; > MODULE_DEVICE_TABLE(i2c, ptn3460_i2c_table); > > diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c > index 9be9cc5b9025..127da22011b3 100644 > --- a/drivers/gpu/drm/bridge/sii902x.c > +++ b/drivers/gpu/drm/bridge/sii902x.c > @@ -1239,8 +1239,8 @@ 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", 0 }, > - { }, > + { "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 0c74cdc07032..cd7837c9a6e0 100644 > --- a/drivers/gpu/drm/bridge/sii9234.c > +++ b/drivers/gpu/drm/bridge/sii9234.c > @@ -945,8 +945,8 @@ 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", 0 }, > - { }, > + { "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 26b8d137bce0..28a2e1ee04b2 100644 > --- a/drivers/gpu/drm/bridge/sil-sii8620.c > +++ b/drivers/gpu/drm/bridge/sil-sii8620.c > @@ -2368,8 +2368,8 @@ 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", 0 }, > - { }, > + { "sii8620" }, > + { } > }; > > MODULE_DEVICE_TABLE(i2c, sii8620_id); > diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c > index 7275e66faefc..4637bf6ea7a3 100644 > --- a/drivers/gpu/drm/bridge/tc358767.c > +++ b/drivers/gpu/drm/bridge/tc358767.c > @@ -2587,7 +2587,7 @@ static void tc_remove(struct i2c_client *client) > } > > static const struct i2c_device_id tc358767_i2c_ids[] = { > - { "tc358767", 0 }, > + { "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 2cb748bbefcd..ec79b0dd0e2c 100644 > --- a/drivers/gpu/drm/bridge/tc358768.c > +++ b/drivers/gpu/drm/bridge/tc358768.c > @@ -1244,8 +1244,8 @@ static const struct regmap_config tc358768_regmap_config = { > }; > > static const struct i2c_device_id tc358768_i2c_ids[] = { > - { "tc358768", 0 }, > - { "tc358778", 0 }, > + { "tc358768" }, > + { "tc358778" }, > { } > }; > MODULE_DEVICE_TABLE(i2c, tc358768_i2c_ids); > diff --git a/drivers/gpu/drm/bridge/ti-dlpc3433.c b/drivers/gpu/drm/bridge/ti-dlpc3433.c > index a0a1b5dd794e..eaec70fa42b6 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", 0 }, > + { "ti,dlpc3433" }, > { /* sentinel */ } > }; > MODULE_DEVICE_TABLE(i2c, dlpc3433_id); > diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c > index 9e31f750fd88..3f31102bef5f 100644 > --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c > +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c > @@ -1971,8 +1971,8 @@ static int ti_sn65dsi86_probe(struct i2c_client *client) > } > > static struct i2c_device_id ti_sn65dsi86_id[] = { > - { "ti,sn65dsi86", 0}, > - {}, > + { "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 b1b1e4d5a24a..7b8492d91df8 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", 0 }, > + { "tfp410" }, > { } > }; > MODULE_DEVICE_TABLE(i2c, tfp410_i2c_ids); > diff --git a/drivers/gpu/drm/i2c/ch7006_drv.c b/drivers/gpu/drm/i2c/ch7006_drv.c > index 131512a5f3bd..fcb0fcd6c897 100644 > --- a/drivers/gpu/drm/i2c/ch7006_drv.c > +++ b/drivers/gpu/drm/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", 0 }, > + { "ch7006" }, > { } > }; > MODULE_DEVICE_TABLE(i2c, ch7006_ids); > diff --git a/drivers/gpu/drm/i2c/sil164_drv.c b/drivers/gpu/drm/i2c/sil164_drv.c > index ff23422727fc..c17afa025d9d 100644 > --- a/drivers/gpu/drm/i2c/sil164_drv.c > +++ b/drivers/gpu/drm/i2c/sil164_drv.c > @@ -413,7 +413,7 @@ sil164_encoder_init(struct i2c_client *client, > } > > static const struct i2c_device_id sil164_ids[] = { > - { "sil164", 0 }, > + { "sil164" }, > { } > }; > MODULE_DEVICE_TABLE(i2c, sil164_ids); > diff --git a/drivers/gpu/drm/i2c/tda9950.c b/drivers/gpu/drm/i2c/tda9950.c > index 82d618c40dce..cbff851e0c85 100644 > --- a/drivers/gpu/drm/i2c/tda9950.c > +++ b/drivers/gpu/drm/i2c/tda9950.c > @@ -486,8 +486,8 @@ static void tda9950_remove(struct i2c_client *client) > } > > static struct i2c_device_id tda9950_ids[] = { > - { "tda9950", 0 }, > - { }, > + { "tda9950" }, > + { } > }; > MODULE_DEVICE_TABLE(i2c, tda9950_ids); > > diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c > index 2160f05bbd16..57ce77c2be24 100644 > --- a/drivers/gpu/drm/i2c/tda998x_drv.c > +++ b/drivers/gpu/drm/i2c/tda998x_drv.c > @@ -2094,7 +2094,7 @@ MODULE_DEVICE_TABLE(of, tda998x_dt_ids); > #endif > > static const struct i2c_device_id tda998x_ids[] = { > - { "tda998x", 0 }, > + { "tda998x" }, > { } > }; > MODULE_DEVICE_TABLE(i2c, tda998x_ids); > > base-commit: f486c8aa16b8172f63bddc70116a0c897a7f3f02
Hi, On Mon, Dec 2, 2024 at 3:47 AM Uwe Kleine-König <u.kleine-koenig@baylibre.com> wrote: > > These drivers don't use the driver_data member of struct i2c_device_id, > so don't explicitly initialize this member. > > This prepares putting driver_data in an anonymous union which requires > either no initialization or named designators. But it's also a nice > cleanup on its own. > > While add it, also remove commas after the sentinel entries. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com> > --- > Hello, > > changes since (implicit) v1 > (https://lore.kernel.org/dri-devel/20240516123837.81664-2-u.kleine-koenig@pengutronix.de): > > - Update author email address > - Rebase (trivially) to latest next > - Also convert drivers/gpu/drm/bridge/ite-it6263.c (which is new since > the series was orginally sent in May) > > Best regards > Uwe > > drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 2 +- > drivers/gpu/drm/bridge/analogix/anx7625.c | 2 +- > 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-lt8912b.c | 4 ++-- > drivers/gpu/drm/bridge/lontium-lt9611.c | 2 +- > drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 2 +- > drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 8 ++++---- > drivers/gpu/drm/bridge/nxp-ptn3460.c | 4 ++-- > drivers/gpu/drm/bridge/sii902x.c | 4 ++-- > drivers/gpu/drm/bridge/sii9234.c | 4 ++-- > drivers/gpu/drm/bridge/sil-sii8620.c | 4 ++-- > drivers/gpu/drm/bridge/tc358767.c | 2 +- > drivers/gpu/drm/bridge/tc358768.c | 4 ++-- > drivers/gpu/drm/bridge/ti-dlpc3433.c | 2 +- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++-- > drivers/gpu/drm/bridge/ti-tfp410.c | 2 +- > drivers/gpu/drm/i2c/ch7006_drv.c | 2 +- > drivers/gpu/drm/i2c/sil164_drv.c | 2 +- > drivers/gpu/drm/i2c/tda9950.c | 4 ++-- > drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- > 22 files changed, 33 insertions(+), 33 deletions(-) This conflicts with commit dbf7986f8a56 ("drm/bridge: Constify struct i2c_device_id"). Can you rebase, please? Other than that: Reviewed-by: Douglas Anderson <dianders@chromium.org>
On 12/02/2024, Uwe Kleine-König wrote: > These drivers don't use the driver_data member of struct i2c_device_id, > so don't explicitly initialize this member. > > This prepares putting driver_data in an anonymous union which requires > either no initialization or named designators. But it's also a nice > cleanup on its own. > > While add it, also remove commas after the sentinel entries. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com> > --- > Hello, > > changes since (implicit) v1 > (https://lore.kernel.org/dri-devel/20240516123837.81664-2-u.kleine-koenig@pengutronix.de): > > - Update author email address > - Rebase (trivially) to latest next > - Also convert drivers/gpu/drm/bridge/ite-it6263.c (which is new since > the series was orginally sent in May) > > Best regards > Uwe > > drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 2 +- > drivers/gpu/drm/bridge/analogix/anx7625.c | 2 +- > drivers/gpu/drm/bridge/chrontel-ch7033.c | 2 +- > drivers/gpu/drm/bridge/ite-it6263.c | 2 +- Reviewed-by: Liu Ying <victor.liu@nxp.com> # ite-it6263.c > drivers/gpu/drm/bridge/ite-it6505.c | 2 +- > drivers/gpu/drm/bridge/lontium-lt8912b.c | 4 ++-- > drivers/gpu/drm/bridge/lontium-lt9611.c | 2 +- > drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 2 +- > drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 8 ++++---- > drivers/gpu/drm/bridge/nxp-ptn3460.c | 4 ++-- > drivers/gpu/drm/bridge/sii902x.c | 4 ++-- > drivers/gpu/drm/bridge/sii9234.c | 4 ++-- > drivers/gpu/drm/bridge/sil-sii8620.c | 4 ++-- > drivers/gpu/drm/bridge/tc358767.c | 2 +- > drivers/gpu/drm/bridge/tc358768.c | 4 ++-- > drivers/gpu/drm/bridge/ti-dlpc3433.c | 2 +- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++-- > drivers/gpu/drm/bridge/ti-tfp410.c | 2 +- > drivers/gpu/drm/i2c/ch7006_drv.c | 2 +- > drivers/gpu/drm/i2c/sil164_drv.c | 2 +- > drivers/gpu/drm/i2c/tda9950.c | 4 ++-- > drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- > 22 files changed, 33 insertions(+), 33 deletions(-)
diff --git a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c index b754947e3e00..83d711ee3a2e 100644 --- a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c +++ b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c @@ -793,7 +793,7 @@ static void anx6345_i2c_remove(struct i2c_client *client) } static const struct i2c_device_id anx6345_id[] = { - { "anx6345", 0 }, + { "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 a2675b121fe4..3427322f986a 100644 --- a/drivers/gpu/drm/bridge/analogix/anx7625.c +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c @@ -2795,7 +2795,7 @@ static void anx7625_i2c_remove(struct i2c_client *client) } static const struct i2c_device_id anx7625_id[] = { - {"anx7625", 0}, + { "anx7625" }, {} }; diff --git a/drivers/gpu/drm/bridge/chrontel-ch7033.c b/drivers/gpu/drm/bridge/chrontel-ch7033.c index c83486cf6b15..da17f0978a79 100644 --- a/drivers/gpu/drm/bridge/chrontel-ch7033.c +++ b/drivers/gpu/drm/bridge/chrontel-ch7033.c @@ -597,7 +597,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", 0 }, + { "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 cbabd4e20d3e..1e4b20abc335 100644 --- a/drivers/gpu/drm/bridge/ite-it6263.c +++ b/drivers/gpu/drm/bridge/ite-it6263.c @@ -878,7 +878,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", 0 }, + { "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 008d86cc562a..0faad10ba8e4 100644 --- a/drivers/gpu/drm/bridge/ite-it6505.c +++ b/drivers/gpu/drm/bridge/ite-it6505.c @@ -3497,7 +3497,7 @@ static void it6505_i2c_remove(struct i2c_client *client) } static const struct i2c_device_id it6505_id[] = { - { "it6505", 0 }, + { "it6505" }, { } }; diff --git a/drivers/gpu/drm/bridge/lontium-lt8912b.c b/drivers/gpu/drm/bridge/lontium-lt8912b.c index e265ab3c8c92..52da204f5740 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", 0}, - {}, + { "lt8912" }, + {} }; MODULE_DEVICE_TABLE(i2c, lt8912_id); diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c index 1b31fdebe164..b3f5e69fb39b 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611.c @@ -1236,7 +1236,7 @@ static void lt9611_remove(struct i2c_client *client) } static struct i2c_device_id lt9611_id[] = { - { "lontium,lt9611", 0 }, + { "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 4d1d40e1f1b4..57f66e4f7843 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c @@ -914,7 +914,7 @@ static void lt9611uxc_remove(struct i2c_client *client) } static struct i2c_device_id lt9611uxc_id[] = { - { "lontium,lt9611uxc", 0 }, + { "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 37f1acf5c0f8..a3dcee62e7a5 100644 --- a/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c +++ b/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c @@ -318,8 +318,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", 0}, - {}, + { "stdp4028_ge_fw" }, + {} }; MODULE_DEVICE_TABLE(i2c, stdp4028_ge_b850v3_fw_i2c_table); @@ -365,8 +365,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", 0}, - {}, + { "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 e77aab965fcf..44e36ae66db4 100644 --- a/drivers/gpu/drm/bridge/nxp-ptn3460.c +++ b/drivers/gpu/drm/bridge/nxp-ptn3460.c @@ -319,8 +319,8 @@ static void ptn3460_remove(struct i2c_client *client) } static const struct i2c_device_id ptn3460_i2c_table[] = { - {"ptn3460", 0}, - {}, + { "ptn3460" }, + {} }; MODULE_DEVICE_TABLE(i2c, ptn3460_i2c_table); diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c index 9be9cc5b9025..127da22011b3 100644 --- a/drivers/gpu/drm/bridge/sii902x.c +++ b/drivers/gpu/drm/bridge/sii902x.c @@ -1239,8 +1239,8 @@ 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", 0 }, - { }, + { "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 0c74cdc07032..cd7837c9a6e0 100644 --- a/drivers/gpu/drm/bridge/sii9234.c +++ b/drivers/gpu/drm/bridge/sii9234.c @@ -945,8 +945,8 @@ 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", 0 }, - { }, + { "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 26b8d137bce0..28a2e1ee04b2 100644 --- a/drivers/gpu/drm/bridge/sil-sii8620.c +++ b/drivers/gpu/drm/bridge/sil-sii8620.c @@ -2368,8 +2368,8 @@ 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", 0 }, - { }, + { "sii8620" }, + { } }; MODULE_DEVICE_TABLE(i2c, sii8620_id); diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c index 7275e66faefc..4637bf6ea7a3 100644 --- a/drivers/gpu/drm/bridge/tc358767.c +++ b/drivers/gpu/drm/bridge/tc358767.c @@ -2587,7 +2587,7 @@ static void tc_remove(struct i2c_client *client) } static const struct i2c_device_id tc358767_i2c_ids[] = { - { "tc358767", 0 }, + { "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 2cb748bbefcd..ec79b0dd0e2c 100644 --- a/drivers/gpu/drm/bridge/tc358768.c +++ b/drivers/gpu/drm/bridge/tc358768.c @@ -1244,8 +1244,8 @@ static const struct regmap_config tc358768_regmap_config = { }; static const struct i2c_device_id tc358768_i2c_ids[] = { - { "tc358768", 0 }, - { "tc358778", 0 }, + { "tc358768" }, + { "tc358778" }, { } }; MODULE_DEVICE_TABLE(i2c, tc358768_i2c_ids); diff --git a/drivers/gpu/drm/bridge/ti-dlpc3433.c b/drivers/gpu/drm/bridge/ti-dlpc3433.c index a0a1b5dd794e..eaec70fa42b6 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", 0 }, + { "ti,dlpc3433" }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(i2c, dlpc3433_id); diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c index 9e31f750fd88..3f31102bef5f 100644 --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c @@ -1971,8 +1971,8 @@ static int ti_sn65dsi86_probe(struct i2c_client *client) } static struct i2c_device_id ti_sn65dsi86_id[] = { - { "ti,sn65dsi86", 0}, - {}, + { "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 b1b1e4d5a24a..7b8492d91df8 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", 0 }, + { "tfp410" }, { } }; MODULE_DEVICE_TABLE(i2c, tfp410_i2c_ids); diff --git a/drivers/gpu/drm/i2c/ch7006_drv.c b/drivers/gpu/drm/i2c/ch7006_drv.c index 131512a5f3bd..fcb0fcd6c897 100644 --- a/drivers/gpu/drm/i2c/ch7006_drv.c +++ b/drivers/gpu/drm/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", 0 }, + { "ch7006" }, { } }; MODULE_DEVICE_TABLE(i2c, ch7006_ids); diff --git a/drivers/gpu/drm/i2c/sil164_drv.c b/drivers/gpu/drm/i2c/sil164_drv.c index ff23422727fc..c17afa025d9d 100644 --- a/drivers/gpu/drm/i2c/sil164_drv.c +++ b/drivers/gpu/drm/i2c/sil164_drv.c @@ -413,7 +413,7 @@ sil164_encoder_init(struct i2c_client *client, } static const struct i2c_device_id sil164_ids[] = { - { "sil164", 0 }, + { "sil164" }, { } }; MODULE_DEVICE_TABLE(i2c, sil164_ids); diff --git a/drivers/gpu/drm/i2c/tda9950.c b/drivers/gpu/drm/i2c/tda9950.c index 82d618c40dce..cbff851e0c85 100644 --- a/drivers/gpu/drm/i2c/tda9950.c +++ b/drivers/gpu/drm/i2c/tda9950.c @@ -486,8 +486,8 @@ static void tda9950_remove(struct i2c_client *client) } static struct i2c_device_id tda9950_ids[] = { - { "tda9950", 0 }, - { }, + { "tda9950" }, + { } }; MODULE_DEVICE_TABLE(i2c, tda9950_ids); diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index 2160f05bbd16..57ce77c2be24 100644 --- a/drivers/gpu/drm/i2c/tda998x_drv.c +++ b/drivers/gpu/drm/i2c/tda998x_drv.c @@ -2094,7 +2094,7 @@ MODULE_DEVICE_TABLE(of, tda998x_dt_ids); #endif static const struct i2c_device_id tda998x_ids[] = { - { "tda998x", 0 }, + { "tda998x" }, { } }; MODULE_DEVICE_TABLE(i2c, tda998x_ids);
These drivers don't use the driver_data member of struct i2c_device_id, so don't explicitly initialize this member. This prepares putting driver_data in an anonymous union which requires either no initialization or named designators. But it's also a nice cleanup on its own. While add it, also remove commas after the sentinel entries. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com> --- Hello, changes since (implicit) v1 (https://lore.kernel.org/dri-devel/20240516123837.81664-2-u.kleine-koenig@pengutronix.de): - Update author email address - Rebase (trivially) to latest next - Also convert drivers/gpu/drm/bridge/ite-it6263.c (which is new since the series was orginally sent in May) Best regards Uwe drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 2 +- drivers/gpu/drm/bridge/analogix/anx7625.c | 2 +- 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-lt8912b.c | 4 ++-- drivers/gpu/drm/bridge/lontium-lt9611.c | 2 +- drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 2 +- drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 8 ++++---- drivers/gpu/drm/bridge/nxp-ptn3460.c | 4 ++-- drivers/gpu/drm/bridge/sii902x.c | 4 ++-- drivers/gpu/drm/bridge/sii9234.c | 4 ++-- drivers/gpu/drm/bridge/sil-sii8620.c | 4 ++-- drivers/gpu/drm/bridge/tc358767.c | 2 +- drivers/gpu/drm/bridge/tc358768.c | 4 ++-- drivers/gpu/drm/bridge/ti-dlpc3433.c | 2 +- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++-- drivers/gpu/drm/bridge/ti-tfp410.c | 2 +- drivers/gpu/drm/i2c/ch7006_drv.c | 2 +- drivers/gpu/drm/i2c/sil164_drv.c | 2 +- drivers/gpu/drm/i2c/tda9950.c | 4 ++-- drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- 22 files changed, 33 insertions(+), 33 deletions(-) base-commit: f486c8aa16b8172f63bddc70116a0c897a7f3f02