Re: [PATCH] ASoC: cs47l15: Ensure regmap_irq is built

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Feb 02, 2022 at 07:17:00PM +0000, Mark Brown wrote:
> The cs47l15 driver uses regmap_irq but does not ensure that it is built,
> causing build failures if nothing else being build requires the helpers.
> Add the appropriate select.
> 
> Signed-off-by: Mark Brown <broonie@xxxxxxxxxx>
> ---
>  sound/soc/codecs/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
> index 2fd21004688c..69fd11b2b046 100644
> --- a/sound/soc/codecs/Kconfig
> +++ b/sound/soc/codecs/Kconfig
> @@ -733,6 +733,7 @@ config SND_SOC_CS4349
>  
>  config SND_SOC_CS47L15
>  	tristate
> +	select REGMAP_IRQ
>  

Would it be better to have depends on MFD_CS47L15 instead,
that way the other dependencies would be covered as well? I think
these were added before implies was a thing, and thus
dependencies didn't matter, since they could only be selected
which ignored deps anyway.

I would expect L35, L85, L90, L92 to be affected as well. Happy
to do a patch to fix this up, if you prefer?

Thanks,
Charles



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux