linux-kernel - Re: [PATCH 1/2] arm64: defconfig: Switch SOUND to module

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <b8783289-6670-4254-9049-786ae7cdd6a1@app.fastmail.com>
Date: Thu, 12 Jun 2025 17:05:02 +0200
From: "Arnd Bergmann" <arnd@...db.de>
To: "Krzysztof Kozlowski" <krzysztof.kozlowski@...aro.org>,
 "Krzysztof Kozlowski" <krzk@...nel.org>
Cc: "Geert Uytterhoeven" <geert@...ux-m68k.org>,
 linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
 "Sven Peter" <sven@...nel.org>, "Janne Grunau" <j@...nau.net>,
 asahi@...ts.linux.dev, linux-samsung-soc@...r.kernel.org
Subject: Re: [PATCH 1/2] arm64: defconfig: Switch SOUND to module

On Thu, Jun 12, 2025, at 15:44, Krzysztof Kozlowski wrote:
> Sound drivers are not essential to boot boards or mount rootfs,
> therefore in effort to reduce the size of kernel image (and boot images)
> switch the ASoC drivers to modules to decrease the size:
>
>   vmlinux: 152864 kB -> 154528 kB
>   Image: 39391 kB -> 39067 kB
>
> No difference in resulting include/generated/autoconf.h, except making
> modules: SND_SOC_SAMSUNG, SND_SOC_SDCA_OPTIONAL, SND_SOC_APPLE_MCA,
> SND_TIMER, SND_COMPRESS_OFFLOAD, SND_PCM, SND_SOC_SOF_OF and
> SND_DMAENGINE_PCM.
>
> Suggested-by: Geert Uytterhoeven <geert@...ux-m68k.org>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>
> ---
>
> If patches are fine, I will take them via Samsung SoC.

Nice find!

Acked-by: Arnd Bergmann <arnd@...db.de>

There are some nasty interactions between DRM drivers calling
into sound drivers for HDMI audio, but I checked that this is fine
here because of CONFIG_DRM=m.