Subject: Re: [PATCH] ALSA: Do not build obsolete API

From: "Christophe Leroy (CS GROUP)" <[email-masked]>
Date: Thu, 4 Dec 2025 08:39:03 +0100

Le 03/12/2025 à 23:34, David Heidelberg via B4 Relay a écrit :
> [Vous ne recevez pas souvent de courriers de [email-masked]. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]
> 
> From: David Heidelberg <[email-masked]>
> 
> ALSA 0.9.0-rc3 is from 2002, 23 years old.
> 
> Signed-off-by: David Heidelberg <[email-masked]>
> ---
> Maybe I could drop also the code and Kconfig option?
> 
> David
> ---
>   arch/arm/configs/am200epdkit_defconfig    | 1 -
>   arch/arm/configs/lpc32xx_defconfig        | 1 -
>   arch/arm/configs/omap1_defconfig          | 1 -
>   arch/arm/configs/tegra_defconfig          | 1 -
>   arch/mips/configs/gcw0_defconfig          | 1 -
>   arch/mips/configs/loongson1_defconfig     | 1 -
>   arch/mips/configs/qi_lb60_defconfig       | 1 -
>   arch/mips/configs/rbtx49xx_defconfig      | 1 -
>   arch/mips/configs/rs90_defconfig          | 1 -
>   arch/powerpc/configs/85xx-hw.config       | 1 -
>   arch/powerpc/configs/86xx-hw.config       | 1 -
>   arch/powerpc/configs/mpc5200_defconfig    | 1 -
>   arch/powerpc/configs/ppc6xx_defconfig     | 1 -

For powerpc:

Reviewed-by: Christophe Leroy (CS GROUP) <[email-masked]>
Acked-by: Christophe Leroy (CS GROUP) <[email-masked]>

>   arch/sh/configs/edosk7760_defconfig       | 1 -
>   arch/sh/configs/se7724_defconfig          | 1 -
>   arch/sh/configs/sh7785lcr_32bit_defconfig | 1 -
>   sound/core/Kconfig                        | 2 +-
>   17 files changed, 1 insertion(+), 17 deletions(-)
> 
> diff --git a/arch/arm/configs/am200epdkit_defconfig b/arch/arm/configs/am200epdkit_defconfig
> index 134a559aba3dd..2367b1685c1cf 100644
> --- a/arch/arm/configs/am200epdkit_defconfig
> +++ b/arch/arm/configs/am200epdkit_defconfig
> @@ -68,7 +68,6 @@ CONFIG_SOUND=m
>   CONFIG_SND=m
>   CONFIG_SND_MIXER_OSS=m
>   CONFIG_SND_PCM_OSS=m
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   CONFIG_SND_PXA2XX_AC97=m
>   CONFIG_USB_GADGET=y
> diff --git a/arch/arm/configs/lpc32xx_defconfig b/arch/arm/configs/lpc32xx_defconfig
> index 2bddb0924a8c0..b9e2e603cd95e 100644
> --- a/arch/arm/configs/lpc32xx_defconfig
> +++ b/arch/arm/configs/lpc32xx_defconfig
> @@ -113,7 +113,6 @@ CONFIG_LOGO=y
>   # CONFIG_LOGO_LINUX_VGA16 is not set
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   CONFIG_SND_DEBUG=y
>   CONFIG_SND_DEBUG_VERBOSE=y
> diff --git a/arch/arm/configs/omap1_defconfig b/arch/arm/configs/omap1_defconfig
> index dee820474f444..df88763fc7c3d 100644
> --- a/arch/arm/configs/omap1_defconfig
> +++ b/arch/arm/configs/omap1_defconfig
> @@ -148,7 +148,6 @@ CONFIG_SOUND=y
>   CONFIG_SND=y
>   CONFIG_SND_MIXER_OSS=y
>   CONFIG_SND_PCM_OSS=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   CONFIG_SND_DUMMY=y
>   CONFIG_SND_USB_AUDIO=y
> diff --git a/arch/arm/configs/tegra_defconfig b/arch/arm/configs/tegra_defconfig
> index ce70ff07c978a..68aedaf92667a 100644
> --- a/arch/arm/configs/tegra_defconfig
> +++ b/arch/arm/configs/tegra_defconfig
> @@ -219,7 +219,6 @@ CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y
>   CONFIG_LOGO=y
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_DRIVERS is not set
>   CONFIG_SND_HDA_TEGRA=y
>   CONFIG_SND_HDA_INPUT_BEEP=y
> diff --git a/arch/mips/configs/gcw0_defconfig b/arch/mips/configs/gcw0_defconfig
> index fda9971bdd8d9..adb9fd62ddb0d 100644
> --- a/arch/mips/configs/gcw0_defconfig
> +++ b/arch/mips/configs/gcw0_defconfig
> @@ -79,7 +79,6 @@ CONFIG_LOGO=y
>   # CONFIG_LOGO_LINUX_VGA16 is not set
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_PROC_FS is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_SPI is not set
> diff --git a/arch/mips/configs/loongson1_defconfig b/arch/mips/configs/loongson1_defconfig
> index 02d29110f7024..1d9781ff96986 100644
> --- a/arch/mips/configs/loongson1_defconfig
> +++ b/arch/mips/configs/loongson1_defconfig
> @@ -119,7 +119,6 @@ CONFIG_WATCHDOG_SYSFS=y
>   CONFIG_LOONGSON1_WDT=y
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_MIPS is not set
>   # CONFIG_SND_USB is not set
> diff --git a/arch/mips/configs/qi_lb60_defconfig b/arch/mips/configs/qi_lb60_defconfig
> index 5f5b0254d75e7..a1bb0792f6eb1 100644
> --- a/arch/mips/configs/qi_lb60_defconfig
> +++ b/arch/mips/configs/qi_lb60_defconfig
> @@ -81,7 +81,6 @@ CONFIG_LOGO=y
>   # CONFIG_LOGO_LINUX_CLUT224 is not set
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_SPI is not set
> diff --git a/arch/mips/configs/rbtx49xx_defconfig b/arch/mips/configs/rbtx49xx_defconfig
> index 03a7bbe28a532..49c709d663beb 100644
> --- a/arch/mips/configs/rbtx49xx_defconfig
> +++ b/arch/mips/configs/rbtx49xx_defconfig
> @@ -53,7 +53,6 @@ CONFIG_TXX9_WDT=m
>   # CONFIG_VGA_ARB is not set
>   CONFIG_SOUND=m
>   CONFIG_SND=m
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_PCI is not set
> diff --git a/arch/mips/configs/rs90_defconfig b/arch/mips/configs/rs90_defconfig
> index a53dd66e9b864..8382d535e6dc1 100644
> --- a/arch/mips/configs/rs90_defconfig
> +++ b/arch/mips/configs/rs90_defconfig
> @@ -105,7 +105,6 @@ CONFIG_LOGO=y
>   CONFIG_SOUND=y
>   CONFIG_SND=y
>   # CONFIG_SND_PCM_TIMER is not set
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_PROC_FS is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_MIPS is not set
> diff --git a/arch/powerpc/configs/85xx-hw.config b/arch/powerpc/configs/85xx-hw.config
> index 8aff832173977..2b19c20a9a2c4 100644
> --- a/arch/powerpc/configs/85xx-hw.config
> +++ b/arch/powerpc/configs/85xx-hw.config
> @@ -117,7 +117,6 @@ CONFIG_SND_INTEL8X0=y
>   CONFIG_SND_POWERPC_SOC=y
>   # CONFIG_SND_PPC is not set
>   CONFIG_SND_SOC=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_USB is not set
>   CONFIG_SND=y
>   CONFIG_SOUND=y
> diff --git a/arch/powerpc/configs/86xx-hw.config b/arch/powerpc/configs/86xx-hw.config
> index e7bd265fae5a4..07f30ab881e59 100644
> --- a/arch/powerpc/configs/86xx-hw.config
> +++ b/arch/powerpc/configs/86xx-hw.config
> @@ -80,7 +80,6 @@ CONFIG_SERIO_LIBPS2=y
>   CONFIG_SND_INTEL8X0=y
>   CONFIG_SND_MIXER_OSS=y
>   CONFIG_SND_PCM_OSS=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   CONFIG_SND=y
>   CONFIG_SOUND=y
>   CONFIG_ULI526X=y
> diff --git a/arch/powerpc/configs/mpc5200_defconfig b/arch/powerpc/configs/mpc5200_defconfig
> index c0fe5e76604a0..617650cea56a9 100644
> --- a/arch/powerpc/configs/mpc5200_defconfig
> +++ b/arch/powerpc/configs/mpc5200_defconfig
> @@ -75,7 +75,6 @@ CONFIG_FB_SM501=m
>   CONFIG_LOGO=y
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_PCI is not set
>   # CONFIG_SND_PPC is not set
> diff --git a/arch/powerpc/configs/ppc6xx_defconfig b/arch/powerpc/configs/ppc6xx_defconfig
> index b082c1fae13c9..787d707f64a42 100644
> --- a/arch/powerpc/configs/ppc6xx_defconfig
> +++ b/arch/powerpc/configs/ppc6xx_defconfig
> @@ -726,7 +726,6 @@ CONFIG_SND_OSSEMUL=y
>   CONFIG_SND_MIXER_OSS=m
>   CONFIG_SND_PCM_OSS=m
>   CONFIG_SND_DYNAMIC_MINORS=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   CONFIG_SND_VERBOSE_PRINTK=y
>   CONFIG_SND_DEBUG=y
>   CONFIG_SND_DEBUG_VERBOSE=y
> diff --git a/arch/sh/configs/edosk7760_defconfig b/arch/sh/configs/edosk7760_defconfig
> index abeae220606a3..905fac1072845 100644
> --- a/arch/sh/configs/edosk7760_defconfig
> +++ b/arch/sh/configs/edosk7760_defconfig
> @@ -79,7 +79,6 @@ CONFIG_FB_TILEBLITTING=y
>   CONFIG_FB_SH_MOBILE_LCDC=m
>   CONFIG_SOUND=y
>   CONFIG_SND=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   CONFIG_SND_VERBOSE_PRINTK=y
>   CONFIG_SND_SOC=y
> diff --git a/arch/sh/configs/se7724_defconfig b/arch/sh/configs/se7724_defconfig
> index 9e3a54936f76f..8ca46d704c8ba 100644
> --- a/arch/sh/configs/se7724_defconfig
> +++ b/arch/sh/configs/se7724_defconfig
> @@ -83,7 +83,6 @@ CONFIG_LOGO=y
>   # CONFIG_LOGO_SUPERH_VGA16 is not set
>   CONFIG_SOUND=y
>   CONFIG_SND=m
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_DRIVERS is not set
>   # CONFIG_SND_SPI is not set
>   # CONFIG_SND_SUPERH is not set
> diff --git a/arch/sh/configs/sh7785lcr_32bit_defconfig b/arch/sh/configs/sh7785lcr_32bit_defconfig
> index eb63aa61b0465..5468cc53cddb4 100644
> --- a/arch/sh/configs/sh7785lcr_32bit_defconfig
> +++ b/arch/sh/configs/sh7785lcr_32bit_defconfig
> @@ -93,7 +93,6 @@ CONFIG_SND_PCM_OSS=y
>   CONFIG_SND_SEQUENCER_OSS=y
>   CONFIG_SND_HRTIMER=y
>   CONFIG_SND_DYNAMIC_MINORS=y
> -# CONFIG_SND_SUPPORT_OLD_API is not set
>   # CONFIG_SND_VERBOSE_PROCFS is not set
>   CONFIG_SND_VERBOSE_PRINTK=y
>   CONFIG_SND_DEBUG=y
> diff --git a/sound/core/Kconfig b/sound/core/Kconfig
> index 48db44fa56feb..4e7bc370ffd7f 100644
> --- a/sound/core/Kconfig
> +++ b/sound/core/Kconfig
> @@ -155,7 +155,7 @@ config SND_MAX_CARDS
> 
>   config SND_SUPPORT_OLD_API
>          bool "Support old ALSA API"
> -       default y
> +       default n
>          help
>            Say Y here to support the obsolete ALSA PCM API (ver.0.9.0 rc3
>            or older).
> 
> ---
> base-commit: b2c27842ba853508b0da00187a7508eb3a96c8f7
> change-id: 20251203-old-alsa-fa2c2cb038e1
> 
> Best regards,
> --
> David Heidelberg <[email-masked]>
> 
>