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

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

[Alexander Sverdlin]

On Fri, 2026-09-04 at 10:06 +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>

Reviewed-by: Alexander Sverdlin <alex@sverdlin.org>

> ---
>  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);

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

[Alexander Sverdlin]

On Fri, 2026-09-04 at 10:06 +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>

Reviewed-by: Alexander Sverdlin <alex@sverdlin.org>

> ---
>  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);

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

[Alexander Sverdlin]

On Fri, 2026-09-04 at 10:06 +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>

Reviewed-by: Alexander Sverdlin <alex@sverdlin.org>

> ---
>  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);
>