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

["Péter Ujfalusi" <peter.ujfalusi@linux.intel.com>]

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