lkml.org 
[lkml]   [2022]   [Feb]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] ASoC: SOF: pcm: Add compress_ops for SOF platform component driver
From


On 23/02/2022 17:38, Daniel Baluta wrote:
> From: Daniel Baluta <daniel.baluta@nxp.com>
>
> Now that sof_compressed_ops initial implementation was merged
> we can enable it in SOF platform component driver.

and fixes the following sparse error:
sound/soc/sof/compress.c:310:25: error: symbol 'sof_compressed_ops' was
not declared. Should it be static?

> This partially reverts commit
> 8a720724589e ("ASoC: SOF: pcm: Remove non existent CONFIG_SND_SOC_SOF_COMPRESS reference")

Back at that time there were _no_ sof_compressed_ops in the tree...

> Reported-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
> Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
> ---
> sound/soc/sof/pcm.c | 4 ++++
> sound/soc/sof/sof-priv.h | 5 +++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/sound/soc/sof/pcm.c b/sound/soc/sof/pcm.c
> index 137f8ed71677..a312ed855f1a 100644
> --- a/sound/soc/sof/pcm.c
> +++ b/sound/soc/sof/pcm.c
> @@ -922,6 +922,10 @@ void snd_sof_new_platform_drv(struct snd_sof_dev *sdev)
> pd->pointer = sof_pcm_pointer;
> pd->ack = sof_pcm_ack;
>
> +#if IS_ENABLED(CONFIG_SND_SOC_SOF_COMPRESS)
> + pd->compress_ops = &sof_compressed_ops;
> +#endif
> +
> pd->pcm_construct = sof_pcm_new;
> pd->ignore_machine = drv_name;
> pd->be_hw_params_fixup = sof_pcm_dai_link_fixup;
> diff --git a/sound/soc/sof/sof-priv.h b/sound/soc/sof/sof-priv.h
> index 2c8e556cd5cc..886787a9997f 100644
> --- a/sound/soc/sof/sof-priv.h
> +++ b/sound/soc/sof/sof-priv.h
> @@ -556,6 +556,11 @@ int snd_sof_debugfs_add_region_item_iomem(struct snd_sof_dev *sdev,
> enum snd_sof_fw_blk_type blk_type, u32 offset, size_t size,
> const char *name, enum sof_debugfs_access_type access_type);
>
> +/*
> + * Platform specific ops.
> + */
> +extern struct snd_compress_ops sof_compressed_ops;

This is not really platform ops?

> +
> /*
> * DSP Architectures.
> */

--
Péter

\
 
 \ /
  Last update: 2022-02-23 17:18    [W:0.031 / U:0.188 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site