Re: [PATCH] arch: sh: defconfig: Drop obsolete CONFIG_NET_CLS_TCINDEX

Hi Johan,

On Sun, 2025-03-23 at 20:13 +0100, Johan Korsnes wrote:
> This option was removed from the Kconfig in commit
> 8c710f75256b ("net/sched: Retire tcindex classifier") but it was not
> removed from the defconfigs.
> 
> Fixes: 8c710f75256b ("net/sched: Retire tcindex classifier")
> Signed-off-by: Johan Korsnes <[email protected]>
> Cc: Yoshinori Sato <[email protected]>
> Cc: Rich Felker <[email protected]>
> Cc: John Paul Adrian Glaubitz <[email protected]>
> ---
>  arch/sh/configs/se7712_defconfig       | 1 -
>  arch/sh/configs/se7721_defconfig       | 1 -
>  arch/sh/configs/sh7710voipgw_defconfig | 1 -
>  arch/sh/configs/titan_defconfig        | 1 -
>  4 files changed, 4 deletions(-)
> 
> diff --git a/arch/sh/configs/se7712_defconfig b/arch/sh/configs/se7712_defconfig
> index 20f07aee5bde..49a4961889de 100644
> --- a/arch/sh/configs/se7712_defconfig
> +++ b/arch/sh/configs/se7712_defconfig
> @@ -57,7 +57,6 @@ CONFIG_NET_SCH_TBF=y
>  CONFIG_NET_SCH_GRED=y
>  CONFIG_NET_SCH_DSMARK=y
>  CONFIG_NET_SCH_NETEM=y
> -CONFIG_NET_CLS_TCINDEX=y
>  CONFIG_NET_CLS_ROUTE4=y
>  CONFIG_NET_CLS_FW=y
>  CONFIG_MTD=y
> diff --git a/arch/sh/configs/se7721_defconfig b/arch/sh/configs/se7721_defconfig
> index 00862d3c030d..de293792db84 100644
> --- a/arch/sh/configs/se7721_defconfig
> +++ b/arch/sh/configs/se7721_defconfig
> @@ -56,7 +56,6 @@ CONFIG_NET_SCH_TBF=y
>  CONFIG_NET_SCH_GRED=y
>  CONFIG_NET_SCH_DSMARK=y
>  CONFIG_NET_SCH_NETEM=y
> -CONFIG_NET_CLS_TCINDEX=y
>  CONFIG_NET_CLS_ROUTE4=y
>  CONFIG_NET_CLS_FW=y
>  CONFIG_MTD=y
> diff --git a/arch/sh/configs/sh7710voipgw_defconfig b/arch/sh/configs/sh7710voipgw_defconfig
> index 99a5d0760532..5b151bb2bc43 100644
> --- a/arch/sh/configs/sh7710voipgw_defconfig
> +++ b/arch/sh/configs/sh7710voipgw_defconfig
> @@ -27,7 +27,6 @@ CONFIG_NETFILTER=y
>  CONFIG_NET_SCHED=y
>  CONFIG_NET_SCH_CBQ=y
>  CONFIG_NET_CLS_BASIC=y
> -CONFIG_NET_CLS_TCINDEX=y
>  CONFIG_NET_CLS_ROUTE4=y
>  CONFIG_NET_CLS_U32=y
>  CONFIG_MTD=y
> diff --git a/arch/sh/configs/titan_defconfig b/arch/sh/configs/titan_defconfig
> index 99bc0e889287..e2928311a126 100644
> --- a/arch/sh/configs/titan_defconfig
> +++ b/arch/sh/configs/titan_defconfig
> @@ -119,7 +119,6 @@ CONFIG_NET_SCH_DSMARK=m
>  CONFIG_NET_SCH_NETEM=m
>  CONFIG_NET_SCH_INGRESS=m
>  CONFIG_NET_CLS_BASIC=m
> -CONFIG_NET_CLS_TCINDEX=m
>  CONFIG_NET_CLS_ROUTE4=m
>  CONFIG_NET_CLS_FW=m
>  CONFIG_NET_CLS_U32=m

Thanks for catching this! Will pick this up shortly.

Reviewed-by: John Paul Adrian Glaubitz <[email protected]>

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913