[PATCH net-next 00/33] Remove comma after sentinel entries in match table

[PATCH net-next 00/33] Remove comma after sentinel entries in match table

[Siddharth Vadapalli]

Hello,

This series removes the comma present after sentinel entries in the match
tables across all networking subsystem drivers, with the exception of the
davinci_mdio.c driver, for which, the patch posted by Geert at:
https://lore.kernel.org/r/e481b6de46cecc6ea604ae165272c667274cc621.1788437140.git.geert+renesas@glider.be/
does the same.

Since I inquired about a cleanup across the entire networking subsystem but
didn't get a response:
https://lore.kernel.org/r/b3d19098-eccd-4619-b667-ad3b7e0a4422@ti.com/
I decided to post this series. Given that Andrew Lunn acknowledged the
cleanup of the davinci_mdio driver:
https://lore.kernel.org/r/abec6831-d344-489d-b70c-1e5dd4d43c2d@lunn.ch/
and I don't see why the cleanup is specific to davinci_mdio driver alone,
I am hoping that this series is treated identically to the patch that
Geert had posted for the davinci_mdio driver.

Series is based on commit
6ebcf5074cff net: openvswitch: don't schedule rebalancing if there are no datapaths
of the 'main' branch of the net-next tree.

Regards,
Siddharth.

Siddharth Vadapalli (33):
  can: bxcan: Remove comma after sentinel-entry in match-table
  can: c_can: Remove comma after sentinel-entry in match-table
  can: flexcan: Remove comma after sentinel-entry in match-table
  can: ifi_canfd: Remove comma after sentinel-entry in match-table
  can: m_can: Remove comma after sentinel-entry in match-table
  can: sja1000: f81601: Remove comma after sentinel-entry in match-table
  can: sja1000_platform: Remove comma after sentinel-entry in
    match-table
  net: dsa: b53: mdio: Remove comma after sentinel-entry in match-table
  net: dsa: b53: srab: Remove comma after sentinel-entry in match-table
  net: dsa: bcm_sf2: Remove comma after sentinel-entry in match-table
  net: dsa: hellcreek: Remove comma after sentinel-entry in match-table
  net: dsa: lan9303: i2c: Remove comma after sentinel-entry in
    match-table
  net: dsa: lan9303: mdio: Remove comma after sentinel-entry in
    match-table
  net: dsa: mxl-gsw1xx: Remove comma after sentinel-entry in match-table
  net: dsa: mt7530: mdio: Remove comma after sentinel-entry in
    match-table
  net: dsa: mt7530: mmio: Remove comma after sentinel-entry in
    match-table
  net: dsa: mv88e6060: Remove comma after sentinel-entry in match-table
  net: dsa: mv88e6xxx: Remove comma after sentinel-entry in match-table
  net: dsa: qca8k: Remove comma after sentinel-entry in match-table
  net: dsa: realtek: rtl8365mb: Remove comma after sentinel-entry in
    match-table
  net: dsa: realtek: rtl8366rb: Remove comma after sentinel-entry in
    match-table
  net: dsa: rzn1_a5psw: Remove comma after sentinel-entry in match-table
  net: dsa: sja1105: Remove comma after sentinel-entry in match-table
  net: bcmasp: Remove comma after sentinel-entry in match-table
  net: spacemit: k1_emac: Remove comma after sentinel-entry in
    match-table
  net: ti: am65-cpsw-nuss: Remove comma after sentinel-entry in
    match-table
  net: ethernet: ti: cpsw: Remove comma after sentinel-entry in
    match-table
  net: ethernet: ti: cpsw_new: Remove comma after sentinel-entry in
    match-table
  net: mdio: bcm-iproc: Remove comma after sentinel-entry in match-table
  net: mdio: bcm-unimac: Remove comma after sentinel-entry in
    match-table
  net: pcs: xpcs-plat: Remove comma after sentinel-entry in match-table
  net: phy: nxp-c45-tja11xx: Remove comma after sentinel-entry in
    match-table
  net: wan: slic_ds26522: Remove comma after sentinel-entry in
    match-table

 drivers/net/can/bxcan.c                     | 2 +-
 drivers/net/can/c_can/c_can_platform.c      | 4 ++--
 drivers/net/can/flexcan/flexcan-core.c      | 2 +-
 drivers/net/can/ifi_canfd/ifi_canfd.c       | 2 +-
 drivers/net/can/m_can/m_can_platform.c      | 2 +-
 drivers/net/can/sja1000/f81601.c            | 2 +-
 drivers/net/can/sja1000/sja1000_platform.c  | 2 +-
 drivers/net/dsa/b53/b53_mdio.c              | 2 +-
 drivers/net/dsa/b53/b53_srab.c              | 2 +-
 drivers/net/dsa/bcm_sf2.c                   | 2 +-
 drivers/net/dsa/hirschmann/hellcreek.c      | 2 +-
 drivers/net/dsa/lan9303_i2c.c               | 2 +-
 drivers/net/dsa/lan9303_mdio.c              | 2 +-
 drivers/net/dsa/lantiq/mxl-gsw1xx.c         | 2 +-
 drivers/net/dsa/mt7530-mdio.c               | 2 +-
 drivers/net/dsa/mt7530-mmio.c               | 2 +-
 drivers/net/dsa/mv88e6060.c                 | 2 +-
 drivers/net/dsa/mv88e6xxx/chip.c            | 2 +-
 drivers/net/dsa/qca/qca8k-8xxx.c            | 2 +-
 drivers/net/dsa/realtek/rtl8365mb_main.c    | 2 +-
 drivers/net/dsa/realtek/rtl8366rb.c         | 2 +-
 drivers/net/dsa/rzn1_a5psw.c                | 2 +-
 drivers/net/dsa/sja1105/sja1105_main.c      | 2 +-
 drivers/net/ethernet/broadcom/asp2/bcmasp.c | 4 ++--
 drivers/net/ethernet/spacemit/k1_emac.c     | 4 ++--
 drivers/net/ethernet/ti/am65-cpsw-nuss.c    | 2 +-
 drivers/net/ethernet/ti/cpsw.c              | 2 +-
 drivers/net/ethernet/ti/cpsw_new.c          | 2 +-
 drivers/net/mdio/mdio-bcm-iproc.c           | 2 +-
 drivers/net/mdio/mdio-bcm-unimac.c          | 2 +-
 drivers/net/pcs/pcs-xpcs-plat.c             | 2 +-
 drivers/net/phy/nxp-c45-tja11xx.c           | 2 +-
 drivers/net/wan/slic_ds26522.c              | 2 +-
 33 files changed, 36 insertions(+), 36 deletions(-)

-- 
2.51.1

[PATCH net-next 01/33] can: bxcan: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/bxcan.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/bxcan.c b/drivers/net/can/bxcan.c
index baf494d20bef..4bddd015775b 100644
--- a/drivers/net/can/bxcan.c
+++ b/drivers/net/can/bxcan.c
@@ -1080,7 +1080,7 @@ static SIMPLE_DEV_PM_OPS(bxcan_pm_ops, bxcan_suspend, bxcan_resume);
 
 static const struct of_device_id bxcan_of_match[] = {
 	{.compatible = "st,stm32f4-bxcan"},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, bxcan_of_match);
 
-- 
2.51.1

[PATCH net-next 02/33] can: c_can: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/c_can/c_can_platform.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/can/c_can/c_can_platform.c b/drivers/net/can/c_can/c_can_platform.c
index 19c86b94a40e..75607b1afd2e 100644
--- a/drivers/net/can/c_can/c_can_platform.c
+++ b/drivers/net/can/c_can/c_can_platform.c
@@ -239,7 +239,7 @@ static const struct platform_device_id c_can_id_table[] = {
 		.name = "d_can",
 		.driver_data = (kernel_ulong_t)&d_can_drvdata,
 	},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(platform, c_can_id_table);
 
@@ -249,7 +249,7 @@ static const struct of_device_id c_can_of_table[] = {
 	{ .compatible = "ti,dra7-d_can", .data = &dra7_dcan_drvdata },
 	{ .compatible = "ti,am3352-d_can", .data = &am3352_dcan_drvdata },
 	{ .compatible = "ti,am4372-d_can", .data = &am3352_dcan_drvdata },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, c_can_of_table);
 
-- 
2.51.1

[PATCH net-next 03/33] can: flexcan: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/flexcan/flexcan-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/flexcan/flexcan-core.c b/drivers/net/can/flexcan/flexcan-core.c
index f5d22c61503f..86b40abe9e76 100644
--- a/drivers/net/can/flexcan/flexcan-core.c
+++ b/drivers/net/can/flexcan/flexcan-core.c
@@ -2070,7 +2070,7 @@ static const struct of_device_id flexcan_of_match[] = {
 	{ .compatible = "fsl,ls1021ar2-flexcan", .data = &fsl_ls1021a_r2_devtype_data, },
 	{ .compatible = "fsl,lx2160ar1-flexcan", .data = &fsl_lx2160a_r1_devtype_data, },
 	{ .compatible = "nxp,s32g2-flexcan", .data = &nxp_s32g2_devtype_data, },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, flexcan_of_match);
 
-- 
2.51.1

[PATCH net-next 04/33] can: ifi_canfd: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/ifi_canfd/ifi_canfd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/ifi_canfd/ifi_canfd.c b/drivers/net/can/ifi_canfd/ifi_canfd.c
index 0f83335e4d07..9eecfe5a87bd 100644
--- a/drivers/net/can/ifi_canfd/ifi_canfd.c
+++ b/drivers/net/can/ifi_canfd/ifi_canfd.c
@@ -1044,7 +1044,7 @@ static void ifi_canfd_plat_remove(struct platform_device *pdev)
 
 static const struct of_device_id ifi_canfd_of_table[] = {
 	{ .compatible = "ifi,canfd-1.0", .data = NULL },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, ifi_canfd_of_table);
 
-- 
2.51.1

[PATCH net-next 05/33] can: m_can: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/m_can/m_can_platform.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/m_can/m_can_platform.c b/drivers/net/can/m_can/m_can_platform.c
index 56da411878af..51dd41dbf2c9 100644
--- a/drivers/net/can/m_can/m_can_platform.c
+++ b/drivers/net/can/m_can/m_can_platform.c
@@ -220,7 +220,7 @@ static const struct dev_pm_ops m_can_pmops = {
 
 static const struct of_device_id m_can_of_table[] = {
 	{ .compatible = "bosch,m_can", .data = NULL },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, m_can_of_table);
 
-- 
2.51.1

Re: [PATCH net-next 05/33] can: m_can: Remove comma after sentinel-entry in match-table

[Markus Schneider-Pargmann]

[-- Attachment #1: Type: text/plain, Size: 500 bytes --]

On Fri Sep 4, 2026 at 6:36 AM CEST, Siddharth Vadapalli wrote:
> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Acked-by: Markus Schneider-Pargmann <msp@baylibre.com>

+Kendall

Note that this and another series by Kendall will probably have a
conflict as both change the compatibles:
https://lore.kernel.org/all/20260902-temp-v4-0-3896f7d16ce8@ti.com/

Best
Markus

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 289 bytes --]

[PATCH net-next 06/33] can: sja1000: f81601: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/sja1000/f81601.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/sja1000/f81601.c b/drivers/net/can/sja1000/f81601.c
index 8f25e95814ef..18cf170db457 100644
--- a/drivers/net/can/sja1000/f81601.c
+++ b/drivers/net/can/sja1000/f81601.c
@@ -40,7 +40,7 @@ struct f81601_pci_card {
 
 static const struct pci_device_id f81601_pci_tbl[] = {
 	{ PCI_DEVICE(0x1c29, 0x1703) },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 
 MODULE_DEVICE_TABLE(pci, f81601_pci_tbl);
-- 
2.51.1

[PATCH net-next 07/33] can: sja1000_platform: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/can/sja1000/sja1000_platform.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/sja1000/sja1000_platform.c b/drivers/net/can/sja1000/sja1000_platform.c
index 2d555f854008..da4e1c820b6e 100644
--- a/drivers/net/can/sja1000/sja1000_platform.c
+++ b/drivers/net/can/sja1000/sja1000_platform.c
@@ -207,7 +207,7 @@ static const struct of_device_id sp_of_table[] = {
 	{ .compatible = "nxp,sja1000", .data = NULL, },
 	{ .compatible = "renesas,rzn1-sja1000", .data = &renesas_data, },
 	{ .compatible = "technologic,sja1000", .data = &technologic_data, },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, sp_of_table);
 
-- 
2.51.1

[PATCH net-next 08/33] net: dsa: b53: mdio: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/b53/b53_mdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/b53/b53_mdio.c b/drivers/net/dsa/b53/b53_mdio.c
index 43a3b37b731b..fc1ce57a80a3 100644
--- a/drivers/net/dsa/b53/b53_mdio.c
+++ b/drivers/net/dsa/b53/b53_mdio.c
@@ -384,7 +384,7 @@ static const struct of_device_id b53_of_match[] = {
 	{ .compatible = "brcm,bcm5395" },
 	{ .compatible = "brcm,bcm5397" },
 	{ .compatible = "brcm,bcm5398" },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, b53_of_match);
 
-- 
2.51.1

[PATCH net-next 09/33] net: dsa: b53: srab: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/b53/b53_srab.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/b53/b53_srab.c b/drivers/net/dsa/b53/b53_srab.c
index b9939bbd2cd5..83a03fca142a 100644
--- a/drivers/net/dsa/b53/b53_srab.c
+++ b/drivers/net/dsa/b53/b53_srab.c
@@ -515,7 +515,7 @@ static const struct of_device_id b53_srab_of_match[] = {
 	{ .compatible = "brcm,cygnus-srab", .data = (void *)BCM583XX_DEVICE_ID },
 	{ .compatible = "brcm,nsp-srab", .data = (void *)BCM58XX_DEVICE_ID },
 	{ .compatible = "brcm,omega-srab", .data = (void *)BCM583XX_DEVICE_ID },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, b53_srab_of_match);
 
-- 
2.51.1

[PATCH net-next 10/33] net: dsa: bcm_sf2: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/bcm_sf2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/bcm_sf2.c b/drivers/net/dsa/bcm_sf2.c
index de3efa3ce9a7..9e5713015187 100644
--- a/drivers/net/dsa/bcm_sf2.c
+++ b/drivers/net/dsa/bcm_sf2.c
@@ -1358,7 +1358,7 @@ static const struct of_device_id bcm_sf2_of_match[] = {
 	{ .compatible = "brcm,bcm7278-switch-v4.8",
 	  .data = &bcm_sf2_7278_data
 	},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, bcm_sf2_of_match);
 
-- 
2.51.1

[PATCH net-next 11/33] net: dsa: hellcreek: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/hirschmann/hellcreek.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/hirschmann/hellcreek.c b/drivers/net/dsa/hirschmann/hellcreek.c
index 2c6cabf61143..aec50414986e 100644
--- a/drivers/net/dsa/hirschmann/hellcreek.c
+++ b/drivers/net/dsa/hirschmann/hellcreek.c
@@ -2093,7 +2093,7 @@ static const struct of_device_id hellcreek_of_match[] = {
 		.compatible = "hirschmann,hellcreek-de1soc-r1",
 		.data	    = &de1soc_r1_pdata,
 	},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, hellcreek_of_match);
 
-- 
2.51.1

Re: [PATCH net-next 11/33] net: dsa: hellcreek: Remove comma after sentinel-entry in match-table

[Kurt Kanzenbach]

[-- Attachment #1: Type: text/plain, Size: 880 bytes --]

On Fri Sep 04 2026, Siddharth Vadapalli wrote:
> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Reviewed-by: Kurt Kanzenbach <kurt@linutronix.de>

> ---
>  drivers/net/dsa/hirschmann/hellcreek.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/dsa/hirschmann/hellcreek.c b/drivers/net/dsa/hirschmann/hellcreek.c
> index 2c6cabf61143..aec50414986e 100644
> --- a/drivers/net/dsa/hirschmann/hellcreek.c
> +++ b/drivers/net/dsa/hirschmann/hellcreek.c
> @@ -2093,7 +2093,7 @@ static const struct of_device_id hellcreek_of_match[] = {
>  		.compatible = "hirschmann,hellcreek-de1soc-r1",
>  		.data	    = &de1soc_r1_pdata,
>  	},
> -	{ /* sentinel */ },
> +	{ /* sentinel */ }
>  };
>  MODULE_DEVICE_TABLE(of, hellcreek_of_match);

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 861 bytes --]

[PATCH net-next 12/33] net: dsa: lan9303: i2c: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/lan9303_i2c.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/lan9303_i2c.c b/drivers/net/dsa/lan9303_i2c.c
index 3a09f3907f44..7b70e22371fc 100644
--- a/drivers/net/dsa/lan9303_i2c.c
+++ b/drivers/net/dsa/lan9303_i2c.c
@@ -96,7 +96,7 @@ MODULE_DEVICE_TABLE(i2c, lan9303_i2c_id);
 
 static const struct of_device_id lan9303_i2c_of_match[] = {
 	{ .compatible = "smsc,lan9303-i2c", },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, lan9303_i2c_of_match);
 
-- 
2.51.1

[PATCH net-next 13/33] net: dsa: lan9303: mdio: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/lan9303_mdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/lan9303_mdio.c b/drivers/net/dsa/lan9303_mdio.c
index 0ac4857e5ee8..d3b5062905b1 100644
--- a/drivers/net/dsa/lan9303_mdio.c
+++ b/drivers/net/dsa/lan9303_mdio.c
@@ -157,7 +157,7 @@ static void lan9303_mdio_shutdown(struct mdio_device *mdiodev)
 static const struct of_device_id lan9303_mdio_of_match[] = {
 	{ .compatible = "smsc,lan9303-mdio" },
 	{ .compatible = "microchip,lan9354-mdio" },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, lan9303_mdio_of_match);
 
-- 
2.51.1

[PATCH net-next 14/33] net: dsa: mxl-gsw1xx: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/lantiq/mxl-gsw1xx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/lantiq/mxl-gsw1xx.c b/drivers/net/dsa/lantiq/mxl-gsw1xx.c
index a1104b2f92a9..347e16e9fdc1 100644
--- a/drivers/net/dsa/lantiq/mxl-gsw1xx.c
+++ b/drivers/net/dsa/lantiq/mxl-gsw1xx.c
@@ -918,7 +918,7 @@ static const struct of_device_id gsw1xx_of_match[] = {
 	{ .compatible = "maxlinear,gsw140", .data = &gsw140_data },
 	{ .compatible = "maxlinear,gsw141", .data = &gsw141_data },
 	{ .compatible = "maxlinear,gsw145", .data = &gsw140_data },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 
 MODULE_DEVICE_TABLE(of, gsw1xx_of_match);
-- 
2.51.1

Re: [PATCH net-next 14/33] net: dsa: mxl-gsw1xx: Remove comma after sentinel-entry in match-table

[Daniel Golle]

On Fri, Sep 04, 2026 at 10:06:20AM +0530, Siddharth Vadapalli wrote:
> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
> 
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
>  drivers/net/dsa/lantiq/mxl-gsw1xx.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/dsa/lantiq/mxl-gsw1xx.c b/drivers/net/dsa/lantiq/mxl-gsw1xx.c
> index a1104b2f92a9..347e16e9fdc1 100644
> --- a/drivers/net/dsa/lantiq/mxl-gsw1xx.c
> +++ b/drivers/net/dsa/lantiq/mxl-gsw1xx.c
> @@ -918,7 +918,7 @@ static const struct of_device_id gsw1xx_of_match[] = {
>  	{ .compatible = "maxlinear,gsw140", .data = &gsw140_data },
>  	{ .compatible = "maxlinear,gsw141", .data = &gsw141_data },
>  	{ .compatible = "maxlinear,gsw145", .data = &gsw140_data },
> -	{ /* sentinel */ },
> +	{ /* sentinel */ }
>  };

Reviewed-by: Daniel Golle <daniel@makrotopia.org>

[PATCH net-next 15/33] net: dsa: mt7530: mdio: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/mt7530-mdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/mt7530-mdio.c b/drivers/net/dsa/mt7530-mdio.c
index 81c5c6f7066d..24ef54f3db2a 100644
--- a/drivers/net/dsa/mt7530-mdio.c
+++ b/drivers/net/dsa/mt7530-mdio.c
@@ -142,7 +142,7 @@ static const struct of_device_id mt7530_of_match[] = {
 	{ .compatible = "mediatek,mt7621", .data = &mt753x_table[ID_MT7621], },
 	{ .compatible = "mediatek,mt7530", .data = &mt753x_table[ID_MT7530], },
 	{ .compatible = "mediatek,mt7531", .data = &mt753x_table[ID_MT7531], },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, mt7530_of_match);
 
-- 
2.51.1

[PATCH net-next 16/33] net: dsa: mt7530: mmio: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/mt7530-mmio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/mt7530-mmio.c b/drivers/net/dsa/mt7530-mmio.c
index fd68b1cd0630..cf3bdb23a2d5 100644
--- a/drivers/net/dsa/mt7530-mmio.c
+++ b/drivers/net/dsa/mt7530-mmio.c
@@ -14,7 +14,7 @@ static const struct of_device_id mt7988_of_match[] = {
 	{ .compatible = "airoha,en7581-switch", .data = &mt753x_table[ID_EN7581], },
 	{ .compatible = "econet,en7528-switch", .data = &mt753x_table[ID_EN7528], },
 	{ .compatible = "mediatek,mt7988-switch", .data = &mt753x_table[ID_MT7988], },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, mt7988_of_match);
 
-- 
2.51.1

Re: [PATCH net-next 16/33] net: dsa: mt7530: mmio: Remove comma after sentinel-entry in match-table

[Daniel Golle]

On Fri, Sep 04, 2026 at 10:06:22AM +0530, Siddharth Vadapalli wrote:
> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
> 
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
>  drivers/net/dsa/mt7530-mmio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/dsa/mt7530-mmio.c b/drivers/net/dsa/mt7530-mmio.c
> index fd68b1cd0630..cf3bdb23a2d5 100644
> --- a/drivers/net/dsa/mt7530-mmio.c
> +++ b/drivers/net/dsa/mt7530-mmio.c
> @@ -14,7 +14,7 @@ static const struct of_device_id mt7988_of_match[] = {
>  	{ .compatible = "airoha,en7581-switch", .data = &mt753x_table[ID_EN7581], },
>  	{ .compatible = "econet,en7528-switch", .data = &mt753x_table[ID_EN7528], },
>  	{ .compatible = "mediatek,mt7988-switch", .data = &mt753x_table[ID_MT7988], },
> -	{ /* sentinel */ },
> +	{ /* sentinel */ }
>  };
>  MODULE_DEVICE_TABLE(of, mt7988_of_match);

Reviewed-by: Daniel Golle <daniel@makrotopia.org>

[PATCH net-next 17/33] net: dsa: mv88e6060: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/mv88e6060.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/mv88e6060.c b/drivers/net/dsa/mv88e6060.c
index 9c8ac14cd4f5..64bf3d759d17 100644
--- a/drivers/net/dsa/mv88e6060.c
+++ b/drivers/net/dsa/mv88e6060.c
@@ -362,7 +362,7 @@ static const struct of_device_id mv88e6060_of_match[] = {
 	{
 		.compatible = "marvell,mv88e6060",
 	},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 
 static struct mdio_driver mv88e6060_driver = {
-- 
2.51.1

[PATCH net-next 18/33] net: dsa: mv88e6xxx: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/mv88e6xxx/chip.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.c
index 80b877c74513..f8aa93e9ee18 100644
--- a/drivers/net/dsa/mv88e6xxx/chip.c
+++ b/drivers/net/dsa/mv88e6xxx/chip.c
@@ -7513,7 +7513,7 @@ static const struct of_device_id mv88e6xxx_of_match[] = {
 		.compatible = "marvell,mv88e6250",
 		.data = &mv88e6xxx_table[MV88E6250],
 	},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 
 MODULE_DEVICE_TABLE(of, mv88e6xxx_of_match);
-- 
2.51.1

[PATCH net-next 19/33] net: dsa: qca8k: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/qca/qca8k-8xxx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/qca/qca8k-8xxx.c b/drivers/net/dsa/qca/qca8k-8xxx.c
index 4c928983b862..60f2a615a5cc 100644
--- a/drivers/net/dsa/qca/qca8k-8xxx.c
+++ b/drivers/net/dsa/qca/qca8k-8xxx.c
@@ -2215,7 +2215,7 @@ static const struct of_device_id qca8k_of_match[] = {
 	{ .compatible = "qca,qca8328", .data = &qca8328 },
 	{ .compatible = "qca,qca8334", .data = &qca833x },
 	{ .compatible = "qca,qca8337", .data = &qca833x },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 
 static struct mdio_driver qca8kmdio_driver = {
-- 
2.51.1

[PATCH net-next 20/33] net: dsa: realtek: rtl8365mb: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/realtek/rtl8365mb_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/realtek/rtl8365mb_main.c b/drivers/net/dsa/realtek/rtl8365mb_main.c
index 4c305756116c..f8220d832e46 100644
--- a/drivers/net/dsa/realtek/rtl8365mb_main.c
+++ b/drivers/net/dsa/realtek/rtl8365mb_main.c
@@ -3352,7 +3352,7 @@ const struct realtek_variant rtl8365mb_variant = {
 
 static const struct of_device_id rtl8365mb_of_match[] = {
 	{ .compatible = "realtek,rtl8365mb", .data = &rtl8365mb_variant, },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, rtl8365mb_of_match);
 
-- 
2.51.1

Re: [PATCH net-next 20/33] net: dsa: realtek: rtl8365mb: Remove comma after sentinel-entry in match-table

[Luiz Angelo Daros de Luca]

Em sex., 4 de set. de 2026 às 01:39, Siddharth Vadapalli
<s-vadapalli@ti.com> escreveu:
>
> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
>  drivers/net/dsa/realtek/rtl8365mb_main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/dsa/realtek/rtl8365mb_main.c b/drivers/net/dsa/realtek/rtl8365mb_main.c
> index 4c305756116c..f8220d832e46 100644
> --- a/drivers/net/dsa/realtek/rtl8365mb_main.c
> +++ b/drivers/net/dsa/realtek/rtl8365mb_main.c
> @@ -3352,7 +3352,7 @@ const struct realtek_variant rtl8365mb_variant = {
>
>  static const struct of_device_id rtl8365mb_of_match[] = {
>         { .compatible = "realtek,rtl8365mb", .data = &rtl8365mb_variant, },
> -       { /* sentinel */ },
> +       { /* sentinel */ }
>  };
>  MODULE_DEVICE_TABLE(of, rtl8365mb_of_match);
>
> --
> 2.51.1
>

Reviewed-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>

[PATCH net-next 21/33] net: dsa: realtek: rtl8366rb: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/realtek/rtl8366rb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/realtek/rtl8366rb.c b/drivers/net/dsa/realtek/rtl8366rb.c
index f11831b66de8..093a06c599d4 100644
--- a/drivers/net/dsa/realtek/rtl8366rb.c
+++ b/drivers/net/dsa/realtek/rtl8366rb.c
@@ -1839,7 +1839,7 @@ const struct realtek_variant rtl8366rb_variant = {
 
 static const struct of_device_id rtl8366rb_of_match[] = {
 	{ .compatible = "realtek,rtl8366rb", .data = &rtl8366rb_variant, },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, rtl8366rb_of_match);
 
-- 
2.51.1

[PATCH net-next 22/33] net: dsa: rzn1_a5psw: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/rzn1_a5psw.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/rzn1_a5psw.c b/drivers/net/dsa/rzn1_a5psw.c
index 4d857e3be10b..85ff1fd2bbfb 100644
--- a/drivers/net/dsa/rzn1_a5psw.c
+++ b/drivers/net/dsa/rzn1_a5psw.c
@@ -1299,7 +1299,7 @@ static void a5psw_shutdown(struct platform_device *pdev)
 
 static const struct of_device_id a5psw_of_mtable[] = {
 	{ .compatible = "renesas,rzn1-a5psw", },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, a5psw_of_mtable);
 
-- 
2.51.1

Re: [PATCH net-next 22/33] net: dsa: rzn1_a5psw: Remove comma after sentinel-entry in match-table

[Geert Uytterhoeven]

On Fri, 4 Sept 2026 at 06:41, Siddharth Vadapalli <s-vadapalli@ti.com> wrote:
> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

[PATCH net-next 23/33] net: dsa: sja1105: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/dsa/sja1105/sja1105_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dsa/sja1105/sja1105_main.c b/drivers/net/dsa/sja1105/sja1105_main.c
index 1653e83d9123..8984ed55ad59 100644
--- a/drivers/net/dsa/sja1105/sja1105_main.c
+++ b/drivers/net/dsa/sja1105/sja1105_main.c
@@ -3374,7 +3374,7 @@ static const struct of_device_id sja1105_dt_ids[] = {
 	{ .compatible = "nxp,sja1110b", .data = &sja1110b_info },
 	{ .compatible = "nxp,sja1110c", .data = &sja1110c_info },
 	{ .compatible = "nxp,sja1110d", .data = &sja1110d_info },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, sja1105_dt_ids);
 
-- 
2.51.1

[PATCH net-next 24/33] net: bcmasp: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/ethernet/broadcom/asp2/bcmasp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/broadcom/asp2/bcmasp.c b/drivers/net/ethernet/broadcom/asp2/bcmasp.c
index 972474893a6b..3393b5dcbc9a 100644
--- a/drivers/net/ethernet/broadcom/asp2/bcmasp.c
+++ b/drivers/net/ethernet/broadcom/asp2/bcmasp.c
@@ -1209,7 +1209,7 @@ static const struct of_device_id bcmasp_of_match[] = {
 	{ .compatible = "brcm,asp-v2.1", .data = &v21_plat_data },
 	{ .compatible = "brcm,asp-v2.2", .data = &v22_plat_data },
 	{ .compatible = "brcm,asp-v3.0", .data = &v30_plat_data },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, bcmasp_of_match);
 
@@ -1217,7 +1217,7 @@ static const struct of_device_id bcmasp_mdio_of_match[] = {
 	{ .compatible = "brcm,asp-v2.1-mdio", },
 	{ .compatible = "brcm,asp-v2.2-mdio", },
 	{ .compatible = "brcm,asp-v3.0-mdio", },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, bcmasp_mdio_of_match);
 
-- 
2.51.1

[PATCH net-next 25/33] net: spacemit: k1_emac: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/ethernet/spacemit/k1_emac.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/spacemit/k1_emac.c b/drivers/net/ethernet/spacemit/k1_emac.c
index f7f16397a2c2..29385d78fc4b 100644
--- a/drivers/net/ethernet/spacemit/k1_emac.c
+++ b/drivers/net/ethernet/spacemit/k1_emac.c
@@ -1161,7 +1161,7 @@ static const struct ethtool_rmon_hist_range emac_rmon_hist_ranges[] = {
 	{  512, 1023 },
 	{ 1024, 1518 },
 	{ 1519, 4096 },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 
 /* Like dev_fetch_dstats(), but we only use tx_drops */
@@ -2066,7 +2066,7 @@ static const struct dev_pm_ops emac_pm_ops = {
 
 static const struct of_device_id emac_of_match[] = {
 	{ .compatible = "spacemit,k1-emac" },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, emac_of_match);
 
-- 
2.51.1

Re: [PATCH net-next 25/33] net: spacemit: k1_emac: Remove comma after sentinel-entry in match-table

[Vivian Wang]

On 9/4/26 12:36, Siddharth Vadapalli wrote:

> Since sentinel is the last entry and we do not expect further entries
> following it, remove the comma.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
>  drivers/net/ethernet/spacemit/k1_emac.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/spacemit/k1_emac.c b/drivers/net/ethernet/spacemit/k1_emac.c
> index f7f16397a2c2..29385d78fc4b 100644
> --- a/drivers/net/ethernet/spacemit/k1_emac.c
> +++ b/drivers/net/ethernet/spacemit/k1_emac.c
> @@ -1161,7 +1161,7 @@ static const struct ethtool_rmon_hist_range emac_rmon_hist_ranges[] = {
>  	{  512, 1023 },
>  	{ 1024, 1518 },
>  	{ 1519, 4096 },
> -	{ /* sentinel */ },
> +	{ /* sentinel */ }
>  };
>  
>  /* Like dev_fetch_dstats(), but we only use tx_drops */
> @@ -2066,7 +2066,7 @@ static const struct dev_pm_ops emac_pm_ops = {
>  
>  static const struct of_device_id emac_of_match[] = {
>  	{ .compatible = "spacemit,k1-emac" },
> -	{ /* sentinel */ },
> +	{ /* sentinel */ }
>  };
>  MODULE_DEVICE_TABLE(of, emac_of_match);
>  

Acked-by: Vivian Wang <wangruikang@iscas.ac.cn>

Regards,
Vivian "dramforever" Wang

[PATCH net-next 26/33] net: ti: am65-cpsw-nuss: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/ethernet/ti/am65-cpsw-nuss.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/ti/am65-cpsw-nuss.c b/drivers/net/ethernet/ti/am65-cpsw-nuss.c
index 6330540979d7..2551554add17 100644
--- a/drivers/net/ethernet/ti/am65-cpsw-nuss.c
+++ b/drivers/net/ethernet/ti/am65-cpsw-nuss.c
@@ -3506,7 +3506,7 @@ static const struct of_device_id am65_cpsw_nuss_of_mtable[] = {
 	{ .compatible = "ti,j7200-cpswxg-nuss", .data = &j7200_cpswxg_pdata},
 	{ .compatible = "ti,j721e-cpswxg-nuss", .data = &j721e_cpswxg_pdata},
 	{ .compatible = "ti,j784s4-cpswxg-nuss", .data = &j784s4_cpswxg_pdata},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, am65_cpsw_nuss_of_mtable);
 
-- 
2.51.1

[PATCH net-next 27/33] net: ethernet: ti: cpsw: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/ethernet/ti/cpsw.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c
index aa3531e844e8..63b64faeaca0 100644
--- a/drivers/net/ethernet/ti/cpsw.c
+++ b/drivers/net/ethernet/ti/cpsw.c
@@ -1529,7 +1529,7 @@ static const struct of_device_id cpsw_of_mtable[] = {
 	{ .compatible = "ti,am335x-cpsw"},
 	{ .compatible = "ti,am4372-cpsw"},
 	{ .compatible = "ti,dra7-cpsw"},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, cpsw_of_mtable);
 
-- 
2.51.1

[PATCH net-next 28/33] net: ethernet: ti: cpsw_new: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/ethernet/ti/cpsw_new.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/ti/cpsw_new.c b/drivers/net/ethernet/ti/cpsw_new.c
index b6e25a4658d1..07832cad5e3c 100644
--- a/drivers/net/ethernet/ti/cpsw_new.c
+++ b/drivers/net/ethernet/ti/cpsw_new.c
@@ -1875,7 +1875,7 @@ static const struct of_device_id cpsw_of_mtable[] = {
 	{ .compatible = "ti,am335x-cpsw-switch"},
 	{ .compatible = "ti,am4372-cpsw-switch"},
 	{ .compatible = "ti,dra7-cpsw-switch"},
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, cpsw_of_mtable);
 
-- 
2.51.1

[PATCH net-next 29/33] net: mdio: bcm-iproc: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/mdio/mdio-bcm-iproc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/mdio/mdio-bcm-iproc.c b/drivers/net/mdio/mdio-bcm-iproc.c
index 91690b496793..22473c988e3b 100644
--- a/drivers/net/mdio/mdio-bcm-iproc.c
+++ b/drivers/net/mdio/mdio-bcm-iproc.c
@@ -195,7 +195,7 @@ static const struct dev_pm_ops iproc_mdio_pm_ops = {
 
 static const struct of_device_id iproc_mdio_of_match[] = {
 	{ .compatible = "brcm,iproc-mdio", },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, iproc_mdio_of_match);
 
-- 
2.51.1

[PATCH net-next 30/33] net: mdio: bcm-unimac: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/mdio/mdio-bcm-unimac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/mdio/mdio-bcm-unimac.c b/drivers/net/mdio/mdio-bcm-unimac.c
index 37e35f282d9a..31e396cc9fbf 100644
--- a/drivers/net/mdio/mdio-bcm-unimac.c
+++ b/drivers/net/mdio/mdio-bcm-unimac.c
@@ -345,7 +345,7 @@ static const struct of_device_id unimac_mdio_ids[] = {
 	{ .compatible = "brcm,genet-mdio-v2", },
 	{ .compatible = "brcm,genet-mdio-v1", },
 	{ .compatible = "brcm,unimac-mdio", },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, unimac_mdio_ids);
 
-- 
2.51.1

[PATCH net-next 31/33] net: pcs: xpcs-plat: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/pcs/pcs-xpcs-plat.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/pcs/pcs-xpcs-plat.c b/drivers/net/pcs/pcs-xpcs-plat.c
index f4b1b8246ce9..e669a7b248ac 100644
--- a/drivers/net/pcs/pcs-xpcs-plat.c
+++ b/drivers/net/pcs/pcs-xpcs-plat.c
@@ -438,7 +438,7 @@ static const struct of_device_id xpcs_of_ids[] = {
 	{ .compatible = "snps,dw-xpcs-gen4-6g", .data = &xpcs_pma_gen4_6g },
 	{ .compatible = "snps,dw-xpcs-gen5-10g", .data = &xpcs_pma_gen5_10g },
 	{ .compatible = "snps,dw-xpcs-gen5-12g", .data = &xpcs_pma_gen5_12g },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, xpcs_of_ids);
 
-- 
2.51.1

[PATCH net-next 32/33] net: phy: nxp-c45-tja11xx: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/phy/nxp-c45-tja11xx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/phy/nxp-c45-tja11xx.c b/drivers/net/phy/nxp-c45-tja11xx.c
index f526528d2e32..8df152f6671f 100644
--- a/drivers/net/phy/nxp-c45-tja11xx.c
+++ b/drivers/net/phy/nxp-c45-tja11xx.c
@@ -2180,7 +2180,7 @@ module_phy_driver(nxp_c45_driver);
 static const struct mdio_device_id __maybe_unused nxp_c45_tbl[] = {
 	{ PHY_ID_MATCH_MODEL(PHY_ID_TJA_1103) },
 	{ PHY_ID_MATCH_MODEL(PHY_ID_TJA_1120) },
-	{ /*sentinel*/ },
+	{ /*sentinel*/ }
 };
 
 MODULE_DEVICE_TABLE(mdio, nxp_c45_tbl);
-- 
2.51.1

[PATCH net-next 33/33] net: wan: slic_ds26522: Remove comma after sentinel-entry in match-table

[Siddharth Vadapalli]

Since sentinel is the last entry and we do not expect further entries
following it, remove the comma.

Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
 drivers/net/wan/slic_ds26522.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wan/slic_ds26522.c b/drivers/net/wan/slic_ds26522.c
index cbb99fc5ea9f..c57e7cb98410 100644
--- a/drivers/net/wan/slic_ds26522.c
+++ b/drivers/net/wan/slic_ds26522.c
@@ -219,7 +219,7 @@ static int slic_ds26522_probe(struct spi_device *spi)
 
 static const struct spi_device_id slic_ds26522_id[] = {
 	{ .name = "ds26522" },
-	{ /* sentinel */ },
+	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(spi, slic_ds26522_id);
 
-- 
2.51.1

Re: [PATCH net-next 00/33] Remove comma after sentinel entries in match table

[Jakub Kicinski]

On Fri, 4 Sep 2026 10:06:06 +0530 Siddharth Vadapalli wrote:
> This series removes the comma present after sentinel entries in the match
> tables across all networking subsystem drivers, with the exception of the
> davinci_mdio.c driver, for which, the patch posted by Geert at:
> https://lore.kernel.org/r/e481b6de46cecc6ea604ae165272c667274cc621.1788437140.git.geert+renesas@glider.be/
> does the same.
> 
> Since I inquired about a cleanup across the entire networking subsystem but
> didn't get a response:
> https://lore.kernel.org/r/b3d19098-eccd-4619-b667-ad3b7e0a4422@ti.com/
> I decided to post this series. Given that Andrew Lunn acknowledged the
> cleanup of the davinci_mdio driver:
> https://lore.kernel.org/r/abec6831-d344-489d-b70c-1e5dd4d43c2d@lunn.ch/
> and I don't see why the cleanup is specific to davinci_mdio driver alone,
> I am hoping that this series is treated identically to the patch that
> Geert had posted for the davinci_mdio driver.
> 
> Series is based on commit
> 6ebcf5074cff net: openvswitch: don't schedule rebalancing if there are no datapaths
> of the 'main' branch of the net-next tree.

You failed to specify why this "cleanup" makes a difference.
have you ever seen the sentinel not be the last entry?