[PATCH 00/37] ASoC: Standardize ASoC menu

[PATCH 00/37] ASoC: Standardize ASoC menu

[Kuninori Morimoto]

Hi Mark

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.
    
Let's standardize ASoC menu, like below
    
	ASoC support for Analog Devices  --->
	ASoC support for AMD  --->
	ASoC support for Apple  --->
	ASoC support for Atmel  --->
	ASoC support for Au1x  ----
	ASoC support for Broadcom  --->
	ASoC support for Cirrus  --->
	ASoC support for DesignWare  --->
	ASoC support for Freescale  --->
	ASoC support for Google  --->
	ASoC support for Hisilicon  --->
	...

Kuninori Morimoto (37):
  ASoC: ti: Standardize ASoC menu
  ASoC: adi: Standardize ASoC menu
  ASoC: amd: Standardize ASoC menu
  ASoC: bcm: Standardize ASoC menu
  ASoC: dwc: Standardize ASoC menu
  ASoC: fsl: Standardize ASoC menu
  ASoC: img: Standardize ASoC menu
  ASoC: mxs: Standardize ASoC menu
  ASoC: pxa: Standardize ASoC menu
  ASoC: sof: Standardize ASoC menu
  ASoC: sti: Standardize ASoC menu
  ASoC: stm: Standardize ASoC menu
  ASoC: au1x: Standardize ASoC menu
  ASoC: sdca: Standardize ASoC menu
  ASoC: sprd: Standardize ASoC menu
  ASoC: apple: Standardize ASoC menu
  ASoC: atmel: Standardize ASoC menu
  ASoC: intel: Standardize ASoC menu
  ASoC: spear: Standardize ASoC menu
  ASoC: sunxi: Standardize ASoC menu
  ASoC: tegra: Standardize ASoC menu
  ASoC: ux500: Standardize ASoC menu
  ASoC: cirrus: Standardize ASoC menu
  ASoC: google: Standardize ASoC menu
  ASoC: jz4740: Standardize ASoC menu
  ASoC: xilinx: Standardize ASoC menu
  ASoC: xtensa: Standardize ASoC menu
  ASoC: renesas: Standardize ASoC menu
  ASoC: generic: Standardize ASoC menu
  ASoC: kirkwood: Standardize ASoC menu
  ASoC: loongson: Standardize ASoC menu
  ASoC: mediatek: Standardize ASoC menu
  ASoC: rockchip: Standardize ASoC menu
  ASoC: starfive: Standardize ASoC menu
  ASoC: uniphier: Standardize ASoC menu
  ASoC: hisilicon: Standardize ASoC menu
  ASoC: sdw_utils: Standardize ASoC menu

 sound/soc/Kconfig           |  7 ++-----
 sound/soc/adi/Kconfig       |  9 +++------
 sound/soc/amd/Kconfig       |  3 +++
 sound/soc/apple/Kconfig     |  4 ++++
 sound/soc/atmel/Kconfig     | 11 ++---------
 sound/soc/au1x/Kconfig      |  4 ++++
 sound/soc/bcm/Kconfig       |  4 ++++
 sound/soc/cirrus/Kconfig    |  3 +++
 sound/soc/dwc/Kconfig       |  3 +++
 sound/soc/fsl/Kconfig       |  2 +-
 sound/soc/generic/Kconfig   |  4 ++++
 sound/soc/google/Kconfig    |  3 +++
 sound/soc/hisilicon/Kconfig |  4 ++++
 sound/soc/img/Kconfig       | 13 +++----------
 sound/soc/intel/Kconfig     |  4 ++++
 sound/soc/jz4740/Kconfig    |  4 ++++
 sound/soc/kirkwood/Kconfig  |  3 +++
 sound/soc/loongson/Kconfig  |  2 +-
 sound/soc/mediatek/Kconfig  |  4 ++++
 sound/soc/mxs/Kconfig       |  2 +-
 sound/soc/pxa/Kconfig       |  4 ++++
 sound/soc/renesas/Kconfig   |  2 +-
 sound/soc/rockchip/Kconfig  | 23 ++++++++---------------
 sound/soc/sdca/Kconfig      |  3 +++
 sound/soc/sdw_utils/Kconfig |  4 ++++
 sound/soc/sof/Kconfig       |  2 +-
 sound/soc/spear/Kconfig     |  4 ++++
 sound/soc/sprd/Kconfig      |  4 ++++
 sound/soc/starfive/Kconfig  | 12 ++++--------
 sound/soc/sti/Kconfig       |  2 +-
 sound/soc/stm/Kconfig       |  2 +-
 sound/soc/sunxi/Kconfig     |  2 +-
 sound/soc/tegra/Kconfig     |  4 ++++
 sound/soc/ti/Kconfig        |  2 +-
 sound/soc/uniphier/Kconfig  | 14 +++-----------
 sound/soc/ux500/Kconfig     |  2 +-
 sound/soc/xilinx/Kconfig    |  4 ++++
 sound/soc/xtensa/Kconfig    |  4 ++++
 38 files changed, 113 insertions(+), 74 deletions(-)

-- 
2.43.0

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Chen-Yu Tsai]

(Dropped non ALSA / ASoC maintainers from CC)

On Tue, Jun 10, 2025 at 8:44 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
>
>
> Hi Mark
>
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
>
> Let's standardize ASoC menu, like below
>
>         ASoC support for Analog Devices  --->
>         ASoC support for AMD  --->
>         ASoC support for Apple  --->
>         ASoC support for Atmel  --->
>         ASoC support for Au1x  ----
>         ASoC support for Broadcom  --->
>         ASoC support for Cirrus  --->
>         ASoC support for DesignWare  --->
>         ASoC support for Freescale  --->
>         ASoC support for Google  --->
>         ASoC support for Hisilicon  --->
>         ...

It seems only this cover letter made it to the mailing lists. None
of the actual patches are there.


ChenYu

> Kuninori Morimoto (37):
>   ASoC: ti: Standardize ASoC menu
>   ASoC: adi: Standardize ASoC menu
>   ASoC: amd: Standardize ASoC menu
>   ASoC: bcm: Standardize ASoC menu
>   ASoC: dwc: Standardize ASoC menu
>   ASoC: fsl: Standardize ASoC menu
>   ASoC: img: Standardize ASoC menu
>   ASoC: mxs: Standardize ASoC menu
>   ASoC: pxa: Standardize ASoC menu
>   ASoC: sof: Standardize ASoC menu
>   ASoC: sti: Standardize ASoC menu
>   ASoC: stm: Standardize ASoC menu
>   ASoC: au1x: Standardize ASoC menu
>   ASoC: sdca: Standardize ASoC menu
>   ASoC: sprd: Standardize ASoC menu
>   ASoC: apple: Standardize ASoC menu
>   ASoC: atmel: Standardize ASoC menu
>   ASoC: intel: Standardize ASoC menu
>   ASoC: spear: Standardize ASoC menu
>   ASoC: sunxi: Standardize ASoC menu
>   ASoC: tegra: Standardize ASoC menu
>   ASoC: ux500: Standardize ASoC menu
>   ASoC: cirrus: Standardize ASoC menu
>   ASoC: google: Standardize ASoC menu
>   ASoC: jz4740: Standardize ASoC menu
>   ASoC: xilinx: Standardize ASoC menu
>   ASoC: xtensa: Standardize ASoC menu
>   ASoC: renesas: Standardize ASoC menu
>   ASoC: generic: Standardize ASoC menu
>   ASoC: kirkwood: Standardize ASoC menu
>   ASoC: loongson: Standardize ASoC menu
>   ASoC: mediatek: Standardize ASoC menu
>   ASoC: rockchip: Standardize ASoC menu
>   ASoC: starfive: Standardize ASoC menu
>   ASoC: uniphier: Standardize ASoC menu
>   ASoC: hisilicon: Standardize ASoC menu
>   ASoC: sdw_utils: Standardize ASoC menu
>
>  sound/soc/Kconfig           |  7 ++-----
>  sound/soc/adi/Kconfig       |  9 +++------
>  sound/soc/amd/Kconfig       |  3 +++
>  sound/soc/apple/Kconfig     |  4 ++++
>  sound/soc/atmel/Kconfig     | 11 ++---------
>  sound/soc/au1x/Kconfig      |  4 ++++
>  sound/soc/bcm/Kconfig       |  4 ++++
>  sound/soc/cirrus/Kconfig    |  3 +++
>  sound/soc/dwc/Kconfig       |  3 +++
>  sound/soc/fsl/Kconfig       |  2 +-
>  sound/soc/generic/Kconfig   |  4 ++++
>  sound/soc/google/Kconfig    |  3 +++
>  sound/soc/hisilicon/Kconfig |  4 ++++
>  sound/soc/img/Kconfig       | 13 +++----------
>  sound/soc/intel/Kconfig     |  4 ++++
>  sound/soc/jz4740/Kconfig    |  4 ++++
>  sound/soc/kirkwood/Kconfig  |  3 +++
>  sound/soc/loongson/Kconfig  |  2 +-
>  sound/soc/mediatek/Kconfig  |  4 ++++
>  sound/soc/mxs/Kconfig       |  2 +-
>  sound/soc/pxa/Kconfig       |  4 ++++
>  sound/soc/renesas/Kconfig   |  2 +-
>  sound/soc/rockchip/Kconfig  | 23 ++++++++---------------
>  sound/soc/sdca/Kconfig      |  3 +++
>  sound/soc/sdw_utils/Kconfig |  4 ++++
>  sound/soc/sof/Kconfig       |  2 +-
>  sound/soc/spear/Kconfig     |  4 ++++
>  sound/soc/sprd/Kconfig      |  4 ++++
>  sound/soc/starfive/Kconfig  | 12 ++++--------
>  sound/soc/sti/Kconfig       |  2 +-
>  sound/soc/stm/Kconfig       |  2 +-
>  sound/soc/sunxi/Kconfig     |  2 +-
>  sound/soc/tegra/Kconfig     |  4 ++++
>  sound/soc/ti/Kconfig        |  2 +-
>  sound/soc/uniphier/Kconfig  | 14 +++-----------
>  sound/soc/ux500/Kconfig     |  2 +-
>  sound/soc/xilinx/Kconfig    |  4 ++++
>  sound/soc/xtensa/Kconfig    |  4 ++++
>  38 files changed, 113 insertions(+), 74 deletions(-)
>
> --
> 2.43.0
>

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Geraldo Nascimento]

On Tue, Jun 10, 2025 at 11:16:22AM +0800, Chen-Yu Tsai wrote:
> 
> It seems only this cover letter made it to the mailing lists. None
> of the actual patches are there.
> 
> 
> ChenYu
>

Chen-Yu, did Morimoto-san forgot to Cc: list or message was dropped
silently (like by spam filter)?

If it's the latter I suggest to contact postmaster@kernel.org and
report the incident.

Thank you,
Geraldo Nascimento

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Chen-Yu Tsai]

On Tue, Jun 10, 2025 at 11:47 AM Geraldo Nascimento
<geraldogabriel@gmail.com> wrote:
>
> On Tue, Jun 10, 2025 at 11:16:22AM +0800, Chen-Yu Tsai wrote:
> >
> > It seems only this cover letter made it to the mailing lists. None
> > of the actual patches are there.
> >
> >
> > ChenYu
> >
>
> Chen-Yu, did Morimoto-san forgot to Cc: list or message was dropped
> silently (like by spam filter)?

Looking at the Cc list I believe it's the former.

ChenYu

> If it's the latter I suggest to contact postmaster@kernel.org and
> report the incident.
>
> Thank you,
> Geraldo Nascimento

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Kuninori Morimoto]

Hi

> > > It seems only this cover letter made it to the mailing lists. None
> > > of the actual patches are there.
(snip)
> > Chen-Yu, did Morimoto-san forgot to Cc: list or message was dropped
> > silently (like by spam filter)?
> 
> Looking at the Cc list I believe it's the former.

Hmm... strange...

I have checked posted patch, but all patches includes your mail address.
One concern is I added your address on "To", instead of "Cc".

Thank you for your help !!

Best regards
---
Kuninori Morimoto

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Geraldo Nascimento]

On Tue, Jun 10, 2025 at 03:57:15AM +0000, Kuninori Morimoto wrote:
> > > Chen-Yu, did Morimoto-san forgot to Cc: list or message was dropped
> > > silently (like by spam filter)?
> > 
> > Looking at the Cc list I believe it's the former.
> 
> Hmm... strange...
> 
> I have checked posted patch, but all patches includes your mail address.
> One concern is I added your address on "To", instead of "Cc".
> 
> Thank you for your help !!
> 
> Best regards
> ---
> Kuninori Morimoto

Hi Morimoto-san,

Sorry to intervene again but I think Chen-Yu's concern is about Cc:
missing to open list (linux-sound) not his own address.

He was referring to the (likely) fact that you forgot to Cc: list...

Thank you!
Geraldo Nascimento

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Kuninori Morimoto]

Hi Geraldo

> > I have checked posted patch, but all patches includes your mail address.
> > One concern is I added your address on "To", instead of "Cc".
(sniop)
> Sorry to intervene again but I think Chen-Yu's concern is about Cc:
> missing to open list (linux-sound) not his own address.
> 
> He was referring to the (likely) fact that you forgot to Cc: list...

Oops, Indeed
Sorry for my miss, will post again.

Thank you for your help !!

Best regards
---
Kuninori Morimoto

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Mark Brown]

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

On Tue, Jun 10, 2025 at 04:18:35AM +0000, Kuninori Morimoto wrote:

> > Sorry to intervene again but I think Chen-Yu's concern is about Cc:
> > missing to open list (linux-sound) not his own address.

> > He was referring to the (likely) fact that you forgot to Cc: list...

> Oops, Indeed
> Sorry for my miss, will post again.

I do see a CC to linux-sound in the mails I've got, and lore appears to
have a full copy of the thread which means it should've hit some list.
A bit confused here but I think it's fine?

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

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Kuninori Morimoto]

Hi Mark

> > > Sorry to intervene again but I think Chen-Yu's concern is about Cc:
> > > missing to open list (linux-sound) not his own address.
> 
> > > He was referring to the (likely) fact that you forgot to Cc: list...
> 
> > Oops, Indeed
> > Sorry for my miss, will post again.
> 
> I do see a CC to linux-sound in the mails I've got, and lore appears to
> have a full copy of the thread which means it should've hit some list.
> A bit confused here but I think it's fine?

Sorry for my noise.

I have re-post the patch-set, so ML member got full patch-set in final.
But To member have received same patch twice (with Cc: linux-sound,
without Cc: linux-sound).

Thank you for your help !!

Best regards
---
Kuninori Morimoto

[PATCH 01/37] ASoC: ti: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/ti/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/ti/Kconfig b/sound/soc/ti/Kconfig
index e22e41af32269..f9c292cb7428f 100644
--- a/sound/soc/ti/Kconfig
+++ b/sound/soc/ti/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
-menu "Audio support for Texas Instruments SoCs"
+menu "ASoC support for Texas Instruments"
 depends on DMA_OMAP || TI_EDMA || TI_K3_UDMA || COMPILE_TEST
 
 config SND_SOC_TI_EDMA_PCM
-- 
2.43.0

[PATCH 02/37] ASoC: adi: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/adi/Kconfig | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/sound/soc/adi/Kconfig b/sound/soc/adi/Kconfig
index 0236dc5b4e9f0..228805e245e10 100644
--- a/sound/soc/adi/Kconfig
+++ b/sound/soc/adi/Kconfig
@@ -1,12 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0-only
-config SND_SOC_ADI
-	tristate "Audio support for Analog Devices reference designs"
-	help
-	  Audio support for various reference designs by Analog Devices.
+menu "ASoC support for Analog Devices"
 
 config SND_SOC_ADI_AXI_I2S
 	tristate "AXI-I2S support"
-	depends on SND_SOC_ADI
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	select REGMAP_MMIO
 	help
@@ -14,8 +10,9 @@ config SND_SOC_ADI_AXI_I2S
 
 config SND_SOC_ADI_AXI_SPDIF
 	tristate "AXI-SPDIF support"
-	depends on SND_SOC_ADI
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	select REGMAP_MMIO
 	help
 	  ASoC driver for the Analog Devices AXI-SPDIF softcore peripheral.
+
+endmenu
-- 
2.43.0

[PATCH 03/37] ASoC: amd: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/amd/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sound/soc/amd/Kconfig b/sound/soc/amd/Kconfig
index c7daae392d746..b1c4dc95129bc 100644
--- a/sound/soc/amd/Kconfig
+++ b/sound/soc/amd/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for AMD"
+
 config SND_SOC_AMD_ACP
 	tristate "AMD Audio Coprocessor support"
 	help
@@ -185,3 +187,4 @@ config SND_SOC_AMD_PS_MACH
           If unsure select "N".
 
 endif
+endmenu
-- 
2.43.0

Re: [PATCH 03/37] ASoC: amd: Standardize ASoC menu

[Mukunda,Vijendar]

On 10/06/25 09:49, Kuninori Morimoto wrote:
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
>
> Let's standardize ASoC menu
LGTM.
Reviewed-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>  sound/soc/amd/Kconfig | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/sound/soc/amd/Kconfig b/sound/soc/amd/Kconfig
> index c7daae392d746..b1c4dc95129bc 100644
> --- a/sound/soc/amd/Kconfig
> +++ b/sound/soc/amd/Kconfig
> @@ -1,4 +1,6 @@
>  # SPDX-License-Identifier: GPL-2.0-only
> +menu "ASoC support for AMD"
> +
>  config SND_SOC_AMD_ACP
>  	tristate "AMD Audio Coprocessor support"
>  	help
> @@ -185,3 +187,4 @@ config SND_SOC_AMD_PS_MACH
>            If unsure select "N".
>  
>  endif
> +endmenu

[PATCH 04/37] ASoC: bcm: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/bcm/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/bcm/Kconfig b/sound/soc/bcm/Kconfig
index 4218057b08742..13eb8a96c2acb 100644
--- a/sound/soc/bcm/Kconfig
+++ b/sound/soc/bcm/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Broadcom"
+
 config SND_BCM2835_SOC_I2S
 	tristate "SoC Audio support for the Broadcom BCM2835 I2S module"
 	depends on ARCH_BCM2835 || COMPILE_TEST
@@ -26,3 +28,5 @@ config SND_BCM63XX_I2S_WHISTLER
 	  DSL/PON chips (bcm63158, bcm63178)
 
 	  If you don't know what to do here, say N
+
+endmenu
-- 
2.43.0

[PATCH 05/37] ASoC: dwc: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/dwc/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sound/soc/dwc/Kconfig b/sound/soc/dwc/Kconfig
index 71a58f7ac13a9..9aa73a5cf0b77 100644
--- a/sound/soc/dwc/Kconfig
+++ b/sound/soc/dwc/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for DesignWare"
+
 config SND_DESIGNWARE_I2S
 	tristate "Synopsys I2S Device Driver"
 	depends on HAVE_CLK
@@ -18,3 +20,4 @@ config SND_DESIGNWARE_PCM
 	 This functionality is specially suited for I2S devices that don't have
 	 DMA support.
 
+endmenu
-- 
2.43.0

[PATCH 06/37] ASoC: fsl: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/fsl/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig
index c4cf3cff58ded..1dad94bd1ff65 100644
--- a/sound/soc/fsl/Kconfig
+++ b/sound/soc/fsl/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
-menu "SoC Audio for Freescale CPUs"
+menu "ASoC support for Freescale"
 
 comment "Common SoC Audio options for Freescale CPUs:"
 
-- 
2.43.0

[PATCH 07/37] ASoC: img: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/img/Kconfig | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/sound/soc/img/Kconfig b/sound/soc/img/Kconfig
index 568efa606ca4b..e2396e285a093 100644
--- a/sound/soc/img/Kconfig
+++ b/sound/soc/img/Kconfig
@@ -1,12 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0-only
-config SND_SOC_IMG
-	bool "Audio support for Imagination Technologies designs"
-	help
-	  Audio support for Imagination Technologies audio hardware
+menu "ASoC support for Imagination"
 
 config SND_SOC_IMG_I2S_IN
 	tristate "Imagination I2S Input Device Driver"
-	depends on SND_SOC_IMG
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for I2S in driver for
@@ -14,7 +10,6 @@ config SND_SOC_IMG_I2S_IN
 
 config SND_SOC_IMG_I2S_OUT
 	tristate "Imagination I2S Output Device Driver"
-	depends on SND_SOC_IMG
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for I2S out driver for
@@ -22,7 +17,6 @@ config SND_SOC_IMG_I2S_OUT
 
 config SND_SOC_IMG_PARALLEL_OUT
 	tristate "Imagination Parallel Output Device Driver"
-	depends on SND_SOC_IMG
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for parallel out driver for
@@ -30,7 +24,6 @@ config SND_SOC_IMG_PARALLEL_OUT
 
 config SND_SOC_IMG_SPDIF_IN
 	tristate "Imagination SPDIF Input Device Driver"
-	depends on SND_SOC_IMG
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for SPDIF input driver for
@@ -38,7 +31,6 @@ config SND_SOC_IMG_SPDIF_IN
 
 config SND_SOC_IMG_SPDIF_OUT
 	tristate "Imagination SPDIF Output Device Driver"
-	depends on SND_SOC_IMG
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for SPDIF out driver for
@@ -47,7 +39,8 @@ config SND_SOC_IMG_SPDIF_OUT
 
 config SND_SOC_IMG_PISTACHIO_INTERNAL_DAC
 	tristate "Support for Pistachio SoC Internal DAC Driver"
-	depends on SND_SOC_IMG
 	help
 	  Say Y or M if you want to add support for Pistachio internal DAC
 	  driver for Imagination Technologies Pistachio internal DAC device.
+
+endmenu
-- 
2.43.0

[PATCH 08/37] ASoC: mxs: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/mxs/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/mxs/Kconfig b/sound/soc/mxs/Kconfig
index 402ef1ee7a325..608ef6583a2a0 100644
--- a/sound/soc/mxs/Kconfig
+++ b/sound/soc/mxs/Kconfig
@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
 menuconfig SND_MXS_SOC
-	tristate "SoC Audio for Freescale MXS CPUs"
+	tristate "ASoC support for Freescale MXS"
 	depends on ARCH_MXS || COMPILE_TEST
 	depends on COMMON_CLK
 	select SND_SOC_GENERIC_DMAENGINE_PCM
-- 
2.43.0

[PATCH 09/37] ASoC: pxa: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/pxa/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/pxa/Kconfig b/sound/soc/pxa/Kconfig
index e05d6ce4c8fa2..321e4d3d61e64 100644
--- a/sound/soc/pxa/Kconfig
+++ b/sound/soc/pxa/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for PXA"
+
 config SND_PXA2XX_SOC
 	tristate "SoC Audio for the Intel PXA2xx chip"
 	depends on ARCH_PXA || COMPILE_TEST
@@ -52,3 +54,5 @@ config SND_PXA910_SOC
 	help
 	  Say Y if you want to add support for SoC audio on the
 	  Marvell PXA910 reference platform.
+
+endmenu
-- 
2.43.0

[PATCH 10/37] ASoC: sof: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/sof/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/sof/Kconfig b/sound/soc/sof/Kconfig
index 32ffd345e07fc..4879e7268bda7 100644
--- a/sound/soc/sof/Kconfig
+++ b/sound/soc/sof/Kconfig
@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
 menuconfig SND_SOC_SOF_TOPLEVEL
-	bool "Sound Open Firmware Support"
+	bool "ASoC support for Sound Open Firmware"
 	help
 	  This adds support for Sound Open Firmware (SOF). SOF is free and
 	  generic open source audio DSP firmware for multiple devices.
-- 
2.43.0

[PATCH 11/37] ASoC: sti: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/sti/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/sti/Kconfig b/sound/soc/sti/Kconfig
index f881da4b6aead..f99c2eedf93b3 100644
--- a/sound/soc/sti/Kconfig
+++ b/sound/soc/sti/Kconfig
@@ -3,7 +3,7 @@
 # STM SoC audio configuration
 #
 menuconfig SND_SOC_STI
-	tristate "SoC Audio support for STI System-On-Chip"
+	tristate "ASoC support for STI"
 	depends on SND_SOC
 	depends on ARCH_STI || COMPILE_TEST
 	select SND_SOC_GENERIC_DMAENGINE_PCM
-- 
2.43.0

[PATCH 12/37] ASoC: stm: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/stm/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/stm/Kconfig b/sound/soc/stm/Kconfig
index da1f7a16605b4..08f32d2e606ce 100644
--- a/sound/soc/stm/Kconfig
+++ b/sound/soc/stm/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
-menu "STMicroelectronics STM32 SOC audio support"
+menu "ASoC support for STMicroelectronics STM32"
 
 config SND_SOC_STM32_SAI
 	tristate "STM32 SAI interface (Serial Audio Interface) support"
-- 
2.43.0

[PATCH 13/37] ASoC: au1x: Standardize ASoC menu

[Kuninori Morimoto]

use "ASoC support for xxxx" menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/au1x/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/au1x/Kconfig b/sound/soc/au1x/Kconfig
index 8a78809e87543..01627fdf392ce 100644
--- a/sound/soc/au1x/Kconfig
+++ b/sound/soc/au1x/Kconfig
@@ -2,6 +2,8 @@
 ##
 ## Au1200/Au1550/Au1300 PSC + DBDMA
 ##
+menu "ASoC support for Au1x"
+
 config SND_SOC_AU1XPSC
 	tristate "SoC Audio for Au12xx/Au13xx/Au1550"
 	depends on MIPS_ALCHEMY
@@ -63,3 +65,5 @@ config SND_SOC_DB1200
 	  Select this option to enable audio (AC97 and I2S) on the
 	  Alchemy/AMD/RMI/NetLogic Db1200, Db1550 and Db1300 evaluation boards.
 	  If you need Db1300 touchscreen support, you definitely want to say Y.
+
+endmenu
-- 
2.43.0

[PATCH 14/37] ASoC: sdca: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/sdca/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sound/soc/sdca/Kconfig b/sound/soc/sdca/Kconfig
index ee20b9914aa1f..c514d2506a3a6 100644
--- a/sound/soc/sdca/Kconfig
+++ b/sound/soc/sdca/Kconfig
@@ -1,4 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for SoundWire (SDCA)"
 
 config SND_SOC_SDCA
 	tristate
@@ -9,3 +10,5 @@ config SND_SOC_SDCA
 
 config SND_SOC_SDCA_OPTIONAL
 	def_tristate SND_SOC_SDCA || !SND_SOC_SDCA
+
+endmenu
-- 
2.43.0

[PATCH 15/37] ASoC: sprd: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/sprd/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/sprd/Kconfig b/sound/soc/sprd/Kconfig
index 5e0ac82785721..acefcad85a1a2 100644
--- a/sound/soc/sprd/Kconfig
+++ b/sound/soc/sprd/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Spreadtrum"
+
 config SND_SOC_SPRD
 	tristate "SoC Audio for the Spreadtrum SoC chips"
 	depends on ARCH_SPRD || COMPILE_TEST
@@ -14,3 +16,5 @@ config SND_SOC_SPRD_MCDT
 	  Say y here to enable multi-channel data transfer support. It
 	  is used for sound stream transmission between audio subsystem
 	  and other AP/CP subsystem.
+
+endmenu
-- 
2.43.0

[PATCH 16/37] ASoC: apple: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/apple/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/apple/Kconfig b/sound/soc/apple/Kconfig
index 793f7782e0d72..1785c0f741ae5 100644
--- a/sound/soc/apple/Kconfig
+++ b/sound/soc/apple/Kconfig
@@ -1,3 +1,5 @@
+menu "ASoC support for Apple"
+
 config SND_SOC_APPLE_MCA
 	tristate "Apple Silicon MCA driver"
 	depends on ARCH_APPLE || COMPILE_TEST
@@ -6,3 +8,5 @@ config SND_SOC_APPLE_MCA
 	help
 	  This option enables an ASoC platform driver for MCA peripherals found
 	  on Apple Silicon SoCs.
+
+endmenu
-- 
2.43.0

[PATCH 17/37] ASoC: atmel: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/atmel/Kconfig | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/sound/soc/atmel/Kconfig b/sound/soc/atmel/Kconfig
index 5d59e00be8238..e1a68c9cca70a 100644
--- a/sound/soc/atmel/Kconfig
+++ b/sound/soc/atmel/Kconfig
@@ -1,13 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
-config SND_ATMEL_SOC
-	tristate "SoC Audio for the Atmel System-on-Chip"
+menu "ASoC support for Atmel"
 	depends on HAS_IOMEM
-	help
-	  Say Y or M if you want to add support for codecs attached to
-	  the ATMEL SSC interface. You will also need
-	  to select the audio interfaces to support below.
-
-if SND_ATMEL_SOC
 
 config SND_ATMEL_SOC_PDC
 	bool
@@ -176,4 +169,4 @@ config SND_MCHP_SOC_PDMC
 	  2 data lines. The signal path includes an audio grade programmable
 	  decimation filter and outputs 24-bit audio words.
 
-endif
+endmenu
-- 
2.43.0

[PATCH 18/37] ASoC: intel: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/intel/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
index 2db494b0e3cff..b30fff51e7392 100644
--- a/sound/soc/intel/Kconfig
+++ b/sound/soc/intel/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Intel"
+
 config SND_SOC_INTEL_SST_TOPLEVEL
 	bool "Intel ASoC SST drivers"
 	default y
@@ -115,3 +117,5 @@ source "sound/soc/intel/avs/boards/Kconfig"
 
 # ASoC codec drivers
 source "sound/soc/intel/boards/Kconfig"
+
+endmenu
-- 
2.43.0

[PATCH 19/37] ASoC: spear: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/spear/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/spear/Kconfig b/sound/soc/spear/Kconfig
index 1b053de1ae7cb..77316396d6da9 100644
--- a/sound/soc/spear/Kconfig
+++ b/sound/soc/spear/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for spear"
+
 config SND_SPEAR_SOC
 	tristate
 	select SND_SOC_GENERIC_DMAENGINE_PCM
@@ -8,3 +10,5 @@ config SND_SPEAR_SPDIF_OUT
 
 config SND_SPEAR_SPDIF_IN
 	tristate
+
+endmenu
-- 
2.43.0

[PATCH 20/37] ASoC: sunxi: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/sunxi/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/sunxi/Kconfig b/sound/soc/sunxi/Kconfig
index 1f18f016acbb8..00ca48ebd7b9b 100644
--- a/sound/soc/sunxi/Kconfig
+++ b/sound/soc/sunxi/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
-menu "Allwinner SoC Audio support"
+menu "ASoC support for Allwinner"
 	depends on ARCH_SUNXI || COMPILE_TEST
 
 config SND_SUN4I_CODEC
-- 
2.43.0

Re: [PATCH 20/37] ASoC: sunxi: Standardize ASoC menu

[Chen-Yu Tsai]

(Dropped non-maintainers from CC list)

On Tue, Jun 10, 2025 at 12:22 PM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
>
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
>
> Let's standardize ASoC menu
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Acked-by: Chen-Yu Tsai <wens@csie.org>

[PATCH 21/37] ASoC: tegra: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/tegra/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/tegra/Kconfig b/sound/soc/tegra/Kconfig
index 2463c22e9cf6d..a7c687123b390 100644
--- a/sound/soc/tegra/Kconfig
+++ b/sound/soc/tegra/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Tegra"
+
 config SND_SOC_TEGRA
 	tristate "SoC Audio for the Tegra System-on-Chip"
 	depends on (ARCH_TEGRA && TEGRA20_APB_DMA) || COMPILE_TEST
@@ -293,3 +295,5 @@ config SND_SOC_TEGRA_SGTL5000
 	  Colibri T30.
 
 endif
+
+endmenu
-- 
2.43.0

[PATCH 22/37] ASoC: ux500: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/ux500/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/ux500/Kconfig b/sound/soc/ux500/Kconfig
index 34b2438b52d36..1df73c5936e93 100644
--- a/sound/soc/ux500/Kconfig
+++ b/sound/soc/ux500/Kconfig
@@ -3,7 +3,7 @@
 # Ux500 SoC audio configuration
 #
 menuconfig SND_SOC_UX500
-	tristate "SoC Audio support for Ux500 platform"
+	tristate "ASoC support for Ux500"
 	depends on SND_SOC
 	depends on MFD_DB8500_PRCMU
 	help
-- 
2.43.0

[PATCH 23/37] ASoC: cirrus: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/cirrus/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sound/soc/cirrus/Kconfig b/sound/soc/cirrus/Kconfig
index 97def4e53fbc7..6c68c54baaf9a 100644
--- a/sound/soc/cirrus/Kconfig
+++ b/sound/soc/cirrus/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Cirrus"
+
 config SND_EP93XX_SOC
 	tristate "SoC Audio support for the Cirrus Logic EP93xx series"
 	depends on ARCH_EP93XX || COMPILE_TEST
@@ -31,3 +33,4 @@ config SND_EP93XX_SOC_I2S_WATCHDOG
 
 endif # if SND_EP93XX_SOC_I2S
 
+endmenu
-- 
2.43.0

Re: [PATCH 23/37] ASoC: cirrus: Standardize ASoC menu

[Alexander Sverdlin]

On Tue, 2025-06-10 at 04:22 +0000, Kuninori Morimoto wrote:
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
> 
> Let's standardize ASoC menu
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

A nitpick below, but other than that:

Reviewed-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>

> ---
>  sound/soc/cirrus/Kconfig | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/sound/soc/cirrus/Kconfig b/sound/soc/cirrus/Kconfig
> index 97def4e53fbc7..6c68c54baaf9a 100644
> --- a/sound/soc/cirrus/Kconfig
> +++ b/sound/soc/cirrus/Kconfig
> @@ -1,4 +1,6 @@
>  # SPDX-License-Identifier: GPL-2.0-only
> +menu "ASoC support for Cirrus"
                          ^^^^^^
It could have been at least "Cirrus Logic".

> +
>  config SND_EP93XX_SOC
>  	tristate "SoC Audio support for the Cirrus Logic EP93xx series"
>  	depends on ARCH_EP93XX || COMPILE_TEST
> @@ -31,3 +33,4 @@ config SND_EP93XX_SOC_I2S_WATCHDOG
>  
>  endif # if SND_EP93XX_SOC_I2S
>  
> +endmenu

-- 
Alexander Sverdlin.

[PATCH 24/37] ASoC: google: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/google/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sound/soc/google/Kconfig b/sound/soc/google/Kconfig
index 7603782fb0602..b174d53e0f12e 100644
--- a/sound/soc/google/Kconfig
+++ b/sound/soc/google/Kconfig
@@ -1,6 +1,9 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Google"
 
 config SND_SOC_CHV3_I2S
 	tristate "Google Chameleon v3 I2S device"
 	help
 	  Enable support for the Google Chameleon v3 I2S device.
+
+endmenu
-- 
2.43.0

[PATCH 25/37] ASoC: jz4740: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/jz4740/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/jz4740/Kconfig b/sound/soc/jz4740/Kconfig
index dd3b4507fbe6d..7b24df5774865 100644
--- a/sound/soc/jz4740/Kconfig
+++ b/sound/soc/jz4740/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for JZ4740"
+
 config SND_JZ4740_SOC_I2S
 	tristate "SoC Audio (I2S protocol) for Ingenic JZ4740 SoC"
 	depends on MIPS || COMPILE_TEST
@@ -8,3 +10,5 @@ config SND_JZ4740_SOC_I2S
 	help
 	  Say Y if you want to use I2S protocol and I2S codec on Ingenic JZ4740
 	  based boards.
+
+endmenu
-- 
2.43.0

[PATCH 26/37] ASoC: xilinx: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/xilinx/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/xilinx/Kconfig b/sound/soc/xilinx/Kconfig
index 5bd2730aab76f..27fa42361bb90 100644
--- a/sound/soc/xilinx/Kconfig
+++ b/sound/soc/xilinx/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Xilinx"
+
 config SND_SOC_XILINX_I2S
 	tristate "Audio support for the Xilinx I2S"
 	help
@@ -21,3 +23,5 @@ config SND_SOC_XILINX_SPDIF
 	  Select this option to enable Xilinx SPDIF Audio.
 	  This provides playback and capture of SPDIF audio in
 	  AES format.
+
+endmenu
-- 
2.43.0

[PATCH 27/37] ASoC: xtensa: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/xtensa/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/xtensa/Kconfig b/sound/soc/xtensa/Kconfig
index 74b778f186cef..83db5e60cb02c 100644
--- a/sound/soc/xtensa/Kconfig
+++ b/sound/soc/xtensa/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Xtensa"
+
 config SND_SOC_XTFPGA_I2S
 	tristate "XTFPGA I2S master"
 	select REGMAP_MMIO
@@ -6,3 +8,5 @@ config SND_SOC_XTFPGA_I2S
 	  Say Y or M if you want to add support for codecs attached to the
 	  I2S interface on XTFPGA daughter board. You will also need to select
 	  the drivers for the rest of XTFPGA audio subsystem.
+
+endmenu
-- 
2.43.0

[PATCH 28/37] ASoC: renesas: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/renesas/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/renesas/Kconfig b/sound/soc/renesas/Kconfig
index dabf02a955ca1..2473880b5caef 100644
--- a/sound/soc/renesas/Kconfig
+++ b/sound/soc/renesas/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0
-menu "SoC Audio support for Renesas SoCs"
+menu "ASoC support for Renesas"
 	depends on SUPERH || ARCH_RENESAS || COMPILE_TEST
 
 config SND_SOC_PCM_SH7760
-- 
2.43.0

[PATCH 29/37] ASoC: generic: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/Kconfig         | 4 +---
 sound/soc/generic/Kconfig | 4 ++++
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/sound/soc/Kconfig b/sound/soc/Kconfig
index 1b983c7006f18..b6a1f6fa26a99 100644
--- a/sound/soc/Kconfig
+++ b/sound/soc/Kconfig
@@ -111,6 +111,7 @@ source "sound/soc/bcm/Kconfig"
 source "sound/soc/cirrus/Kconfig"
 source "sound/soc/dwc/Kconfig"
 source "sound/soc/fsl/Kconfig"
+source "sound/soc/generic/Kconfig"
 source "sound/soc/google/Kconfig"
 source "sound/soc/hisilicon/Kconfig"
 source "sound/soc/jz4740/Kconfig"
@@ -146,8 +147,5 @@ source "sound/soc/codecs/Kconfig"
 
 source "sound/soc/sdw_utils/Kconfig"
 
-# generic frame-work
-source "sound/soc/generic/Kconfig"
-
 endif	# SND_SOC
 
diff --git a/sound/soc/generic/Kconfig b/sound/soc/generic/Kconfig
index b6df4e26bc4ad..2acc9fda019c5 100644
--- a/sound/soc/generic/Kconfig
+++ b/sound/soc/generic/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Generic drivers"
+
 config SND_SIMPLE_CARD_UTILS
 	tristate
 
@@ -37,3 +39,5 @@ config SND_TEST_COMPONENT
 	depends on OF
 	help
 	  This option enables test component sound driver support.
+
+endmenu
-- 
2.43.0

Re: [PATCH 29/37] ASoC: generic: Standardize ASoC menu

[Geert Uytterhoeven]

Hi Morimoto-san,

On Tue, 10 Jun 2025 at 06:23, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
>
> Let's standardize ASoC menu
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thanks for your patch, which is now commit acc84d15e45393fb ("ASoC:
generic: Standardize ASoC menu") upstream.

> --- a/sound/soc/Kconfig
> +++ b/sound/soc/Kconfig
> @@ -111,6 +111,7 @@ source "sound/soc/bcm/Kconfig"
>  source "sound/soc/cirrus/Kconfig"
>  source "sound/soc/dwc/Kconfig"
>  source "sound/soc/fsl/Kconfig"
> +source "sound/soc/generic/Kconfig"

This sort of hides the generic menu in between the Freescale and Google
hardware-specific menus in the "# All the supported SoCs" section.
What about moving it either before or after "# All the supported SoCs"?

>  source "sound/soc/google/Kconfig"
>  source "sound/soc/hisilicon/Kconfig"
>  source "sound/soc/jz4740/Kconfig"
> @@ -146,8 +147,5 @@ source "sound/soc/codecs/Kconfig"
>
>  source "sound/soc/sdw_utils/Kconfig"
>
> -# generic frame-work
> -source "sound/soc/generic/Kconfig"
> -
>  endif  # SND_SOC
>

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

Re: [PATCH 29/37] ASoC: generic: Standardize ASoC menu

[Kuninori Morimoto]

Hi Geert, Mark

> > --- a/sound/soc/Kconfig
> > +++ b/sound/soc/Kconfig
> > @@ -111,6 +111,7 @@ source "sound/soc/bcm/Kconfig"
> >  source "sound/soc/cirrus/Kconfig"
> >  source "sound/soc/dwc/Kconfig"
> >  source "sound/soc/fsl/Kconfig"
> > +source "sound/soc/generic/Kconfig"
> 
> This sort of hides the generic menu in between the Freescale and Google
> hardware-specific menus in the "# All the supported SoCs" section.
> What about moving it either before or after "# All the supported SoCs"?

Thank you for your reporting.
OK, thanks. Indeed it is a little bit strange. Will fix if merge-windows was
closed.

Thank you for your help !!

Best regards
---
Kuninori Morimoto

[PATCH 30/37] ASoC: kirkwood: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/kirkwood/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sound/soc/kirkwood/Kconfig b/sound/soc/kirkwood/Kconfig
index 5d8a86b26fa2f..f2ec6ba020bff 100644
--- a/sound/soc/kirkwood/Kconfig
+++ b/sound/soc/kirkwood/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Kirkwood"
+
 config SND_KIRKWOOD_SOC
 	tristate "SoC Audio for the Marvell Kirkwood and Dove chips"
 	depends on ARCH_DOVE || ARCH_MVEBU || COMPILE_TEST
@@ -16,3 +18,4 @@ config SND_KIRKWOOD_SOC_ARMADA370_DB
 	  Say Y if you want to add support for SoC audio on
 	  the Armada 370 Development Board.
 
+endmenu
-- 
2.43.0

[PATCH 31/37] ASoC: loongson: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/loongson/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/loongson/Kconfig b/sound/soc/loongson/Kconfig
index 1a3c28816e7ad..d52bc2a6e0780 100644
--- a/sound/soc/loongson/Kconfig
+++ b/sound/soc/loongson/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0
-menu "SoC Audio for Loongson CPUs"
+menu "ASoC support for Loongson"
 
 config SND_SOC_LOONGSON_CARD
 	tristate "Loongson Sound Card Driver"
-- 
2.43.0

[PATCH 32/37] ASoC: mediatek: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/mediatek/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/mediatek/Kconfig b/sound/soc/mediatek/Kconfig
index 90e367586493d..9dfaf3d1dde66 100644
--- a/sound/soc/mediatek/Kconfig
+++ b/sound/soc/mediatek/Kconfig
@@ -1,4 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Mediatek"
+
 config SND_SOC_MEDIATEK
 	tristate
 	select REGMAP_MMIO
@@ -319,3 +321,5 @@ config SND_SOC_MT8365_MT6357
 	  boards with the MT6357 PMIC codec.
 	  Select Y if you have such device.
 	  If unsure select "N".
+
+endmenu
-- 
2.43.0

Re: [PATCH 32/37] ASoC: mediatek: Standardize ASoC menu

[Matthias Brugger]

On 10/6/25 06:23, Kuninori Morimoto wrote:
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
> 
> Let's standardize ASoC menu
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

I wonder if we really need to split this up into patch per vendor if we 
could do the whole thing in one go. Anyway for the MediaTek part:

Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

> ---
>   sound/soc/mediatek/Kconfig | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/sound/soc/mediatek/Kconfig b/sound/soc/mediatek/Kconfig
> index 90e367586493d..9dfaf3d1dde66 100644
> --- a/sound/soc/mediatek/Kconfig
> +++ b/sound/soc/mediatek/Kconfig
> @@ -1,4 +1,6 @@
>   # SPDX-License-Identifier: GPL-2.0-only
> +menu "ASoC support for Mediatek"
> +
>   config SND_SOC_MEDIATEK
>   	tristate
>   	select REGMAP_MMIO
> @@ -319,3 +321,5 @@ config SND_SOC_MT8365_MT6357
>   	  boards with the MT6357 PMIC codec.
>   	  Select Y if you have such device.
>   	  If unsure select "N".
> +
> +endmenu

Re: [PATCH 32/37] ASoC: mediatek: Standardize ASoC menu

[Kuninori Morimoto]

Hi Matthias

> > Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> > So, some venders are using menu style, some venders are listed each drivers
> > on top page, etc. It is difficult to find target vender and/or drivers
> > because it is very random.
> > 
> > Let's standardize ASoC menu
> > 
> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> 
> I wonder if we really need to split this up into patch per vendor if we 
> could do the whole thing in one go. Anyway for the MediaTek part:

I believe it makes backport easy for each vender.

Thank you for your help !!

Best regards
---
Kuninori Morimoto

[PATCH 33/37] ASoC: rockchip: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/rockchip/Kconfig | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/sound/soc/rockchip/Kconfig b/sound/soc/rockchip/Kconfig
index a08544827b2a3..b39b32d7bbc54 100644
--- a/sound/soc/rockchip/Kconfig
+++ b/sound/soc/rockchip/Kconfig
@@ -1,15 +1,10 @@
 # SPDX-License-Identifier: GPL-2.0-only
-config SND_SOC_ROCKCHIP
-	tristate "ASoC support for Rockchip"
+menu "ASoC support for Rockchip"
 	depends on COMPILE_TEST || ARCH_ROCKCHIP
-	help
-	  Say Y or M if you want to add support for codecs attached to
-	  the Rockchip SoCs' Audio interfaces. You will also need to
-	  select the audio interfaces to support below.
+	depends on HAVE_CLK
 
 config SND_SOC_ROCKCHIP_I2S
 	tristate "Rockchip I2S Device Driver"
-	depends on HAVE_CLK && SND_SOC_ROCKCHIP
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for I2S driver for
@@ -18,7 +13,6 @@ config SND_SOC_ROCKCHIP_I2S
 
 config SND_SOC_ROCKCHIP_I2S_TDM
 	tristate "Rockchip I2S/TDM Device Driver"
-	depends on HAVE_CLK && SND_SOC_ROCKCHIP
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for the I2S/TDM driver for
@@ -29,7 +23,6 @@ config SND_SOC_ROCKCHIP_I2S_TDM
 
 config SND_SOC_ROCKCHIP_PDM
 	tristate "Rockchip PDM Controller Driver"
-	depends on HAVE_CLK && SND_SOC_ROCKCHIP
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	select RATIONAL
 	help
@@ -39,7 +32,6 @@ config SND_SOC_ROCKCHIP_PDM
 
 config SND_SOC_ROCKCHIP_SAI
 	tristate "Rockchip SAI Controller Driver"
-	depends on HAVE_CLK && SND_SOC_ROCKCHIP
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for the Rockchip Serial Audio
@@ -49,7 +41,6 @@ config SND_SOC_ROCKCHIP_SAI
 
 config SND_SOC_ROCKCHIP_SPDIF
 	tristate "Rockchip SPDIF Device Driver"
-	depends on HAVE_CLK && SND_SOC_ROCKCHIP
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for SPDIF driver for
@@ -57,7 +48,7 @@ config SND_SOC_ROCKCHIP_SPDIF
 
 config SND_SOC_ROCKCHIP_MAX98090
 	tristate "ASoC support for Rockchip boards using a MAX98090 codec"
-	depends on SND_SOC_ROCKCHIP && I2C && GPIOLIB && HAVE_CLK
+	depends on I2C && GPIOLIB
 	select SND_SOC_ROCKCHIP_I2S
 	select SND_SOC_MAX98090
 	select SND_SOC_TS3A227E
@@ -68,7 +59,7 @@ config SND_SOC_ROCKCHIP_MAX98090
 
 config SND_SOC_ROCKCHIP_RT5645
 	tristate "ASoC support for Rockchip boards using a RT5645/RT5650 codec"
-	depends on SND_SOC_ROCKCHIP && I2C && GPIOLIB && HAVE_CLK
+	depends on I2C && GPIOLIB
 	select SND_SOC_ROCKCHIP_I2S
 	select SND_SOC_RT5645
 	help
@@ -77,7 +68,7 @@ config SND_SOC_ROCKCHIP_RT5645
 
 config SND_SOC_RK3288_HDMI_ANALOG
 	tristate "ASoC support multiple codecs for Rockchip RK3288 boards"
-	depends on SND_SOC_ROCKCHIP && I2C && GPIOLIB && HAVE_CLK
+	depends on I2C && GPIOLIB
 	select SND_SOC_ROCKCHIP_I2S
 	select SND_SOC_HDMI_CODEC
 	select SND_SOC_ES8328_I2C
@@ -89,7 +80,7 @@ config SND_SOC_RK3288_HDMI_ANALOG
 
 config SND_SOC_RK3399_GRU_SOUND
 	tristate "ASoC support multiple codecs for Rockchip RK3399 GRU boards"
-	depends on SND_SOC_ROCKCHIP && I2C && GPIOLIB && HAVE_CLK && SPI
+	depends on I2C && GPIOLIB && SPI
 	select SND_SOC_ROCKCHIP_I2S
 	select SND_SOC_MAX98357A
 	select SND_SOC_RT5514
@@ -100,3 +91,5 @@ config SND_SOC_RK3399_GRU_SOUND
 	help
 	  Say Y or M here if you want to add support multiple codecs for SoC
 	  audio on Rockchip RK3399 GRU boards.
+
+endmenu
-- 
2.43.0

[PATCH 34/37] ASoC: starfive: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/starfive/Kconfig | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/sound/soc/starfive/Kconfig b/sound/soc/starfive/Kconfig
index 279ac5c1d309b..50f515dea620c 100644
--- a/sound/soc/starfive/Kconfig
+++ b/sound/soc/starfive/Kconfig
@@ -1,15 +1,10 @@
 # SPDX-License-Identifier: GPL-2.0-only
-config SND_SOC_STARFIVE
-	tristate "Audio support for StarFive SoC"
+menu "ASoC support for StarFive"
 	depends on COMPILE_TEST || ARCH_STARFIVE
-	help
-	  Say Y or M if you want to add support for codecs attached to
-	  the Starfive SoCs' Audio interfaces. You will also need to
-	  select the audio interfaces to support below.
+	depends on HAVE_CLK
 
 config SND_SOC_JH7110_PWMDAC
 	tristate "JH7110 PWM-DAC device driver"
-	depends on HAVE_CLK && SND_SOC_STARFIVE
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	select SND_SOC_SPDIF
 	help
@@ -18,7 +13,8 @@ config SND_SOC_JH7110_PWMDAC
 
 config SND_SOC_JH7110_TDM
 	tristate "JH7110 TDM device driver"
-	depends on HAVE_CLK && SND_SOC_STARFIVE
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Say Y or M if you want to add support for StarFive TDM driver.
+
+endmenu
-- 
2.43.0

[PATCH 35/37] ASoC: uniphier: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/uniphier/Kconfig | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/sound/soc/uniphier/Kconfig b/sound/soc/uniphier/Kconfig
index ddfa6424c656b..45140d49c6adf 100644
--- a/sound/soc/uniphier/Kconfig
+++ b/sound/soc/uniphier/Kconfig
@@ -1,18 +1,11 @@
 # SPDX-License-Identifier: GPL-2.0
-config SND_SOC_UNIPHIER
-	tristate "ASoC support for UniPhier"
+menu "ASoC support for UniPhier"
 	depends on (ARCH_UNIPHIER || COMPILE_TEST)
-	help
-	  Say Y or M if you want to add support for the Socionext
-	  UniPhier SoC audio interfaces. You will also need to select the
-	  audio interfaces to support below.
-	  If unsure select "N".
 
 config SND_SOC_UNIPHIER_AIO
 	tristate "UniPhier AIO DAI Driver"
 	select REGMAP_MMIO
 	select SND_SOC_COMPRESS
-	depends on SND_SOC_UNIPHIER
 	help
 	  This adds ASoC driver support for Socionext UniPhier
 	  'AIO' Audio Input/Output subsystem.
@@ -21,7 +14,6 @@ config SND_SOC_UNIPHIER_AIO
 
 config SND_SOC_UNIPHIER_LD11
 	tristate "UniPhier LD11/LD20 Device Driver"
-	depends on SND_SOC_UNIPHIER
 	select SND_SOC_UNIPHIER_AIO
 	help
 	  This adds ASoC driver for Socionext UniPhier LD11/LD20
@@ -31,7 +23,6 @@ config SND_SOC_UNIPHIER_LD11
 
 config SND_SOC_UNIPHIER_PXS2
 	tristate "UniPhier PXs2 Device Driver"
-	depends on SND_SOC_UNIPHIER
 	select SND_SOC_UNIPHIER_AIO
 	help
 	  This adds ASoC driver for Socionext UniPhier PXs2
@@ -41,10 +32,11 @@ config SND_SOC_UNIPHIER_PXS2
 
 config SND_SOC_UNIPHIER_EVEA_CODEC
 	tristate "UniPhier SoC internal audio codec"
-	depends on SND_SOC_UNIPHIER
 	select REGMAP_MMIO
 	help
 	  This adds Codec driver for Socionext UniPhier LD11/20 SoC
 	  internal DAC. This driver supports Line In / Out and HeadPhone.
 	  Select Y if you use such device.
 	  If unsure select "N".
+
+endmenu
-- 
2.43.0

Re: [PATCH 35/37] ASoC: uniphier: Standardize ASoC menu

[Masami Hiramatsu]

On Tue, 10 Jun 2025 04:23:32 +0000
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote:

> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
> 
> Let's standardize ASoC menu
> 

Looks good to me.

Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>

Thanks,

> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>  sound/soc/uniphier/Kconfig | 14 +++-----------
>  1 file changed, 3 insertions(+), 11 deletions(-)
> 
> diff --git a/sound/soc/uniphier/Kconfig b/sound/soc/uniphier/Kconfig
> index ddfa6424c656b..45140d49c6adf 100644
> --- a/sound/soc/uniphier/Kconfig
> +++ b/sound/soc/uniphier/Kconfig
> @@ -1,18 +1,11 @@
>  # SPDX-License-Identifier: GPL-2.0
> -config SND_SOC_UNIPHIER
> -	tristate "ASoC support for UniPhier"
> +menu "ASoC support for UniPhier"
>  	depends on (ARCH_UNIPHIER || COMPILE_TEST)
> -	help
> -	  Say Y or M if you want to add support for the Socionext
> -	  UniPhier SoC audio interfaces. You will also need to select the
> -	  audio interfaces to support below.
> -	  If unsure select "N".
>  
>  config SND_SOC_UNIPHIER_AIO
>  	tristate "UniPhier AIO DAI Driver"
>  	select REGMAP_MMIO
>  	select SND_SOC_COMPRESS
> -	depends on SND_SOC_UNIPHIER
>  	help
>  	  This adds ASoC driver support for Socionext UniPhier
>  	  'AIO' Audio Input/Output subsystem.
> @@ -21,7 +14,6 @@ config SND_SOC_UNIPHIER_AIO
>  
>  config SND_SOC_UNIPHIER_LD11
>  	tristate "UniPhier LD11/LD20 Device Driver"
> -	depends on SND_SOC_UNIPHIER
>  	select SND_SOC_UNIPHIER_AIO
>  	help
>  	  This adds ASoC driver for Socionext UniPhier LD11/LD20
> @@ -31,7 +23,6 @@ config SND_SOC_UNIPHIER_LD11
>  
>  config SND_SOC_UNIPHIER_PXS2
>  	tristate "UniPhier PXs2 Device Driver"
> -	depends on SND_SOC_UNIPHIER
>  	select SND_SOC_UNIPHIER_AIO
>  	help
>  	  This adds ASoC driver for Socionext UniPhier PXs2
> @@ -41,10 +32,11 @@ config SND_SOC_UNIPHIER_PXS2
>  
>  config SND_SOC_UNIPHIER_EVEA_CODEC
>  	tristate "UniPhier SoC internal audio codec"
> -	depends on SND_SOC_UNIPHIER
>  	select REGMAP_MMIO
>  	help
>  	  This adds Codec driver for Socionext UniPhier LD11/20 SoC
>  	  internal DAC. This driver supports Line In / Out and HeadPhone.
>  	  Select Y if you use such device.
>  	  If unsure select "N".
> +
> +endmenu
> -- 
> 2.43.0
> 


-- 
Masami Hiramatsu (Google) <mhiramat@kernel.org>

[PATCH 36/37] ASoC: hisilicon: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/hisilicon/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sound/soc/hisilicon/Kconfig b/sound/soc/hisilicon/Kconfig
index df8fbd8bb4b03..5acb6c35a4970 100644
--- a/sound/soc/hisilicon/Kconfig
+++ b/sound/soc/hisilicon/Kconfig
@@ -1,6 +1,10 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for Hisilicon"
+
 config SND_I2S_HI6210_I2S
 	tristate "Hisilicon I2S controller"
 	select SND_SOC_GENERIC_DMAENGINE_PCM
 	help
 	  Hisilicon I2S
+
+endmenu
-- 
2.43.0

[PATCH 37/37] ASoC: sdw_utils: Standardize ASoC menu

[Kuninori Morimoto]

Current Kconfig menu at [ALSA for SoC audio support] has no rules.
So, some venders are using menu style, some venders are listed each drivers
on top page, etc. It is difficult to find target vender and/or drivers
because it is very random.

Let's standardize ASoC menu

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/Kconfig           | 3 +--
 sound/soc/sdw_utils/Kconfig | 4 ++++
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/sound/soc/Kconfig b/sound/soc/Kconfig
index b6a1f6fa26a99..d2934022344d9 100644
--- a/sound/soc/Kconfig
+++ b/sound/soc/Kconfig
@@ -128,6 +128,7 @@ source "sound/soc/renesas/Kconfig"
 source "sound/soc/rockchip/Kconfig"
 source "sound/soc/samsung/Kconfig"
 source "sound/soc/sdca/Kconfig"
+source "sound/soc/sdw_utils/Kconfig"
 source "sound/soc/sof/Kconfig"
 source "sound/soc/spear/Kconfig"
 source "sound/soc/sprd/Kconfig"
@@ -145,7 +146,5 @@ source "sound/soc/xtensa/Kconfig"
 # Supported codecs
 source "sound/soc/codecs/Kconfig"
 
-source "sound/soc/sdw_utils/Kconfig"
-
 endif	# SND_SOC
 
diff --git a/sound/soc/sdw_utils/Kconfig b/sound/soc/sdw_utils/Kconfig
index d915083c38895..a3e0f660fe2ec 100644
--- a/sound/soc/sdw_utils/Kconfig
+++ b/sound/soc/sdw_utils/Kconfig
@@ -1,6 +1,10 @@
 # SPDX-License-Identifier: GPL-2.0-only
+menu "ASoC support for SoundWire (Utils)"
+
 config SND_SOC_SDW_UTILS
 	tristate
 	help
 	  This option enables to use SoundWire common helper functions and
 	  SoundWire codec helper functions in machine driver.
+
+endmenu
-- 
2.43.0

Re: [PATCH 37/37] ASoC: sdw_utils: Standardize ASoC menu

[Péter Ujfalusi]

On 10/06/2025 07:23, Kuninori Morimoto wrote:
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.
> 
> Let's standardize ASoC menu
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>  sound/soc/Kconfig           | 3 +--
>  sound/soc/sdw_utils/Kconfig | 4 ++++
>  2 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/sound/soc/Kconfig b/sound/soc/Kconfig
> index b6a1f6fa26a99..d2934022344d9 100644
> --- a/sound/soc/Kconfig
> +++ b/sound/soc/Kconfig
> @@ -128,6 +128,7 @@ source "sound/soc/renesas/Kconfig"
>  source "sound/soc/rockchip/Kconfig"
>  source "sound/soc/samsung/Kconfig"
>  source "sound/soc/sdca/Kconfig"
> +source "sound/soc/sdw_utils/Kconfig"
>  source "sound/soc/sof/Kconfig"
>  source "sound/soc/spear/Kconfig"
>  source "sound/soc/sprd/Kconfig"
> @@ -145,7 +146,5 @@ source "sound/soc/xtensa/Kconfig"
>  # Supported codecs
>  source "sound/soc/codecs/Kconfig"
>  
> -source "sound/soc/sdw_utils/Kconfig"
> -
>  endif	# SND_SOC
>  
> diff --git a/sound/soc/sdw_utils/Kconfig b/sound/soc/sdw_utils/Kconfig
> index d915083c38895..a3e0f660fe2ec 100644
> --- a/sound/soc/sdw_utils/Kconfig
> +++ b/sound/soc/sdw_utils/Kconfig
> @@ -1,6 +1,10 @@
>  # SPDX-License-Identifier: GPL-2.0-only
> +menu "ASoC support for SoundWire (Utils)"
> +
>  config SND_SOC_SDW_UTILS
>  	tristate
>  	help
>  	  This option enables to use SoundWire common helper functions and
>  	  SoundWire codec helper functions in machine driver.
> +
> +endmenu

There is no reason to add a menu entry for a single option which does
not even have prompt.


-- 
Péter

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Péter Ujfalusi]

Hi Morimoto-san,

(reduced To list to core maintainers)
On 10/06/2025 03:43, Kuninori Morimoto wrote:
> 
> Hi Mark
> 
> Current Kconfig menu at [ALSA for SoC audio support] has no rules.
> So, some venders are using menu style, some venders are listed each drivers
> on top page, etc. It is difficult to find target vender and/or drivers
> because it is very random.

I think the effort is good, however I think this still needs some work.

>     
> Let's standardize ASoC menu, like below
>     
> 	ASoC support for Analog Devices  --->
> 	ASoC support for AMD  --->
> 	ASoC support for Apple  --->
> 	ASoC support for Atmel  --->
> 	ASoC support for Au1x  ----
> 	ASoC support for Broadcom  --->
> 	ASoC support for Cirrus  --->
> 	ASoC support for DesignWare  --->
> 	ASoC support for Freescale  --->
> 	ASoC support for Google  --->
> 	ASoC support for Hisilicon  --->
> 	...

This is how the menu looks like after applying this series on x64:

--- ALSA for SoC audio support
      ASoC support for Analog Devices  --->
      ASoC support for AMD  --->
      ASoC support for Apple  ----
      ASoC support for Atmel  --->
      ASoC support for Au1x  ----
      ASoC support for Broadcom  --->
      ASoC support for Cirrus  ----
      ASoC support for DesignWare  --->
      ASoC support for Freescale  --->
      ASoC support for Generic drivers  --->
      ASoC support for Google  --->
      ASoC support for Hisilicon  --->
      ASoC support for JZ4740  ----
      ASoC support for Kirkwood  ----
      ASoC support for Loongson  ----
      ASoC support for Imagination  --->
      ASoC support for Intel  --->
      ASoC support for Mediatek  --->
      ASoC support for PXA  ----
      ASoC support for SoundWire (SDCA)  ----
      ASoC support for SoundWire (Utils)  ----
[*]   ASoC support for Sound Open Firmware  --->
      ASoC support for spear  ----
      ASoC support for Spreadtrum  ----
      ASoC support for STMicroelectronics STM32  ----
      ASoC support for Tegra  ----
      ASoC support for Xilinx  --->
      ASoC support for Xtensa  --->
      CODEC drivers  --->

We have quite menu empty sub-menus, partially because they would include
something which would not be select-able in this arch (Apple, Au1x, etc)
and things that does not even have select-able options (SoundWire things
for example).

I personally would prefer to keep the 'Sound Open Firmware Support' as
it was, probably moved above the CODEC drivers. SOF is more like a small
subsystem within ASoC and the stack supports various vendors (Intel,
NXP, AMD, Mediatek atm).

The other awkward thing with this change (which was awkward before, but
going to be double awkwarded) is that the Intel Machine drivers will be
quite detached from SOF, not visible of hinted that they are now under
"ASoC support for Intel"

I guess one would figure this out eventually if needed, it is juts a bit
odd.
Note: I'm not sure where you can select AMD/NXP/MTK machine drivers,
they might be of a distance as well.

The other thing which made me wonder is the 'ASoC support for' repeated,
would not be simpler:

--- ALSA for SoC audio support
      Analog Devices  --->
      AMD  --->
      Apple  ----
...

The title already tells that this menu is for ASoC.

-- 
Péter

> 
> Kuninori Morimoto (37):
>   ASoC: ti: Standardize ASoC menu
>   ASoC: adi: Standardize ASoC menu
>   ASoC: amd: Standardize ASoC menu
>   ASoC: bcm: Standardize ASoC menu
>   ASoC: dwc: Standardize ASoC menu
>   ASoC: fsl: Standardize ASoC menu
>   ASoC: img: Standardize ASoC menu
>   ASoC: mxs: Standardize ASoC menu
>   ASoC: pxa: Standardize ASoC menu
>   ASoC: sof: Standardize ASoC menu
>   ASoC: sti: Standardize ASoC menu
>   ASoC: stm: Standardize ASoC menu
>   ASoC: au1x: Standardize ASoC menu
>   ASoC: sdca: Standardize ASoC menu
>   ASoC: sprd: Standardize ASoC menu
>   ASoC: apple: Standardize ASoC menu
>   ASoC: atmel: Standardize ASoC menu
>   ASoC: intel: Standardize ASoC menu
>   ASoC: spear: Standardize ASoC menu
>   ASoC: sunxi: Standardize ASoC menu
>   ASoC: tegra: Standardize ASoC menu
>   ASoC: ux500: Standardize ASoC menu
>   ASoC: cirrus: Standardize ASoC menu
>   ASoC: google: Standardize ASoC menu
>   ASoC: jz4740: Standardize ASoC menu
>   ASoC: xilinx: Standardize ASoC menu
>   ASoC: xtensa: Standardize ASoC menu
>   ASoC: renesas: Standardize ASoC menu
>   ASoC: generic: Standardize ASoC menu
>   ASoC: kirkwood: Standardize ASoC menu
>   ASoC: loongson: Standardize ASoC menu
>   ASoC: mediatek: Standardize ASoC menu
>   ASoC: rockchip: Standardize ASoC menu
>   ASoC: starfive: Standardize ASoC menu
>   ASoC: uniphier: Standardize ASoC menu
>   ASoC: hisilicon: Standardize ASoC menu
>   ASoC: sdw_utils: Standardize ASoC menu
> 
>  sound/soc/Kconfig           |  7 ++-----
>  sound/soc/adi/Kconfig       |  9 +++------
>  sound/soc/amd/Kconfig       |  3 +++
>  sound/soc/apple/Kconfig     |  4 ++++
>  sound/soc/atmel/Kconfig     | 11 ++---------
>  sound/soc/au1x/Kconfig      |  4 ++++
>  sound/soc/bcm/Kconfig       |  4 ++++
>  sound/soc/cirrus/Kconfig    |  3 +++
>  sound/soc/dwc/Kconfig       |  3 +++
>  sound/soc/fsl/Kconfig       |  2 +-
>  sound/soc/generic/Kconfig   |  4 ++++
>  sound/soc/google/Kconfig    |  3 +++
>  sound/soc/hisilicon/Kconfig |  4 ++++
>  sound/soc/img/Kconfig       | 13 +++----------
>  sound/soc/intel/Kconfig     |  4 ++++
>  sound/soc/jz4740/Kconfig    |  4 ++++
>  sound/soc/kirkwood/Kconfig  |  3 +++
>  sound/soc/loongson/Kconfig  |  2 +-
>  sound/soc/mediatek/Kconfig  |  4 ++++
>  sound/soc/mxs/Kconfig       |  2 +-
>  sound/soc/pxa/Kconfig       |  4 ++++
>  sound/soc/renesas/Kconfig   |  2 +-
>  sound/soc/rockchip/Kconfig  | 23 ++++++++---------------
>  sound/soc/sdca/Kconfig      |  3 +++
>  sound/soc/sdw_utils/Kconfig |  4 ++++
>  sound/soc/sof/Kconfig       |  2 +-
>  sound/soc/spear/Kconfig     |  4 ++++
>  sound/soc/sprd/Kconfig      |  4 ++++
>  sound/soc/starfive/Kconfig  | 12 ++++--------
>  sound/soc/sti/Kconfig       |  2 +-
>  sound/soc/stm/Kconfig       |  2 +-
>  sound/soc/sunxi/Kconfig     |  2 +-
>  sound/soc/tegra/Kconfig     |  4 ++++
>  sound/soc/ti/Kconfig        |  2 +-
>  sound/soc/uniphier/Kconfig  | 14 +++-----------
>  sound/soc/ux500/Kconfig     |  2 +-
>  sound/soc/xilinx/Kconfig    |  4 ++++
>  sound/soc/xtensa/Kconfig    |  4 ++++
>  38 files changed, 113 insertions(+), 74 deletions(-)
> 

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Mark Brown]

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

On Tue, Jun 10, 2025 at 11:51:41AM +0300, Péter Ujfalusi wrote:

> This is how the menu looks like after applying this series on x64:

> --- ALSA for SoC audio support
>       ASoC support for Analog Devices  --->
>       ASoC support for AMD  --->
>       ASoC support for Apple  ----
>       ASoC support for Atmel  --->
>       ASoC support for Au1x  ----

> We have quite menu empty sub-menus, partially because they would include
> something which would not be select-able in this arch (Apple, Au1x, etc)
> and things that does not even have select-able options (SoundWire things
> for example).

It'd be good if we could hide those, indeed.

> I personally would prefer to keep the 'Sound Open Firmware Support' as
> it was, probably moved above the CODEC drivers. SOF is more like a small
> subsystem within ASoC and the stack supports various vendors (Intel,
> NXP, AMD, Mediatek atm).

> The other awkward thing with this change (which was awkward before, but
> going to be double awkwarded) is that the Intel Machine drivers will be
> quite detached from SOF, not visible of hinted that they are now under
> "ASoC support for Intel"

That does make things in line with other platforms, though (as you
mention later).  The issue I guess is to an extent that people might not
know that SOF is important for a given platform, prehaps it should be
selected by platforms that need it similarly to what we do with CODEC
drivers?

> The other thing which made me wonder is the 'ASoC support for' repeated,
> would not be simpler:

> --- ALSA for SoC audio support
>       Analog Devices  --->
>       AMD  --->
>       Apple  ----
> ...

> The title already tells that this menu is for ASoC.

I don't have a strong opinion on that one.

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

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Péter Ujfalusi]

On 10/06/2025 14:07, Mark Brown wrote:
> On Tue, Jun 10, 2025 at 11:51:41AM +0300, Péter Ujfalusi wrote:
>> We have quite menu empty sub-menus, partially because they would include
>> something which would not be select-able in this arch (Apple, Au1x, etc)
>> and things that does not even have select-able options (SoundWire things
>> for example).
> 
> It'd be good if we could hide those, indeed.
> 
>> I personally would prefer to keep the 'Sound Open Firmware Support' as
>> it was, probably moved above the CODEC drivers. SOF is more like a small
>> subsystem within ASoC and the stack supports various vendors (Intel,
>> NXP, AMD, Mediatek atm).

With some internal discussion about the menu entry name change for SOF,
we concluded that it would be better to use the following to imply that
it encompasses multiple vendors and platforms:

'Support for Sound Open Firmware (SOF) platforms'

>> The other awkward thing with this change (which was awkward before, but
>> going to be double awkwarded) is that the Intel Machine drivers will be
>> quite detached from SOF, not visible of hinted that they are now under
>> "ASoC support for Intel"
> 
> That does make things in line with other platforms, though (as you
> mention later).  The issue I guess is to an extent that people might not
> know that SOF is important for a given platform, prehaps it should be
> selected by platforms that need it similarly to what we do with CODEC
> drivers?

At least the match modules are selected from SOF stack and if I'm not
mistaken some or most machine driver will be selected as well, likely no
need for users to hunt down the needed one.

But this is the reverse I guess than what you suggest, The platform
selects the machine driver and not the machine driver the platform(s).
We have generic machine drivers for SOF, used on several platforms (HDA,
SDW as main examples).
Probably this can be reversed, but touching Kconfing never goes without
regression ;)

Apart from this, we also discussed that from our side it is OK to move
Intel SST, AVS (and it's machine drivers) and the 'Intel Machine
drivers' under ASoC support for Intel.
This Intel submenu is going to be a bit unorganized as it will contain
only a portion of the drivers for Intel hardware and some of the
platform drivers (SST/AVS), but we don't want to move things around at
this point.

> 
>> The other thing which made me wonder is the 'ASoC support for' repeated,
>> would not be simpler:
> 
>> --- ALSA for SoC audio support
>>       Analog Devices  --->
>>       AMD  --->
>>       Apple  ----
>> ...
> 
>> The title already tells that this menu is for ASoC.
> 
> I don't have a strong opinion on that one.

I'm not sure about this now, probably it is better to keep the 'ASoC
support for' in case one searches for symbols? But that will show t*he
path to the menu entry, indicating that it is ASoC.
GPIO for example adds a postfix, some subsystems do not add the
subsystem name in entries.

I guess whichever looks better?

-- 
Péter

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Mark Brown]

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

On Tue, Jun 10, 2025 at 03:35:09PM +0300, Péter Ujfalusi wrote:
> On 10/06/2025 14:07, Mark Brown wrote:
> > On Tue, Jun 10, 2025 at 11:51:41AM +0300, Péter Ujfalusi wrote:

> >> The other awkward thing with this change (which was awkward before, but
> >> going to be double awkwarded) is that the Intel Machine drivers will be
> >> quite detached from SOF, not visible of hinted that they are now under
> >> "ASoC support for Intel"

> > That does make things in line with other platforms, though (as you
> > mention later).  The issue I guess is to an extent that people might not
> > know that SOF is important for a given platform, prehaps it should be
> > selected by platforms that need it similarly to what we do with CODEC
> > drivers?

> At least the match modules are selected from SOF stack and if I'm not
> mistaken some or most machine driver will be selected as well, likely no
> need for users to hunt down the needed one.

> But this is the reverse I guess than what you suggest, The platform
> selects the machine driver and not the machine driver the platform(s).

Yes, that - I know I've got whatever machine sitting in front of me.

> We have generic machine drivers for SOF, used on several platforms (HDA,
> SDW as main examples).

Those are a bit different (like the generic DT cards).

> Probably this can be reversed, but touching Kconfing never goes without
> regression ;)

Indeed, it's error prone.

> >> The title already tells that this menu is for ASoC.

> > I don't have a strong opinion on that one.

> I'm not sure about this now, probably it is better to keep the 'ASoC
> support for' in case one searches for symbols? But that will show t*he
> path to the menu entry, indicating that it is ASoC.
> GPIO for example adds a postfix, some subsystems do not add the
> subsystem name in entries.

> I guess whichever looks better?

Yeah, the reason I don't have a strong opinion is that I mostly edit
.config with 'vi .config', people who actually use the UIs will have
a relevant opinion here but I don't really.

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

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Kuninori Morimoto]

Hi Péter, Mark

Thank you for your review

> I personally would prefer to keep the 'Sound Open Firmware Support' as
> it was, probably moved above the CODEC drivers. SOF is more like a small
> subsystem within ASoC and the stack supports various vendors (Intel,
> NXP, AMD, Mediatek atm).
(snip)
> With some internal discussion about the menu entry name change for SOF,
> we concluded that it would be better to use the following to imply that
> it encompasses multiple vendors and platforms:
>
> 'Support for Sound Open Firmware (SOF) platforms'

OK, thank. So it will be like below in v2...

	ASoC support for ... --->
=>	Support for Sound Open Firmware (SOF) platforms --->
	CODEC drivers  --->

> We have quite menu empty sub-menus, partially because they would include
> something which would not be select-able in this arch (Apple, Au1x, etc)
> and things that does not even have select-able options (SoundWire things
> for example).

In my quick check, for example Apple case, it has only 1 item, I think
top menu can use "depends on" and we can hide it for non ARCH_APPLE user.

But for example Au1x case, it has many items and complex dependency,
so having "depends on" on top menu is maybe difficult or maybe can do
via Au1x developer ? (Because I'm not 100% sure about detail dependency)

> > >> The other awkward thing with this change (which was awkward before, but
> > >> going to be double awkwarded) is that the Intel Machine drivers will be
> > >> quite detached from SOF, not visible of hinted that they are now under
> > >> "ASoC support for Intel"
(snip)
> Apart from this, we also discussed that from our side it is OK to move
> Intel SST, AVS (and it's machine drivers) and the 'Intel Machine
> drivers' under ASoC support for Intel.
> This Intel submenu is going to be a bit unorganized as it will contain
> only a portion of the drivers for Intel hardware and some of the
> platform drivers (SST/AVS), but we don't want to move things around at
> this point.

I'm not 100% sure detail of Intel dependency. So how about this ?
I will post 1st menu patch-set as v2. And your team will tidyup Intel
side details after that ? It can avoid my patch-bomb
I will post v2 if we can agree about this.

> > >> The title already tells that this menu is for ASoC.
> 
> > > I don't have a strong opinion on that one.
> 
> > I'm not sure about this now, probably it is better to keep the 'ASoC
> > support for' in case one searches for symbols? But that will show t*he
> > path to the menu entry, indicating that it is ASoC.
> > GPIO for example adds a postfix, some subsystems do not add the
> > subsystem name in entries.
> 
> > I guess whichever looks better?
> 
> Yeah, the reason I don't have a strong opinion is that I mostly edit
> .config with 'vi .config', people who actually use the UIs will have
> a relevant opinion here but I don't really.

I have no strong opinion. Let's remove "ASoC subsystem for" prefix in v2.

Thank you for your help !!

Best regards
---
Kuninori Morimoto

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Péter Ujfalusi]

Hi Morimoto-san,

On 11/06/2025 08:40, Kuninori Morimoto wrote:
>> 'Support for Sound Open Firmware (SOF) platforms'
> 
> OK, thank. So it will be like below in v2...
> 
> 	ASoC support for ... --->
> =>	Support for Sound Open Firmware (SOF) platforms --->
> 	CODEC drivers  --->

Yes, this looks better, thank you.

>> We have quite menu empty sub-menus, partially because they would include
>> something which would not be select-able in this arch (Apple, Au1x, etc)
>> and things that does not even have select-able options (SoundWire things
>> for example).
> 
> In my quick check, for example Apple case, it has only 1 item, I think
> top menu can use "depends on" and we can hide it for non ARCH_APPLE user.

But does it make sense to add a submenu for a single entry?

> But for example Au1x case, it has many items and complex dependency,
> so having "depends on" on top menu is maybe difficult or maybe can do
> via Au1x developer ? (Because I'm not 100% sure about detail dependency)

It might be possible to aggregate them to show the menu entry if any of
the entries would be available for selection?

My other alter-ego is hiding the TI directory somehow...
>> Apart from this, we also discussed that from our side it is OK to move
>> Intel SST, AVS (and it's machine drivers) and the 'Intel Machine
>> drivers' under ASoC support for Intel.
>> This Intel submenu is going to be a bit unorganized as it will contain
>> only a portion of the drivers for Intel hardware and some of the
>> platform drivers (SST/AVS), but we don't want to move things around at
>> this point.
> 
> I'm not 100% sure detail of Intel dependency. So how about this ?
> I will post 1st menu patch-set as v2. And your team will tidyup Intel
> side details after that ? It can avoid my patch-bomb
> I will post v2 if we can agree about this.

Sure, we will figure out something for this. Better to keep this churn
as mechanical with as less functional change as possible.
>>>>> The title already tells that this menu is for ASoC.
>>
>>>> I don't have a strong opinion on that one.
>>
>>> I'm not sure about this now, probably it is better to keep the 'ASoC
>>> support for' in case one searches for symbols? But that will show t*he
>>> path to the menu entry, indicating that it is ASoC.
>>> GPIO for example adds a postfix, some subsystems do not add the
>>> subsystem name in entries.
>>
>>> I guess whichever looks better?
>>
>> Yeah, the reason I don't have a strong opinion is that I mostly edit
>> .config with 'vi .config', people who actually use the UIs will have
>> a relevant opinion here but I don't really.
> 
> I have no strong opinion. Let's remove "ASoC subsystem for" prefix in v2.

--- ALSA for SoC audio support
      ASoC support for Analog Devices  --->
      ASoC support for AMD  --->

VS

--- ALSA for SoC audio support
      Analog Devices  --->
      AMD  --->

At least for Intel (and probably for AMD/NXP/MTK) the later looks better
as not stating that the ASoC support for VendorX is _under_ this menu
item but you can find VendorX related options under that.

In context of SOF vs VendorX this translates better, imho.

-- 
Péter

Re: [PATCH 00/37] ASoC: Standardize ASoC menu

[Kuninori Morimoto]

Hi Péter

> > In my quick check, for example Apple case, it has only 1 item, I think
> > top menu can use "depends on" and we can hide it for non ARCH_APPLE user.
> 
> But does it make sense to add a submenu for a single entry?

Hmm...
Current Applie has single entry. We can replace it like below
("It is using "ASoC support for ..." for now)

	config SND_SOC_APPLE_MCA
-		tristate "Apple Silicon MCA driver"
+		tristate "ASoC support for Apple"
		...

But, apple people will confuse if they want to add more driver ?
Indeed it is not yet happen, but can be happen in the future.
So, I will keep current style in v2 (so far).

> > But for example Au1x case, it has many items and complex dependency,
> > so having "depends on" on top menu is maybe difficult or maybe can do
> > via Au1x developer ? (Because I'm not 100% sure about detail dependency)
> 
> It might be possible to aggregate them to show the menu entry if any of
> the entries would be available for selection?

Yeah, I think it is possible (from technical point of view)
But unfortunately, I'm not 100% sure detail of Au1x dependency.
So I don't know what kind of depends-on-tree is better (from politics point
of view)...

So, I hope Au1x member will update it after my patch-set.

> > I'm not 100% sure detail of Intel dependency. So how about this ?
> > I will post 1st menu patch-set as v2. And your team will tidyup Intel
> > side details after that ? It can avoid my patch-bomb
> > I will post v2 if we can agree about this.
> 
> Sure, we will figure out something for this. Better to keep this churn
> as mechanical with as less functional change as possible.

Thanks a lot !

> --- ALSA for SoC audio support
>       Analog Devices  --->
>       AMD  --->
> 
> At least for Intel (and probably for AMD/NXP/MTK) the later looks better
> as not stating that the ASoC support for VendorX is _under_ this menu
> item but you can find VendorX related options under that.
> 
> In context of SOF vs VendorX this translates better, imho.

Thank you for clearfy details.
Will do it in v2 !


Best regards
---
Kuninori Morimoto