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

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?

On 04/09/26 9:28 PM, Jakub Kicinski wrote:
> 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?

The 'cleanup' in a way 'guarantees'/'enforces' sentinel being the last 
entry, by catching 'copy-paste' errors at compile-time rather than being 
a silent bug that shows up at runtime. If a new entry were to be 
accidentally added after the sentinel entry in the current drivers, it 
will not cause any compile-time error, but will fail silently at 
runtime. This series 'cleans up' the drivers by making it a compile-time 
error going forward (adding a new entry below sentinel will cause a 
build error with the comma removed after the sentinel entry). I do not 
doubt the review process on the mailing lists where such errors will 
certainly be caught, but isn't it better if they are caught at 
build-time instead, thereby saving reviewers the trouble of pointing out 
such trivial mistakes?

Trailing commas are an indication of 'scope-for-extension' in the 
future, which holds true for 'enums' or other structures where newer 
entires are added towards the end rather than being inserted in the 
middle. Since that isn't the case here, this series makes it clear by 
removing the trailing comma.

On Sat, 5 Sep 2026 11:09:54 +0530 Siddharth Vadapalli wrote:
> > You failed to specify why this "cleanup" makes a difference.
> > have you ever seen the sentinel not be the last entry?  
> 
> The 'cleanup' in a way 'guarantees'/'enforces' sentinel being the last 
> entry, by catching 'copy-paste' errors at compile-time rather than being 
> a silent bug that shows up at runtime. If a new entry were to be 
> accidentally added after the sentinel entry in the current drivers, it 
> will not cause any compile-time error, but will fail silently at 
> runtime. This series 'cleans up' the drivers by making it a compile-time 
> error going forward (adding a new entry below sentinel will cause a 
> build error with the comma removed after the sentinel entry). I do not 
> doubt the review process on the mailing lists where such errors will 
> certainly be caught, but isn't it better if they are caught at 
> build-time instead, thereby saving reviewers the trouble of pointing out 
> such trivial mistakes?

Is it polite to answer a question with a question? 🧐️

Please be mindful of maintainer's time going forward.

> Trailing commas are an indication of 'scope-for-extension' in the 
> future, which holds true for 'enums' or other structures where newer 
> entires are added towards the end rather than being inserted in the 
> middle. Since that isn't the case here, this series makes it clear by 
> removing the trailing comma.

On 05/09/26 23:33, Jakub Kicinski wrote:
> On Sat, 5 Sep 2026 11:09:54 +0530 Siddharth Vadapalli wrote:
>>> You failed to specify why this "cleanup" makes a difference.
>>> have you ever seen the sentinel not be the last entry?
>>
>> The 'cleanup' in a way 'guarantees'/'enforces' sentinel being the last
>> entry, by catching 'copy-paste' errors at compile-time rather than being
>> a silent bug that shows up at runtime. If a new entry were to be
>> accidentally added after the sentinel entry in the current drivers, it
>> will not cause any compile-time error, but will fail silently at
>> runtime. This series 'cleans up' the drivers by making it a compile-time
>> error going forward (adding a new entry below sentinel will cause a
>> build error with the comma removed after the sentinel entry). I do not
>> doubt the review process on the mailing lists where such errors will
>> certainly be caught, but isn't it better if they are caught at
>> build-time instead, thereby saving reviewers the trouble of pointing out
>> such trivial mistakes?
> 
> Is it polite to answer a question with a question? 🧐️

I apologize if it came across as an impolite response. That isn't my 
intent. The 'question' was meant to be more of a "It's better to do so, 
right?" kind of 'question' which is actually a statement, but I posed it as 
a question to make it sound polite. Writing the same as a statement of fact 
appeared to be impolite to me, but I realize now that posing it as a 
question could also be taken as an impolite response.

I should have omitted the closing remark entirely and will keep this in 
mind during future interactions.

> 
> Please be mindful of maintainer's time going forward.
> 
>> Trailing commas are an indication of 'scope-for-extension' in the
>> future, which holds true for 'enums' or other structures where newer
>> entires are added towards the end rather than being inserted in the
>> middle. Since that isn't the case here, this series makes it clear by
>> removing the trailing comma.
>