--- Begin Message ---
- To: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>, alsa-devel@xxxxxxxxxxxxxxxx
- Subject: Re: [PATCH v2] ASoC: max98090: make it selectable
- From: Maxim Kochetkov <fido_max@xxxxxxxx>
- Date: Mon, 24 Apr 2023 08:07:46 +0300
- Cc: lgirdwood@xxxxxxxxx, broonie@xxxxxxxxxx, tiwai@xxxxxxxx, pierre-louis.bossart@xxxxxxxxxxxxxxx, rf@xxxxxxxxxxxxxxxxxxxxx, ckeepax@xxxxxxxxxxxxxxxxxxxxx, shumingf@xxxxxxxxxxx, ryans.lee@xxxxxxxxxx, herve.codina@xxxxxxxxxxx, wangweidong.a@xxxxxxxxxx, ajye_huang@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
- In-reply-to: <0a2eca07-f266-164c-4a9f-f1000ae4efbc@linaro.org>
- References: <20230421211950.20972-1-fido_max@inbox.ru> <0a2eca07-f266-164c-4a9f-f1000ae4efbc@linaro.org>
- User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0
On 23.04.2023 11:22, Krzysztof Kozlowski wrote:
On 21/04/2023 23:19, Maxim Kochetkov wrote:
There is no way to select max98090 from menuconfig right now.
Add a Kconfig menu description to allow building the driver standalone.
It will allow this codec to be used by any other I2S master without
adding extra sound card entry in Kconfig.
For your sound card, you need extra sound card entry in Kconfig.
Ok. But why do we allow some other codecs to be selectable?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.3-rc7&id=24ae67c582500
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.3-rc7&id=99afc8df8b6f2
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.3-rc7&id=3ac9ce8858ae3
--- End Message ---