Re: [PATCHv3 1/6] ALSA: pcm: Export soc_dpcm_runtime_update

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

 



On Tue, 25 Jul 2017 17:10:25 +0200,
Sebastian Reichel wrote:
> 
> From: Tony Lindgren <tony@xxxxxxxxxxx>
> 
> Some codecs may need to use this from loadable modules. Without
> this patch compilation for this kind of codec will fail with the
> following error:
> 
> ERROR: "soc_dpcm_runtime_update" [sound/soc/codecs/snd-soc-cpcap.ko]
> undefined!
> 
> Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
> Signed-off-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxxxx>

If you do export the function, please make its comments prettier, so
that it appears in kernel documentation properly as an API function.

Also, in general, we use snd_ prefix for the exported stuff.
You'd need to rename it accordingly, too.

Last but not least, this is specific to ASoC, so use "ASoC" prefix to
the subject line.


thanks,

Takashi

> ---
>  sound/soc/soc-pcm.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c
> index dcc5ece08668..4de7d86ea6fb 100644
> --- a/sound/soc/soc-pcm.c
> +++ b/sound/soc/soc-pcm.c
> @@ -2539,6 +2539,8 @@ int soc_dpcm_runtime_update(struct snd_soc_card *card)
>  	mutex_unlock(&card->mutex);
>  	return 0;
>  }
> +EXPORT_SYMBOL_GPL(soc_dpcm_runtime_update);
> +
>  int soc_dpcm_be_digital_mute(struct snd_soc_pcm_runtime *fe, int mute)
>  {
>  	struct snd_soc_dpcm *dpcm;
> -- 
> 2.13.2
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux